Bug 31481 - lift the ban from one letter names, like E or R.
Summary: lift the ban from one letter names, like E or R.
Status: CLOSED FIXED
Alias: None
Product: Sisyphus
Classification: Development
Component: sisyphus_check (show other bugs)
Version: unstable
Hardware: all Linux
: P3 normal
Assignee: Dmitry V. Levin
QA Contact: qa-sisyphus
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2015-11-13 02:06 MSK by viy
Modified: 2015-11-13 13:12 MSK (History)
6 users (show)

See Also:

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
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).