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

Re: What's cooking in git.git (Feb 2013, #05; Tue, 12)

From
Junio C Hamano <gitster@pobox.com>
Date
Feb 13, 2013, 00:34 UTC
Message-ID
<7vsj51caqb.fsf@alter.siamese.dyndns.org>
In-Reply-To
<CAH5451nPKq8DKwo+Bkxh08N-wqrYCY4BihbvaE14z5iGVA1iZw@mail.gmail.com>
Andrew Ardill <andrew.ardill@gmail.com> writes:
Show 17 quoted lines
> On 13 February 2013 11:06, Junio C Hamano <gitster@pobox.com> wrote:
>> * jc/add-delete-default (2012-08-13) 1 commit
>>  - git add: notice removal of tracked paths by default
>>
>>  "git add dir/" updated modified files and added new files, but does
>>  not notice removed files, which may be "Huh?" to some users.  They
>>  can of course use "git add -A dir/", but why should they?
>>
>>  Resurrected from graveyard, as I thought it was a worthwhile thing
>>  to do in the longer term.
>>
>>  Stalled mostly due to lack of responses.
>
> What do you need to progress this?
>
> I have been bitten by this before (the 'huh?' reaction) and think the
> previous discussions and patch look reasonable. Does it need testing?

I _think_ the code does what it claims it does; I do not think that is what is lacking (more testing would not _hurt_, of course).

> Further input??

The updated behaviour is a departure from the traditional norm, and it would surprise people who do not expect "git add ." to update the index for removed paths. For many of them, it may be a pleasant surprise, but "many" is not "all".

The change could negatively affect people who expect that removing files that are not used for their purpose (e.g. a large file that is unnecessary for their build) will _not_ affect what they get from "git add ."; obviously they must have trained themselves not to do "git add -u" or "git commit -a".

Previous: Andrew ArdillNext: Andrew Ardill
Message 5 of 15 in “What's cooking in git.git (Feb 2013, #05; Tue, 12)”
  1. Junio C HamanoFeb 13, 2013
  2. jn/shell-disable-interactive (Re: What's cooking in git.git (Feb 2013, #05; Tue, 12))Jonathan Nieder, Feb 13, 2013
  3. Junio C HamanoFeb 13, 2013
  4. Andrew ArdillFeb 13, 2013
  5. Junio C HamanoFeb 13, 2013
  6. Andrew ArdillFeb 13, 2013
  7. Junio C HamanoFeb 13, 2013
  8. Andrew ArdillFeb 14, 2013
  9. Junio C HamanoFeb 14, 2013
  10. Andrew ArdillFeb 14, 2013
  11. Junio C HamanoFeb 14, 2013
  12. Junio C HamanoFeb 14, 2013
  13. Miles BaderFeb 22, 2013
  14. Junio C HamanoFeb 22, 2013
  15. greened@obbligato.orgFeb 18, 2013

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.