git/list[1] front-page[2] threads[3] people[4] search[5] about
 

Re: Very simple popen() code request, ground-shaking functionality openned by it

From
Duy Nguyen <pclouds@gmail.com>
Date
Sep 23, 2018, 13:06 UTC
Message-ID
<CACsJy8C5-kFivA5eEJu7pKA5=PajW5rLB6V4tVEueMtr58QC=A@mail.gmail.com>
In-Reply-To
<87musajun7.fsf@evledraar.gmail.com>

On Sat, Sep 22, 2018 at 1:30 AM Ævar Arnfjörð Bjarmason <avarab@gmail.com> wrote:

Show 23 quoted lines
> Duy's
> https://public-inbox.org/git/20180920161928.GA13379@duynguyen.home/ is
> another recent thing that reminded me of this, i.e. that suggested
> "\\|/-" spinner could be made much neater with non-ASCII.
>
> >   1. Add a trace_key for sending machine-readable progress output to a
> >      descriptor or file. E.g., via setting GIT_TRACE_PROGRESS=2 in the
> >      environment.
> >
> >   2. Teach the trace code to open a command for piping, so that you
> >      could do something like GIT_TRACE_PROGRESS='|mygauge'.
> >
> > That would make your use case work, and I think many other use cases
> > would benefit from both of those features independently.
>
> Yup, that's all sensible, and would be great both for this and other
> stuff if we wanted true extensibility for this sort of thing.
>
> I'll just add that a 3rd thing that would also make sense would be to
> add a feature to configure the value of these GIT_TRACE_*=* variables
> via the .gitconfig, that's been suggested before (too lazy to dig up a
> ML archive reference), and would make this as easy to configure as
> Sebastian's suggestion.

I'm less concern about prettiness than showing numbers that are hard to interpret. My other option was just showing ".", "..", "..." sequence, which achieves the same thing.

I'm not opposed to having core.progressor or something like that. We already have core.pager and this new config serves more or less the same purpose. But I don't think I'll put time into implementing it.

-- 
Duy
Previous: Sebastian Gniazdowski
Message 6 of 6 in “Very simple popen() code request, ground-shaking functionality openned by it”
  1. Sebastian GniazdowskiSep 21, 2018
  2. Jeff KingSep 21, 2018
  3. Ævar Arnfjörð BjarmasonSep 21, 2018
  4. Jeff KingSep 21, 2018
  5. Sebastian GniazdowskiSep 22, 2018
  6. Duy NguyenSep 23, 2018

Read the whole thread, see it on lore, or plain text.

$ cat FOOTERMessages come from the public archive at lore.kernel.org/git, fetched every hour. The front page is chosen and written each morning by an AI editor and can be wrong; the threads themselves are the record. About and API. For agents: an MCP server at https://gitlist.dev/mcp, and any thread, story or person page as Markdown by adding .md to its URL (or sending Accept: text/markdown). Details in /llms.txt.