Gentoo Archives: gentoo-portage-dev

From: Mike Gilbert <floppym@g.o>
To: gentoo-portage-dev@l.g.o
Cc: "Michał Górny" <mgorny@g.o>
Subject: Re: [gentoo-portage-dev] [PATCH 2/2] repoman: Enable testing exp profiles by default
Date: Thu, 11 Jan 2018 02:45:53
Message-Id: CAJ0EP40aHJcPYSzKpxUj_3rfQFb97vxd_fKPS6LsmygX630pww@mail.gmail.com
In Reply to: Re: [gentoo-portage-dev] [PATCH 2/2] repoman: Enable testing exp profiles by default by Zac Medico
1 On Wed, Jan 10, 2018 at 5:56 PM, Zac Medico <zmedico@g.o> wrote:
2 > On 01/10/2018 02:24 PM, Michał Górny wrote:
3 >> Enable repoman checks on exp profiles by default to improve
4 >> the dependency graph integrity on those profiles and help them on their
5 >> way towards stable status. This is possible now that the dependency
6 >> graph problems are warnings rather than errors.
7 >> ---
8 >> repoman/pym/repoman/argparser.py | 2 +-
9 >> 1 file changed, 1 insertion(+), 1 deletion(-)
10 >>
11 >> diff --git a/repoman/pym/repoman/argparser.py b/repoman/pym/repoman/argparser.py
12 >> index f32972288..d49147366 100644
13 >> --- a/repoman/pym/repoman/argparser.py
14 >> +++ b/repoman/pym/repoman/argparser.py
15 >> @@ -167,7 +167,7 @@ def parse_args(argv, qahelp, repoman_default_opts):
16 >>
17 >> parser.add_argument(
18 >> '-e', '--include-exp-profiles', choices=('y', 'n'), metavar='<y|n>',
19 >> - default=False,
20 >> + default='y',
21 >> help='include exp profiles in dependency checks')
22 >>
23 >> parser.add_argument(
24 >>
25 >
26 > We have dev and exp profiles disabled by default because the time
27 > consumed by repoman is proportional to the number of profiles.
28 >
29 > The current counts are as follows:
30 >
31 > stable 87
32 > dev 88
33 > exp 149
34
35 Yeah, I really don't like the idea of making repoman even slower by default.
36
37 Also, I'm not sure why exp profiles are to be treated as more stable
38 than dev profiles. I was always under the impression that the reverse
39 was true, though I cannot recall where that assumption was formed.
40
41 I would like to see this formally documented somewhere before we start
42 changing repoman defaults.

Replies