Re: [PATCH v3] gdb: Improve debuginfod progress updates

[Tom Tromey <tom@tromey.com>]

>> Instead of this, which seems like it can hide useful info from the user,
>> what do you think about a "cylon"-style display:
>> 
>> Downloading mumble.so:
>> [      #             ]
>> 
>> ... where the '#' moves back and forth inside the [].

Aaron> I'd like to restrict the update messages to a single line so that an
Aaron> entire message can be overwritten with \r.  Then when many downloads
Aaron> occur we avoid filling the terminal with "Downloading XY MB separate
Aaron> debuginfo for libxyz" messages.

Could you show what your proposed output looks like?
I mean, after everything is downloaded?

To me it seems like having a "Downloading..." message per .so is no big deal.
gdb is often chatty.

>> Technically MI does have a progress notification approach, see
>> mi_load_progress.  I don't know if anything MI consumer actually uses
>> this, though, and so I'm not sure if it makes sense to try to wire this
>> up to debuginfo downloads.

Aaron> These MI implementations are the easiest way to see progress updates when
Aaron> using gdb+debuginfod with IDEs, for instance.  Otherwise I think each IDE
Aaron> would have to learn how to parse and display the mi_load_progress output,
Aaron> preferably in a way that's consistent with the CLI progress messages.

I am not sure we're talking about the same thing... MI defines a
progress indication message, but your patch isn't using that.  So I
wonder if any MI notification is needed at all.

Tom