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 925B6138247 for ; Fri, 17 Jan 2014 21:37:55 +0000 (UTC) Received: from pigeon.gentoo.org (localhost [127.0.0.1]) by pigeon.gentoo.org (Postfix) with SMTP id DF098E0A8F; Fri, 17 Jan 2014 21:37:53 +0000 (UTC) Received: from andre.telenet-ops.be (andre.telenet-ops.be [195.130.132.53]) by pigeon.gentoo.org (Postfix) with ESMTP id 2F360E0A8D for ; Fri, 17 Jan 2014 21:37:52 +0000 (UTC) Received: from localhost.localdomain ([94.226.55.127]) by andre.telenet-ops.be with bizsmtp id F9dr1n00s2khLEN019drVE; Fri, 17 Jan 2014 22:37:52 +0100 From: Tom Wijsman To: gentoo-portage-dev@lists.gentoo.org Subject: [gentoo-portage-dev] [PATCH 2/3 v2] Have repoman check that a package directory contains at least one ebuild (bug #245305). Date: Fri, 17 Jan 2014 22:36:54 +0100 Message-Id: <1389994614-7459-1-git-send-email-tomwij@gentoo.org> X-Mailer: git-send-email 1.8.5.2 In-Reply-To: <1389830840-25848-3-git-send-email-tomwij@gentoo.org> References: <1389830840-25848-3-git-send-email-tomwij@gentoo.org> Precedence: bulk List-Post: List-Help: List-Unsubscribe: List-Subscribe: List-Id: Gentoo Linux mail X-BeenThere: gentoo-portage-dev@lists.gentoo.org Reply-to: gentoo-portage-dev@lists.gentoo.org X-Archives-Salt: fe721675-f37b-4176-84a0-8a5b82608c65 X-Archives-Hash: bb24973c7273b47125b1fb6418516be0 --- bin/repoman | 8 ++++++++ man/repoman.1 | 3 +++ 2 files changed, 11 insertions(+) diff --git a/bin/repoman b/bin/repoman index d1542e9..44f3d3d 100755 --- a/bin/repoman +++ b/bin/repoman @@ -326,6 +326,7 @@ qahelp = { "SRC_URI.mirror": "A uri listed in profiles/thirdpartymirrors is found in SRC_URI", "ebuild.syntax": "Error generating cache entry for ebuild; typically caused by ebuild syntax error or digest verification failure", "ebuild.output": "A simple sourcing of the ebuild produces output; this breaks ebuild policy.", + "ebuild.missing": "A package directory must at least contain one ebuild or be treecleaned.", "ebuild.nesteddie": "Placing 'die' inside ( ) prints an error, but doesn't stop the ebuild.", "variable.invalidchar": "A variable contains an invalid character that is not part of the ASCII character set", "variable.readonly": "Assigning a readonly variable", @@ -1442,6 +1443,13 @@ for x in effective_scanlist: can_force = False continue + if not ebuildlist: + stats["ebuild.missing"] += 1 + fails["ebuild.missing"].append("%s must at least contain one " % x + \ + "ebuild or be treecleaned.") + can_force = False + continue + # Sort ebuilds in ascending order for the KEYWORDS.dropped check. ebuildlist = sorted(pkgs.values()) ebuildlist = [pkg.pf for pkg in ebuildlist] diff --git a/man/repoman.1 b/man/repoman.1 index a78f94e..6315ea9 100644 --- a/man/repoman.1 +++ b/man/repoman.1 @@ -301,6 +301,9 @@ Ebuilds that exist but have not been added to cvs .B ebuild.output A simple sourcing of the ebuild produces output; this breaks ebuild policy. .TP +.B ebuild.missing +A package directory must at least contain one ebuild or be treecleaned. +.TP .B ebuild.patches PATCHES variable should be a bash array to ensure white space safety .TP -- 1.8.5.2