Bug 31481

Summary:	lift the ban from one letter names, like E or R.
Product:	Sisyphus	Reporter:	viy <viy>
Component:	sisyphus_check	Assignee:	Dmitry V. Levin <ldv>
Status:	CLOSED FIXED	QA Contact:	qa-sisyphus
Severity:	normal
Priority:	P3	CC:	at, glebfm, imz, ldv, legion, placeholder
Version:	unstable
Hardware:	all
OS:	Linux

Description

viy 2015-11-13 02:06:32 MSK

у меня в autoimports сломалось обновление некоторых пакетов, оказалось,
их стал банить sisyphus_check.
однако для таких пакетов можно добавить в regexp еще одну ветвь
и банить тогда ничего не нужно. см. патч.

/people/viy/packages/sisyphus_check.git
commit df94ca43517f33bbd806a8d8e51a662b3248f61f
Author: Igor Vlasenko <viy@altlinux.org>
Date:   Fri Nov 13 00:53:55 2015 +0200

    lift the ban from one letter names, like E or R.
diff --git a/sisyphus_check/sisyphus_check.d/090-check-nvr b/sisyphus_check/sisy
index 2cbe688..fadce02 100644
--- a/sisyphus_check/sisyphus_check.d/090-check-nvr
+++ b/sisyphus_check/sisyphus_check.d/090-check-nvr
@@ -11,7 +11,7 @@ check_nvr()
        local f="$1" && shift || return 1
        local rc=0
 
-       if ! printf %s "$rpm_name" |grep -Exqs '[A-Za-z0-9][A-Za-z0-9._+-]*[A-Za
+       if ! printf %s "$rpm_name" |grep -Exqs '([A-Za-z0-9]|[A-Za-z0-9][A-Za-z0
                FileError "invalid package NAME: $rpm_name" "$f"
                rc=1
        fi

Comment 1

Repository Robot 2015-11-13 13:12:04 MSK

sisyphus_check-0.8.40-alt1 -> sisyphus:

* Fri Nov 13 2015 Dmitry V. Levin <ldv@altlinux> 0.8.40-alt1
- 090-check-nvr (check_nvr):
  + allowed single symbol package names (closes: #28834, #31481).