1 |
On Mon, 2018-12-17 at 21:09 -0500, Alec Warner wrote: |
2 |
> On Mon, Dec 17, 2018 at 10:51 AM Michał Górny <mgorny@g.o> wrote: |
3 |
> |
4 |
> > On Mon, 2018-12-17 at 15:44 +0000, M. J. Everitt wrote: |
5 |
> > > On 17/12/18 12:54, Michał Górny wrote: |
6 |
> > > > > Not only this, but as noted, unless you know the man pages for |
7 |
> > |
8 |
> > portage and |
9 |
> > > > > make.conf in order to recite them in your sleep, they are confusing |
10 |
> > |
11 |
> > for |
12 |
> > > > > users, as they do not necessarily follow an obvious pattern, and it |
13 |
> > |
14 |
> > wasn't |
15 |
> > > > > until I was attempting to debug something that I noticed that despite |
16 |
> > > > > believing I had the correct settings in my make.conf (set over a |
17 |
> > |
18 |
> > period of |
19 |
> > > > > YEARS) they were in fact completely useless, and it wasn't until I |
20 |
> > |
21 |
> > had to |
22 |
> > > > > spend time with somebody debugging WTF was happening, that this |
23 |
> > |
24 |
> > particular |
25 |
> > > > > issue even became apparent... |
26 |
> > > > |
27 |
> > > > I don't see how this is an argument for anything. You have to read |
28 |
> > > > the manual in order to know that such variable exists and what it |
29 |
> > |
30 |
> > does. |
31 |
> > > > Or, well, technically you don't since it's provided in |
32 |
> > |
33 |
> > make.conf.example |
34 |
> > > > already where you only need to uncomment it. |
35 |
> > > > |
36 |
> > > > Either way, the variable name is trivial. Even if you don't follow |
37 |
> > > > the usual pattern of uncommenting it from make.conf.example or copying |
38 |
> > > > from the manual, remembering it for the time needed to retype shoudln't |
39 |
> > > > be a problem. |
40 |
> > > > |
41 |
> > > > So, is this a solution to a real problem? Or is it merely a half- |
42 |
> > > > thought-out partial change that's going to require people to update |
43 |
> > > > their configuration for no long-term benefit? And then they will have |
44 |
> > > > to update it again when someone decides to take another variable for |
45 |
> > > > a spin. |
46 |
> > > > |
47 |
> > > |
48 |
> > > In the case you hadn't noticed, clearly you haven't .. the change is |
49 |
> > > backwards compatible.. that has already been thought out. |
50 |
> > > |
51 |
> > > But you haven't actually looked at the patch have you, Michal ? |
52 |
> > > |
53 |
> > |
54 |
> > I did look at it. However, that doesn't change what I said. Being |
55 |
> > 'backwards compatible' does not change the fact that the old variable |
56 |
> > becomes deprecated now. Ergo, users are expected to eventually switch |
57 |
> > to the new one. |
58 |
> > |
59 |
> |
60 |
> So if we rewrote the patches to not deprecate the old one but to support |
61 |
> both; how would that strike you? |
62 |
> |
63 |
|
64 |
As a horrible idea, violating SPOT rule? |
65 |
|
66 |
-- |
67 |
Best regards, |
68 |
Michał Górny |