1 |
On Sun, Mar 31, 2013 at 02:31:45AM +0000, Jorge Manuel B. S. Vicetto (jmbsvicetto) wrote: |
2 |
> diff --git a/catalyst b/catalyst |
3 |
> index 9f8dd30..f885dd4 100755 |
4 |
> --- a/catalyst |
5 |
> +++ b/catalyst |
6 |
> @@ -147,6 +147,9 @@ def parse_config(myconfig): |
7 |
> if "pkgcache" in string.split(conf_values["options"]): |
8 |
> print "Package cache support enabled." |
9 |
> conf_values["PKGCACHE"]="1" |
10 |
> + if "preserve_libs" in string.split(conf_values["options"]); |
11 |
> + print "Preserving libs during unmerge." |
12 |
> + conf_values["PRESERVE_LIBS"]="1" |
13 |
> |
14 |
> if "purge" in string.split(conf_values["options"]): |
15 |
> print "Purge support enabled." |
16 |
> diff --git a/files/catalyst.conf b/files/catalyst.conf |
17 |
> index f88f105..2983acc 100644 |
18 |
> --- a/files/catalyst.conf |
19 |
> +++ b/files/catalyst.conf |
20 |
> @@ -59,6 +59,8 @@ hash_function="crc32" |
21 |
> # build stops in livecd-stage2) |
22 |
> # pkgcache = keeps a tbz2 of every built package (useful if your build stops |
23 |
> # prematurely) |
24 |
> +# preserve_libs = enables portage to preserve used libs when unmerging packages |
25 |
> +# (used on installcd-stage2 and stage4 targets) |
26 |
> # seedcache = use the build output of a previous target if it exists to speed up |
27 |
> # the copy |
28 |
> # snapcache = cache the snapshot so that it can be bind-mounted into the chroot. |
29 |
|
30 |
Your whitespace looks inconsistent. You're missing a blank line |
31 |
before `if "preserve_libs" …`, and you've got some extra space in the |
32 |
wrapped catalyst.conf comment. It looks like the existing |
33 |
catalyst.conf comments aren't particularly consistent, but the two |
34 |
existing choices seem to be `#<TAB><COMMENT>` and |
35 |
`#<SPACE><TAB><COMMENT>`. |
36 |
|
37 |
Other than that, I like this patch. |
38 |
|
39 |
Cheers, |
40 |
Trevor |
41 |
|
42 |
-- |
43 |
This email may be signed or encrypted with GnuPG (http://www.gnupg.org). |
44 |
For more information, see http://en.wikipedia.org/wiki/Pretty_Good_Privacy |