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

Re: [PATCH] archive: Refuse to write the archive to a terminal.

From
Matthieu Moy <matthieu.moy@grenoble-inp.fr>
Date
Sep 16, 2009, 12:57 UTC
Message-ID
<vpq8wgfm4ho.fsf@bauges.imag.fr>
In-Reply-To
<3f4fd2640909160448x1fbb7a64s1ce0adca2af5010@mail.gmail.com>
Reece Dunn <msclrhd@googlemail.com> writes:
Show 14 quoted lines
> 2009/9/16 Mikael Magnusson <mikachu@gmail.com>:
>> 2009/9/16 Johannes Sixt <j.sixt@viscovery.net>:
>>> Josh Triplett schrieb:
>>>> I considered adding a -f/--force option, like gzip has, but writing an
>>>> archive to a tty seems like a sufficiently insane use case that I'll let
>>>> whoever actually needs that write the patch for it. ;)
>>>
>>> How about '--output -' instead?
>>
>> You could always just add '|cat'.
>
> Except when running on Windows. Yes MSYS and cygwin provide a version
> of cat, but this cannot be guaranteed (e.g. with the series to support
> building with MSVC).

In general, autodectection features sometimes fail, so it's good to have an explicit override option.

> The `--output -` / `-o -` syntax looks reasonable
I like this too.
-- 
Matthieu Moy
http://www-verimag.imag.fr/~moy/
Previous: Reece DunnNext: Josh Triplett
Message 5 of 7 in “archive: Refuse to write the archive to a terminal.”
  1. archive: Refuse to write the archive to a terminal.Josh Triplett, Sep 16, 2009
  2. Johannes SixtSep 16, 2009
  3. Mikael MagnussonSep 16, 2009
  4. Reece DunnSep 16, 2009
  5. Matthieu MoySep 16, 2009
  6. Josh TriplettSep 17, 2009
  7. Johannes SixtSep 17, 2009

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.