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

Re: [PATCH RFC] Add a config verbose option fetch and push

From
Ævar Arnfjörð Bjarmason <avarab@gmail.com>
Date
May 22, 2010, 12:01 UTC
Message-ID
<AANLkTimQzAM7qA32FRFvQC1cx7UEEKtBxjU89whrSqF5@mail.gmail.com>
In-Reply-To
<201005221244.32213.trast@student.ethz.ch>
On Sat, May 22, 2010 at 10:44, Thomas Rast <trast@student.ethz.ch> wrote:
Show 12 quoted lines
> Ævar Arnfjörð Bjarmason wrote:
> That would completely ruin the scriptability of almost all commands.
> Imagine the user added the following options as default:
>
>  add --edit
>  checkout --patch
>  cherry-pick --no-commit
>  commit --amend
>  pull --rebase
>
> I'm sure you can find one option that changes the command in something
> completely different *for every command*.
Sure. But so would adding this as git-add to your $PATH:
    #!/bin/sh
    /usr/lib/git-core/git-add --edit $@

Git already has plenty of ways to shoot yourself in the foot. I don't see how it's worse if that's done through some generalized facility which results in less special-case code in individual tools.

Previous: Thomas RastNext: Thomas Rast
Message 4 of 6 in “Re: [PATCH RFC] Add a config verbose option fetch and push”
  1. Nathan W. PanikeMay 21, 2010
  2. Ævar Arnfjörð BjarmasonMay 21, 2010
  3. Thomas RastMay 22, 2010
  4. Ævar Arnfjörð BjarmasonMay 22, 2010
  5. Thomas RastMay 22, 2010
  6. Ævar Arnfjörð BjarmasonMay 22, 2010

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.