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

Re: [PATCH 0/3] preparatory patches for the submodule groups

From
Stefan Beller <sbeller@google.com>
Date
May 3, 2016, 21:12 UTC
Message-ID
<CAGZ79kbm5y+LeyA_dwQWKFNaa42280cOvwQYZTP=-aRnySsB1A@mail.gmail.com>
In-Reply-To
<xmqqoa8mlutg.fsf@gitster.mtv.corp.google.com>
On Tue, May 3, 2016 at 2:01 PM, Junio C Hamano <gitster@pobox.com> wrote:
Show 18 quoted lines
> Stefan Beller <sbeller@google.com> writes:
>
>> I have your patch here and have a "-a and pathspec are incompatible" fix
>> build on top.
>> * I do wonder if we want to have the shortform '-a' though.
>
> I do not particularly care.  I was merely matching the other two
> options there.
>
>> * I think we want to head for consistency, eventually.
>>    e.g. commands with no arguments such as tag, branch
>>    give a list of their respective domain.
>
> Isn't that a historical mistake we are regretting, though?  Only
> after many other operation modes were invented and "create X" proves
> not to be the only primary modes we had to invent "tag -l" and
> "branch -l".  Aren't we better off not having "no option means list"
> kind of default?

listing is not destructive, and I really like to not type a single dash for some commands.

Show 9 quoted lines
>
>>    Subcommands do not give lists by default, e.g.
>>    `git stash clear`, `git remote prune`
>>    which are the moral equivalent to
>>    `git submodule deinit` just work as they were told, no --switch needed.
>
> I wouldn't say "git rm" should remove everything by extending that
> logic, but I can certainly buy if somebody argues "git submodule
> deinit" is not destructive enough to warrant extra safety.
`git rm` is a command, not a subcommand though.
Previous: Junio C HamanoNext: Junio C Hamano
Message 14 of 17 in “preparatory patches for the submodule groups”
  1. 0/3 preparatory patches for the submodule groupsStefan Beller, May 2, 2016
  2. 1/3 submodule deinit test: fix broken && chain in subshellStefan Beller, May 2, 2016
  3. Junio C HamanoMay 3, 2016
  4. Stefan BellerMay 3, 2016
  5. 2/3 submodule deinit: lose requirement for giving '.'Stefan Beller, May 2, 2016
  6. Junio C HamanoMay 3, 2016
  7. Stefan BellerMay 3, 2016
  8. Junio C HamanoMay 3, 2016
  9. Stefan BellerMay 3, 2016
  10. 3/3 submodule init: redirect stdout to stderrStefan Beller, May 2, 2016
  11. Junio C HamanoMay 3, 2016
  12. Stefan BellerMay 3, 2016
  13. Junio C HamanoMay 3, 2016
  14. Stefan BellerMay 3, 2016
  15. Junio C HamanoMay 3, 2016
  16. Stefan BellerMay 3, 2016
  17. Junio C HamanoMay 3, 2016

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.