1 |
On Fri, Jan 17, 2014 at 4:36 PM, Tom Wijsman <tomwij@g.o> wrote: |
2 |
|
3 |
> --- |
4 |
> bin/repoman | 8 ++++++++ |
5 |
> man/repoman.1 | 3 +++ |
6 |
> 2 files changed, 11 insertions(+) |
7 |
> |
8 |
> diff --git a/bin/repoman b/bin/repoman |
9 |
> index d1542e9..44f3d3d 100755 |
10 |
> --- a/bin/repoman |
11 |
> +++ b/bin/repoman |
12 |
> @@ -326,6 +326,7 @@ qahelp = { |
13 |
> "SRC_URI.mirror": "A uri listed in profiles/thirdpartymirrors is |
14 |
> found in SRC_URI", |
15 |
> "ebuild.syntax": "Error generating cache entry for ebuild; |
16 |
> typically caused by ebuild syntax error or digest verification failure", |
17 |
> "ebuild.output": "A simple sourcing of the ebuild produces output; |
18 |
> this breaks ebuild policy.", |
19 |
> + "ebuild.missing": "A package directory must at least contain one |
20 |
> ebuild or be treecleaned.", |
21 |
> "ebuild.nesteddie": "Placing 'die' inside ( ) prints an error, but |
22 |
> doesn't stop the ebuild.", |
23 |
> "variable.invalidchar": "A variable contains an invalid character |
24 |
> that is not part of the ASCII character set", |
25 |
> "variable.readonly": "Assigning a readonly variable", |
26 |
> @@ -1442,6 +1443,13 @@ for x in effective_scanlist: |
27 |
> can_force = False |
28 |
> continue |
29 |
> |
30 |
> + if not ebuildlist: |
31 |
> + stats["ebuild.missing"] += 1 |
32 |
> + fails["ebuild.missing"].append("%s must at least contain |
33 |
> one " % x + \ |
34 |
> + "ebuild or be treecleaned.") |
35 |
> + can_force = False |
36 |
> + continue |
37 |
> + |
38 |
> # Sort ebuilds in ascending order for the KEYWORDS.dropped check. |
39 |
> ebuildlist = sorted(pkgs.values()) |
40 |
> ebuildlist = [pkg.pf for pkg in ebuildlist] |
41 |
> diff --git a/man/repoman.1 b/man/repoman.1 |
42 |
> index a78f94e..6315ea9 100644 |
43 |
> --- a/man/repoman.1 |
44 |
> +++ b/man/repoman.1 |
45 |
> @@ -301,6 +301,9 @@ Ebuilds that exist but have not been added to cvs |
46 |
> .B ebuild.output |
47 |
> A simple sourcing of the ebuild produces output; this breaks ebuild |
48 |
> policy. |
49 |
> .TP |
50 |
> +.B ebuild.missing |
51 |
> +A package directory must at least contain one ebuild or be treecleaned. |
52 |
> +.TP |
53 |
> .B ebuild.patches |
54 |
> PATCHES variable should be a bash array to ensure white space safety |
55 |
> .TP |
56 |
> -- |
57 |
> 1.8.5.2 |
58 |
> |
59 |
> |
60 |
> |
61 |
Looks fine. |
62 |
|
63 |
-- |
64 |
Jesus Rivero (Neurogeek) |
65 |
Gentoo Developer |