From: Zac Medico <zmedico@gentoo.org>
Date: Wed, 11 Mar 2009 03:40:36 +0000 (-0000)
Subject: Warn about WANT_AUTO(CONF|MAKE)=latest settings since they are redundant.
X-Git-Tag: v2.1.6.8~213
X-Git-Url: http://git.tremily.us/?a=commitdiff_plain;h=49e640ef614ddd5c087457e158f6956a17fe0a26;p=portage.git

Warn about WANT_AUTO(CONF|MAKE)=latest settings since they are redundant.
Thanks to Petteri Räty <betelgeuse@g.o> for the initial patch. (trunk r12583)

svn path=/main/branches/2.1.6/; revision=12864
---

diff --git a/pym/repoman/checks.py b/pym/repoman/checks.py
index c9ef91cfc..e89020980 100644
--- a/pym/repoman/checks.py
+++ b/pym/repoman/checks.py
@@ -328,13 +328,25 @@ class DeprecatedBindnowFlags(LineCheck):
 	re = re.compile(r'.*\$\(bindnow-flags\)')
 	error = errors.DEPRECATED_BINDNOW_FLAGS
 
+class WantAutoDefaultValue(LineCheck):
+	"""Check setting WANT_AUTO* to latest (default value)."""
+	repoman_check_name = 'ebuild.minorsyn'
+	_re = re.compile(r'^WANT_AUTO(CONF|MAKE)=(\'|")?latest')
+
+	def check(self, num, line):
+		m = self._re.match(line)
+		if m is not None:
+			return 'WANT_AUTO' + m.group(1) + \
+				' redundantly set to default value "latest" on line: %d'
+
 _constant_checks = tuple((c() for c in (
 	EbuildHeader, EbuildWhitespace, EbuildQuote,
 	EbuildAssignment, EbuildUselessDodoc,
 	EbuildUselessCdS, EbuildNestedDie,
 	EbuildPatches, EbuildQuotedA,
 	IUseUndefined, ImplicitRuntimeDeps, InheritAutotools,
-	EMakeParallelDisabled, DeprecatedBindnowFlags)))
+	EMakeParallelDisabled, DeprecatedBindnowFlags,
+	WantAutoDefaultValue)))
 
 def run_checks(contents, pkg):
 	checks = _constant_checks