1 |
On Thu, Jan 11, 2018 at 2:48 AM, Michał Górny <mgorny@g.o> wrote: |
2 |
> Dnia 11 stycznia 2018 07:54:40 CET, Mike Gilbert <floppym@g.o> napisał(a): |
3 |
>>On Wed, Jan 10, 2018 at 11:10 PM, Michał Górny <mgorny@g.o> |
4 |
>>wrote: |
5 |
>>> W dniu śro, 10.01.2018 o godzinie 21∶45 -0500, użytkownik Mike |
6 |
>>Gilbert |
7 |
>>> napisał: |
8 |
>>>> On Wed, Jan 10, 2018 at 5:56 PM, Zac Medico <zmedico@g.o> |
9 |
>>wrote: |
10 |
>>>> > On 01/10/2018 02:24 PM, Michał Górny wrote: |
11 |
>>>> > > Enable repoman checks on exp profiles by default to improve |
12 |
>>>> > > the dependency graph integrity on those profiles and help them |
13 |
>>on their |
14 |
>>>> > > way towards stable status. This is possible now that the |
15 |
>>dependency |
16 |
>>>> > > graph problems are warnings rather than errors. |
17 |
>>>> > > --- |
18 |
>>>> > > repoman/pym/repoman/argparser.py | 2 +- |
19 |
>>>> > > 1 file changed, 1 insertion(+), 1 deletion(-) |
20 |
>>>> > > |
21 |
>>>> > > diff --git a/repoman/pym/repoman/argparser.py |
22 |
>>b/repoman/pym/repoman/argparser.py |
23 |
>>>> > > index f32972288..d49147366 100644 |
24 |
>>>> > > --- a/repoman/pym/repoman/argparser.py |
25 |
>>>> > > +++ b/repoman/pym/repoman/argparser.py |
26 |
>>>> > > @@ -167,7 +167,7 @@ def parse_args(argv, qahelp, |
27 |
>>repoman_default_opts): |
28 |
>>>> > > |
29 |
>>>> > > parser.add_argument( |
30 |
>>>> > > '-e', '--include-exp-profiles', choices=('y', |
31 |
>>'n'), metavar='<y|n>', |
32 |
>>>> > > - default=False, |
33 |
>>>> > > + default='y', |
34 |
>>>> > > help='include exp profiles in dependency checks') |
35 |
>>>> > > |
36 |
>>>> > > parser.add_argument( |
37 |
>>>> > > |
38 |
>>>> > |
39 |
>>>> > We have dev and exp profiles disabled by default because the time |
40 |
>>>> > consumed by repoman is proportional to the number of profiles. |
41 |
>>>> > |
42 |
>>>> > The current counts are as follows: |
43 |
>>>> > |
44 |
>>>> > stable 87 |
45 |
>>>> > dev 88 |
46 |
>>>> > exp 149 |
47 |
>>>> |
48 |
>>>> Yeah, I really don't like the idea of making repoman even slower by |
49 |
>>default. |
50 |
>>> |
51 |
>>> The alternative is to go for all profiles being stable. Because |
52 |
>>> accepting developers randomly breaking profiles 'because repoman |
53 |
>>speed' |
54 |
>>> is not acceptable. |
55 |
>> |
56 |
>>I disagree. Many of these are profiles that are seldom used and very |
57 |
>>few users are affected by any breakage. |
58 |
>> |
59 |
>>There's a balance to be struck here, and I think it's pretty good |
60 |
>>right where it is. |
61 |
> |
62 |
> Just to be clear, the current counts are irrelevant. The goal is to have a second status that covers profiles on their way to becoming stable. This patch is part of that thread. |
63 |
|
64 |
I see. I have no objection to adding on a smaller number of profiles |
65 |
by default to help support this. |