From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from lists.gentoo.org (pigeon.gentoo.org [208.92.234.80]) by finch.gentoo.org (Postfix) with ESMTP id 895541393EB for ; Fri, 30 May 2014 13:03:35 +0000 (UTC) Received: from pigeon.gentoo.org (localhost [127.0.0.1]) by pigeon.gentoo.org (Postfix) with SMTP id 1E521E0888; Fri, 30 May 2014 13:03:35 +0000 (UTC) Received: from smtp.gentoo.org (smtp.gentoo.org [140.211.166.183]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by pigeon.gentoo.org (Postfix) with ESMTPS id 6770BE0888 for ; Fri, 30 May 2014 13:03:34 +0000 (UTC) Received: from spoonbill.gentoo.org (spoonbill.gentoo.org [81.93.255.5]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by smtp.gentoo.org (Postfix) with ESMTPS id 5898033FDC1 for ; Fri, 30 May 2014 13:03:33 +0000 (UTC) Received: from localhost.localdomain (localhost [127.0.0.1]) by spoonbill.gentoo.org (Postfix) with ESMTP id 0714B182D6 for ; Fri, 30 May 2014 13:03:32 +0000 (UTC) From: "Brian Dolbec" To: gentoo-commits@lists.gentoo.org Content-Transfer-Encoding: 8bit Content-type: text/plain; charset=UTF-8 Reply-To: gentoo-dev@lists.gentoo.org, "Brian Dolbec" Message-ID: <1401397059.ccaae712e8d42f5d9bbcc38195d8c072a7b9bc41.dol-sen@gentoo> Subject: [gentoo-commits] proj/portage:repoman commit in: pym/repoman/ X-VCS-Repository: proj/portage X-VCS-Files: pym/repoman/main.py X-VCS-Directories: pym/repoman/ X-VCS-Committer: dol-sen X-VCS-Committer-Name: Brian Dolbec X-VCS-Revision: ccaae712e8d42f5d9bbcc38195d8c072a7b9bc41 X-VCS-Branch: repoman Date: Fri, 30 May 2014 13:03:32 +0000 (UTC) Precedence: bulk List-Post: List-Help: List-Unsubscribe: List-Subscribe: List-Id: Gentoo Linux mail X-BeenThere: gentoo-commits@lists.gentoo.org X-Archives-Salt: 7f65744c-83e3-4804-add3-d12e09410371 X-Archives-Hash: cd72429a61ff5090f042984435eb050e commit: ccaae712e8d42f5d9bbcc38195d8c072a7b9bc41 Author: Brian Dolbec gentoo org> AuthorDate: Thu May 29 20:57:39 2014 +0000 Commit: Brian Dolbec gmail com> CommitDate: Thu May 29 20:57:39 2014 +0000 URL: http://git.overlays.gentoo.org/gitweb/?p=proj/portage.git;a=commit;h=ccaae712 repoman/main.py: Replace y with y_ebuild in the loop This too helps clarify what it is. --- pym/repoman/main.py | 36 ++++++++++++++++++------------------ 1 file changed, 18 insertions(+), 18 deletions(-) diff --git a/pym/repoman/main.py b/pym/repoman/main.py index 99238d9..ee70735 100755 --- a/pym/repoman/main.py +++ b/pym/repoman/main.py @@ -746,9 +746,9 @@ for xpkg in effective_scanlist: # detect unused local USE-descriptions used_useflags = set() - for y in ebuildlist: + for y_ebuild in ebuildlist: ################## - ebuild = Ebuild(repo_settings, repolevel, pkgdir, catdir, vcs_settings, xpkg, y) + ebuild = Ebuild(repo_settings, repolevel, pkgdir, catdir, vcs_settings, xpkg, y_ebuild) ################## if check_changelog and not changelog_modified \ @@ -756,11 +756,11 @@ for xpkg in effective_scanlist: stats['changelog.ebuildadded'] += 1 fails['changelog.ebuildadded'].append(ebuild.relative_path) - if ebuild.untracked(check_ebuild_notadded, y, eadded): + if ebuild.untracked(check_ebuild_notadded, y_ebuild, eadded): # ebuild not added to vcs stats["ebuild.notadded"] += 1 - fails["ebuild.notadded"].append(xpkg + "/" + y + ".ebuild") - myesplit = portage.pkgsplit(y) + fails["ebuild.notadded"].append(xpkg + "/" + y_ebuild + ".ebuild") + myesplit = portage.pkgsplit(y_ebuild) is_bad_split = myesplit is None or myesplit[0] != xpkg.split("/")[-1] @@ -770,15 +770,15 @@ for xpkg in effective_scanlist: if is_pv_toolong or is_pv_toolong2: stats["ebuild.invalidname"] += 1 - fails["ebuild.invalidname"].append(xpkg + "/" + y + ".ebuild") + fails["ebuild.invalidname"].append(xpkg + "/" + y_ebuild + ".ebuild") continue elif myesplit[0] != pkgdir: print(pkgdir, myesplit[0]) stats["ebuild.namenomatch"] += 1 - fails["ebuild.namenomatch"].append(xpkg + "/" + y + ".ebuild") + fails["ebuild.namenomatch"].append(xpkg + "/" + y_ebuild + ".ebuild") continue - pkg = pkgs[y] + pkg = pkgs[y_ebuild] if pkg.invalid: allvalid = False @@ -834,7 +834,7 @@ for xpkg in effective_scanlist: continue myqakey = missingvars[pos] + ".missing" stats[myqakey] += 1 - fails[myqakey].append(xpkg + "/" + y + ".ebuild") + fails[myqakey].append(xpkg + "/" + y_ebuild + ".ebuild") if catdir == "virtual": for var in ("HOMEPAGE", "LICENSE"): @@ -868,7 +868,7 @@ for xpkg in effective_scanlist: stats["KEYWORDS.stable"] += 1 fails["KEYWORDS.stable"].append( "%s/%s.ebuild added with stable keywords: %s" % - (xpkg, y, " ".join(stable_keywords))) + (xpkg, y_ebuild, " ".join(stable_keywords))) ebuild_archs = set( kw.lstrip("~") for kw in keywords if not kw.startswith("-")) @@ -897,7 +897,7 @@ for xpkg in effective_scanlist: haskeyword = True if not haskeyword: stats["KEYWORDS.stupid"] += 1 - fails["KEYWORDS.stupid"].append(xpkg + "/" + y + ".ebuild") + fails["KEYWORDS.stupid"].append(xpkg + "/" + y_ebuild + ".ebuild") """ Ebuilds that inherit a "Live" eclass (darcs,subversion,git,cvs,etc..) should @@ -914,7 +914,7 @@ for xpkg in effective_scanlist: stats["LIVEVCS.stable"] += 1 fails["LIVEVCS.stable"].append( "%s/%s.ebuild with stable keywords:%s " % - (xpkg, y, bad_stable_keywords)) + (xpkg, y_ebuild, bad_stable_keywords)) del bad_stable_keywords if keywords and not has_global_mask(pkg): @@ -959,7 +959,7 @@ for xpkg in effective_scanlist: baddepsyntax = False badlicsyntax = False badprovsyntax = False - catpkg = catdir + "/" + y + catpkg = catdir + "/" + y_ebuild inherited_java_eclass = "java-pkg-2" in inherited or \ "java-pkg-opt-2" in inherited @@ -1086,7 +1086,7 @@ for xpkg in effective_scanlist: for mypos in range(len(myuse)): stats["IUSE.invalid"] += 1 - fails["IUSE.invalid"].append(xpkg + "/" + y + ".ebuild: %s" % myuse[mypos]) + fails["IUSE.invalid"].append(xpkg + "/" + y_ebuild + ".ebuild: %s" % myuse[mypos]) # Check for outdated RUBY targets old_ruby_eclasses = ["ruby-ng", "ruby-fakegem", "ruby"] @@ -1112,7 +1112,7 @@ for xpkg in effective_scanlist: # function will remove it without removing values. if lic not in liclist and lic != "||": stats["LICENSE.invalid"] += 1 - fails["LICENSE.invalid"].append(xpkg + "/" + y + ".ebuild: %s" % lic) + fails["LICENSE.invalid"].append(xpkg + "/" + y_ebuild + ".ebuild: %s" % lic) elif lic in liclist_deprecated: stats["LICENSE.deprecated"] += 1 fails["LICENSE.deprecated"].append("%s: %s" % (ebuild.relative_path, lic)) @@ -1129,11 +1129,11 @@ for xpkg in effective_scanlist: if myskey not in kwlist: stats["KEYWORDS.invalid"] += 1 fails["KEYWORDS.invalid"].append( - "%s/%s.ebuild: %s" % (xpkg, y, mykey)) + "%s/%s.ebuild: %s" % (xpkg, y_ebuild, mykey)) elif myskey not in profiles: stats["KEYWORDS.invalid"] += 1 fails["KEYWORDS.invalid"].append( - "%s/%s.ebuild: %s (profile invalid)" % (xpkg, y, mykey)) + "%s/%s.ebuild: %s (profile invalid)" % (xpkg, y_ebuild, mykey)) # restrict checks myrestrict = None @@ -1151,7 +1151,7 @@ for xpkg in effective_scanlist: if mybadrestrict: stats["RESTRICT.invalid"] += len(mybadrestrict) for mybad in mybadrestrict: - fails["RESTRICT.invalid"].append(xpkg + "/" + y + ".ebuild: %s" % mybad) + fails["RESTRICT.invalid"].append(xpkg + "/" + y_ebuild + ".ebuild: %s" % mybad) # REQUIRED_USE check required_use = myaux["REQUIRED_USE"] if required_use: