1 |
On Sat, Oct 02, 2021 at 06:04:36PM -0400, Ionen Wolkens wrote: |
2 |
> On Sat, Oct 02, 2021 at 09:22:49AM +0200, Ulrich Mueller wrote: |
3 |
> > >>>>> On Wed, 29 Sep 2021, A Schenck wrote: |
4 |
> > |
5 |
> > > On 9/28/21 8:36 AM, Michał Górny wrote: |
6 |
> > >> [WW] some message |
7 |
> > >> [EE] other message |
8 |
> > >> [QA] hell if i know what this is |
9 |
> > >> |
10 |
> > >> I've also added more colors to explicitly distinguish einfo from elog, |
11 |
> > >> and ewarn from eqawarn. Then, I've replaced most of '>>>' and '!!!' |
12 |
> > >> used by Portage with four-character versions to keep messages aligned. |
13 |
> > >> The 'zings' used for merging files remain three-character, so now it's |
14 |
> > >> easily possible to distinguish messages from installed file list. |
15 |
> > |
16 |
> > > Didn't expect to be the only dissenting opinion on something like this |
17 |
> > > but. . . Some applications parse portage output looking for these |
18 |
> > > 'zings'. At very least app-portage/kuroo does it this way; there must be |
19 |
> > > others, right? This is obviously not the ideal way to get information |
20 |
> > > out of portage, but it's been stable for the 15 years Kuroo has existed. |
21 |
> > > 10-ish years ago dol-sen started some work on building and API for |
22 |
> > > portage, but then got sucked into core portage development to the point |
23 |
> > > of abandoning their GTK+ portage GUI porthole, which was the original |
24 |
> > > impetus for an API, and as far as we know, the API never made it to the |
25 |
> > > point it could replace parsing the output. |
26 |
> > |
27 |
> > If only the length of the >>> and !!! strings is a problem, then why not |
28 |
> > leave them alone and go for single-letter tags instead? Like this: |
29 |
> > |
30 |
> > [.] einfo |
31 |
> > [I] elog |
32 |
> > [W] ewarn |
33 |
> > [E] eerror |
34 |
> > [Q] eqawarn |
35 |
> > |
36 |
> > The only problematic one is [Q] instead of [QA] which is no longer |
37 |
> > self-explanatory. Then again, only devs and experienced users should see |
38 |
> > eqawarn messages. |
39 |
> |
40 |
> fwiw eqawarn is currently displayed for everyone in a similar manner |
41 |
> as einfo, just not post-emerge/elog without adding to ELOG classes. |
42 |
> |
43 |
> If users aren't hiding build logs entirely, they may notice those |
44 |
> done at the end (often shown after size report) -- and possibly |
45 |
> think it's a problem. |
46 |
> |
47 |
> Not to say whether [Q] vs [QA] would help with this much so I have |
48 |
> no strong opinion here. |
49 |
|
50 |
Guess there's a lot of other options that could be considered as well. |
51 |
|
52 |
--- files |
53 |
>>> text |
54 |
* current, it wasn't aligned now that I look at it again |
55 |
(relying only on color to convey type feels clearly wrong to me) |
56 |
|
57 |
--- files |
58 |
>>>> text |
59 |
[QA] new based on current PR |
60 |
|
61 |
>>> text |
62 |
[QA] aligned 1 character further, maybe skipping changing >>> is fine? |
63 |
(then again that it's further is what threw me off at first) |
64 |
|
65 |
>>> text |
66 |
QA* similar to before, but aligned using only 3 chars |
67 |
|
68 |
>>> text |
69 |
[Q] kinda more obscure but it can work |
70 |
|
71 |
>>>> text |
72 |
QA* probably closest to how it was before alignment-wise, but meh at 4 > |
73 |
|
74 |
>>> message |
75 |
QA> not convinced about this one, but throwing it here anyway |
76 |
(other characters could be considered as well) |
77 |
|
78 |
Maybe a poll of some kind may help, personally undecided on what I |
79 |
like better beside agreeing that a change is needed. |
80 |
|
81 |
-- |
82 |
ionen |