diff options
author | Zac Medico <zmedico@gentoo.org> | 2008-12-08 03:24:55 +0000 |
---|---|---|
committer | Zac Medico <zmedico@gentoo.org> | 2008-12-08 03:24:55 +0000 |
commit | ecefa1d9eb50e9b38d209d2b27d68625c1f8f61d (patch) | |
tree | 82a223e4784a6bba02a2c08b43691d07a8c7f0c8 | |
parent | 69dda52ded834f4664ce33c1b1f369517196100c (diff) | |
download | portage-ecefa1d9eb50e9b38d209d2b27d68625c1f8f61d.tar.gz portage-ecefa1d9eb50e9b38d209d2b27d68625c1f8f61d.tar.bz2 portage-ecefa1d9eb50e9b38d209d2b27d68625c1f8f61d.zip |
Bug #250212 - Add a new 'upstream.workaround' qa category, and use it for
the EMakeParallelDisabled check. Thanks to Mike Auty <ikelos@g.o> for this
patch.
svn path=/main/trunk/; revision=12179
-rwxr-xr-x | bin/repoman | 4 | ||||
-rw-r--r-- | man/repoman.1 | 4 | ||||
-rw-r--r-- | pym/repoman/checks.py | 2 | ||||
-rw-r--r-- | pym/repoman/errors.py | 2 |
4 files changed, 9 insertions, 3 deletions
diff --git a/bin/repoman b/bin/repoman index 9dc6ed6c9..fc249250a 100755 --- a/bin/repoman +++ b/bin/repoman @@ -314,7 +314,8 @@ qahelp={ "virtual.versioned":"PROVIDE contains virtuals with versions", "virtual.exists":"PROVIDE contains existing package names", "virtual.unavailable":"PROVIDE contains a virtual which contains no profile default", - "usage.obsolete":"The ebuild makes use of an obsolete construct" + "usage.obsolete":"The ebuild makes use of an obsolete construct", + "upstream.workaround":"The ebuild works around an upstream bug, an upstream bug should be filed and tracked in bugs.gentoo.org" } qacats = qahelp.keys() @@ -350,6 +351,7 @@ qawarnings = set(( "virtual.exists", "virtual.unavailable", "usage.obsolete", +"upstream.workaround", "LIVEVCS.stable" )) diff --git a/man/repoman.1 b/man/repoman.1 index c6d83d5cd..dde0d15d2 100644 --- a/man/repoman.1 +++ b/man/repoman.1 @@ -263,6 +263,10 @@ Bad metadata.xml files .B metadata.missing Missing metadata.xml files .TP +.B upstream.workaround +The ebuild works around an upstream bug, an upstream bug should be filed and +tracked in bugs.gentoo.org +.TP .B usage.obsolete The ebuild makes use of an obsolete construct .TP diff --git a/pym/repoman/checks.py b/pym/repoman/checks.py index c9415b759..c90b54f7b 100644 --- a/pym/repoman/checks.py +++ b/pym/repoman/checks.py @@ -291,7 +291,7 @@ class IUseUndefined(LineCheck): class EMakeParallelDisabled(LineCheck): """Check for emake -j1 calls which disable parallelization.""" - repoman_check_name = 'ebuild.minorsyn' + repoman_check_name = 'upstream.workaround' re = re.compile(r'^\s*emake\s+-j\s*1\s') error = errors.EMAKE_PARALLEL_DISABLED diff --git a/pym/repoman/errors.py b/pym/repoman/errors.py index 7c54cc13d..7f45bea21 100644 --- a/pym/repoman/errors.py +++ b/pym/repoman/errors.py @@ -13,5 +13,5 @@ MISSING_QUOTES_ERROR = 'Unquoted Variable on line: %d' NESTED_DIE_ERROR = 'Ebuild calls die in a subshell on line: %d' PATCHES_ERROR = 'PATCHES is not a bash array on line: %d' REDUNDANT_CD_S_ERROR = 'Ebuild has redundant cd ${S} statement on line: %d' -EMAKE_PARALLEL_DISABLED = 'Ebuild calls emake -j1 on line: %d' +EMAKE_PARALLEL_DISABLED = 'Upstream parallel compilation bug (ebuild calls emake -j1 on line: %d)' DEPRECATED_BINDNOW_FLAGS = 'Deprecated bindnow-flags call on line: %d' |