1 |
Zac Medico posted on Sun, 28 Jan 2018 22:21:48 -0800 as excerpted: |
2 |
|
3 |
> On 01/28/2018 09:49 PM, Zac Medico wrote: |
4 |
>>> 3) Show a NOTE telling users about --changed-deps=y |
5 |
>> |
6 |
>> This is in the HINT section, which is displayed if both --changed-deps |
7 |
>> and --dynamic-deps are disabled in PATCH v2. |
8 |
> |
9 |
> Actually, the whole report should be suppressed if either --changed-deps |
10 |
> or --dynamic-deps is enabled, so I'll send PATCH v4 for that. |
11 |
|
12 |
This is shaping up quite nicely and by (1) dramatically shortening the |
13 |
original "wall of text" report and (2) aborting the report if no affected |
14 |
packages are in the graph, it's vastly improved from the original. |
15 |
|
16 |
I definitely expect it to be rather helpful here, since I have both |
17 |
--dynamic-deps and --changed-deps off by default, and seeing that list |
18 |
could be /quite/ helpful. Looking forward to it! =:^) |
19 |
|
20 |
My remaining concern, and I'm not sure there's a solution, is that for |
21 |
routine 30-day-plus updaters, the warning could quickly become "routine |
22 |
noise", due to valid no-r-bump exceptions such as the llvm example mgorny |
23 |
provided, which very well /could/ happen often enough to trigger the |
24 |
warning nearly every time for 30-day-plus updaters. Then when it really |
25 |
counts and could help, people will likely be ignoring it. |
26 |
|
27 |
Maybe someone else has an idea, but as I said it's already vastly |
28 |
improved from the original, and I believe usable as-is, now, while I'd |
29 |
have found the original quite irritating by about the third time I saw |
30 |
it, even if also helpful. |
31 |
|
32 |
-- |
33 |
Duncan - List replies preferred. No HTML msgs. |
34 |
"Every nonfree program has a lord, a master -- |
35 |
and if you use the program, he is your master." Richard Stallman |