Gentoo Archives: gentoo-dev

From: Ionen Wolkens <ionen@g.o>
To: gentoo-dev@l.g.o
Subject: Re: [gentoo-dev] [RFC] Portage einfo, elog... output format change
Date: Sun, 03 Oct 2021 02:53:46
Message-Id: YVkbMWRJK1R0MRyw@eversor
In Reply to: Re: [gentoo-dev] [RFC] Portage einfo, elog... output format change by Ionen Wolkens
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

Attachments

File name MIME type
signature.asc application/pgp-signature

Replies