* Re: [binutils-gdb] Style the gdb welcome message [not found] <20181228210008.91892.qmail@sourceware.org> @ 2019-10-04 22:15 ` Maciej W. Rozycki 2019-10-14 22:45 ` Tom Tromey 0 siblings, 1 reply; 5+ messages in thread From: Maciej W. Rozycki @ 2019-10-04 22:15 UTC (permalink / raw) To: Tom Tromey; +Cc: gdb-patches Hi Tom, On Fri, 28 Dec 2018, Tom Tromey wrote: > https://sourceware.org/git/gitweb.cgi?p=binutils-gdb.git;h=d47032b7ca85852acb1d2bc645f477ffee4373fc > > commit d47032b7ca85852acb1d2bc645f477ffee4373fc > Author: Tom Tromey <tom@tromey.com> > Date: Thu Sep 6 14:44:17 2018 -0600 > > Style the gdb welcome message > > This changes gdb to style the welcome message that is shown by > default. The styling is only done interactively. I missed that in the various distractions that kept me from using an up-to-date version of GDB, but now that I came across it the hard way the bright pink version line hurts my eyes and disturbs me. Setting: set style enabled off in ~/.gdbinit does not make any change to it, so I find it a functional regression. I'm sure there were important reasons to make this change, but can we please get it under user control somehow? Maciej ^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [binutils-gdb] Style the gdb welcome message 2019-10-04 22:15 ` [binutils-gdb] Style the gdb welcome message Maciej W. Rozycki @ 2019-10-14 22:45 ` Tom Tromey 2019-10-15 6:32 ` Eli Zaretskii 2019-10-15 8:32 ` Andrew Burgess 0 siblings, 2 replies; 5+ messages in thread From: Tom Tromey @ 2019-10-14 22:45 UTC (permalink / raw) To: Maciej W. Rozycki; +Cc: Tom Tromey, gdb-patches Maciej> I missed that in the various distractions that kept me from using an Maciej> up-to-date version of GDB, but now that I came across it the hard way the Maciej> bright pink version line hurts my eyes and disturbs me. Setting: Maciej> set style enabled off Maciej> in ~/.gdbinit does not make any change to it, so I find it a functional Maciej> regression. I'm sure there were important reasons to make this change, Maciej> but can we please get it under user control somehow? I added this to make it clear to users that gdb is style-enabled now. I guess I don't care much about this particular instance though. Maybe removing it is the answer. I'm curious to hear what others think. In the absence of comments I'll remove it sometime before 9.1, I think. Or I'll approve a patch to do so. thanks, Tom ^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [binutils-gdb] Style the gdb welcome message 2019-10-14 22:45 ` Tom Tromey @ 2019-10-15 6:32 ` Eli Zaretskii 2019-10-15 8:32 ` Andrew Burgess 1 sibling, 0 replies; 5+ messages in thread From: Eli Zaretskii @ 2019-10-15 6:32 UTC (permalink / raw) To: Tom Tromey; +Cc: macro, tromey, gdb-patches > From: Tom Tromey <tom@tromey.com> > Cc: Tom Tromey <tromey@sourceware.org>, gdb-patches@sourceware.org > Date: Mon, 14 Oct 2019 16:45:54 -0600 > > I'm curious to hear what others think. I like it, FWIW. ^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [binutils-gdb] Style the gdb welcome message 2019-10-14 22:45 ` Tom Tromey 2019-10-15 6:32 ` Eli Zaretskii @ 2019-10-15 8:32 ` Andrew Burgess 2019-10-19 11:21 ` Maciej W. Rozycki 1 sibling, 1 reply; 5+ messages in thread From: Andrew Burgess @ 2019-10-15 8:32 UTC (permalink / raw) To: Tom Tromey; +Cc: Maciej W. Rozycki, Tom Tromey, gdb-patches * Tom Tromey <tom@tromey.com> [2019-10-14 16:45:54 -0600]: > Maciej> I missed that in the various distractions that kept me from using an > Maciej> up-to-date version of GDB, but now that I came across it the hard way the > Maciej> bright pink version line hurts my eyes and disturbs me. Setting: > Maciej> set style enabled off > Maciej> in ~/.gdbinit does not make any change to it, so I find it a functional > Maciej> regression. I'm sure there were important reasons to make this change, > Maciej> but can we please get it under user control somehow? > > I added this to make it clear to users that gdb is style-enabled now. > I guess I don't care much about this particular instance though. Maybe > removing it is the answer. > > I'm curious to hear what others think. In the absence of comments I'll > remove it sometime before 9.1, I think. Or I'll approve a patch to > do so. FWIW I hadn't even noticed the use of colour there, so it certainly doesn't bother me. On my machine with both a light background and a dark background I don't see a bright pink, but a darker pink. I guess there's no simple way that we could postpone printing the welcome message until some later point, say, just before GDB tries to print anything else? What this might mean is that if ~/.gdbinit contains only (or starts with) commands that produce no output, for example 'set style enabled off' then these would be processed before GDB printed the welcome message. I've not looked at the code at all, so have no idea how hard this would be to actually do... Thanks, Andrew ^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [binutils-gdb] Style the gdb welcome message 2019-10-15 8:32 ` Andrew Burgess @ 2019-10-19 11:21 ` Maciej W. Rozycki 0 siblings, 0 replies; 5+ messages in thread From: Maciej W. Rozycki @ 2019-10-19 11:21 UTC (permalink / raw) To: Andrew Burgess; +Cc: Tom Tromey, Tom Tromey, gdb-patches On Tue, 15 Oct 2019, Andrew Burgess wrote: > > I'm curious to hear what others think. In the absence of comments I'll > > remove it sometime before 9.1, I think. Or I'll approve a patch to > > do so. > > What this might mean is that if ~/.gdbinit contains only (or starts > with) commands that produce no output, for example 'set style enabled > off' then these would be processed before GDB printed the welcome > message. Thank you all who responded for your input. As this is a matter of a human interface I realise that personal perception will be very different among various people, from positive, through neutral to disturbing. The classic single-colour interface is uniform and does not, in my personal experience, cause any disturbance or distraction to me. Other people may appreciate the highlighting. I think a way to move forward might be to borrow from other GNU projects, such as GCC and coreutils, and have an environment variable defined called GDB_COLORS, analogously to GCC_COLORS or LS_COLORS, and make GDB interpret it ahead of its usual initialisation files (which could optionally override the environment settings). Thoughts? Maciej ^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2019-10-19 11:21 UTC | newest] Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- [not found] <20181228210008.91892.qmail@sourceware.org> 2019-10-04 22:15 ` [binutils-gdb] Style the gdb welcome message Maciej W. Rozycki 2019-10-14 22:45 ` Tom Tromey 2019-10-15 6:32 ` Eli Zaretskii 2019-10-15 8:32 ` Andrew Burgess 2019-10-19 11:21 ` Maciej W. Rozycki
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).