From b0b3fdd9b572d9341e0456c7e1e241c795929224 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Micha=C5=82=20G=C3=B3rny?= Date: Thu, 26 Aug 2010 18:38:52 +0200 Subject: Disable 'unused local USE-description' check if there are any invalids. If any of the ebuilds is considered invalid, the used_useflags list may not be complete. Thus, performing the test could cause a lot of useless noise. --- bin/repoman | 14 +++++++++----- 1 file changed, 9 insertions(+), 5 deletions(-) (limited to 'bin') diff --git a/bin/repoman b/bin/repoman index 5c01c3696..86b4c1c46 100755 --- a/bin/repoman +++ b/bin/repoman @@ -1394,6 +1394,7 @@ for x in scanlist: changelog_path = os.path.join(checkdir_relative, "ChangeLog") changelog_modified = changelog_path in modified_changelogs + allvalid = True allmasked = True # detect unused local USE-descriptions used_useflags = set() @@ -1432,6 +1433,7 @@ for x in scanlist: pkg = pkgs[y] if pkg.invalid: + allvalid = False for k, msgs in pkg.invalid.items(): for msg in msgs: stats[k] = stats[k] + 1 @@ -1927,11 +1929,13 @@ for x in scanlist: fails["ebuild.allmasked"].append(x) # check if there are unused local USE-descriptions in metadata.xml - for myflag in muselist.difference(used_useflags): - stats["metadata.warning"] += 1 - fails["metadata.warning"].append( - "%s/metadata.xml: unused local USE-description: '%s'" % \ - (x, myflag)) + # (unless there are any invalids, to avoid noise) + if allvalid: + for myflag in muselist.difference(used_useflags): + stats["metadata.warning"] += 1 + fails["metadata.warning"].append( + "%s/metadata.xml: unused local USE-description: '%s'" % \ + (x, myflag)) if options.mode == "manifest": sys.exit(dofail) -- cgit v1.2.3-1-g7c22