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

Re: [BUG] 'add -u' doesn't work from untracked subdir

From
Junio C Hamano <gitster@pobox.com>
Date
Sep 5, 2009, 08:23 UTC
Message-ID
<7vy6ot4x61.fsf@alter.siamese.dyndns.org>
In-Reply-To
<20090905080249.GA8801@coredump.intra.peff.net>
Jeff King <peff@peff.net> writes:
> I agree that we agreed to disagree. But there is still one open
> question: would you take a patch for a "full-tree" config variable that
> would impact add (and probably grep) for 1.7.0?

Unless there is a simple and sane way for script writers (and by "script", I do not mean "Porcelain that is supposed to be written using only the plumbing", but things like scripts you would give to "git bisect run", which would freely use Porcelains like "git reset" etc.) to defeat the configuration without much pain, I am fairly negative on adding any configuration variable of that nature.

We could probably declare "In 1.X.0 everything will be relative to the root and you have to give an explicit '.' if you mean the cwd".

Three questions:
 #1 What are the commands that will be affected, other than "add -u" and
    "grep"?  Are there others?
 #2 Do all the commands in the answer to #1 currently behave exactly the
    same when run without any path parameter and when run with a single
    '.'?
 #3 Do all the commands that are already relative to the root currently
    limit their action to the cwd when run with a single '.'?

If the number of commands in the answer to #1 is not too excessive, it is a plus, but even if it is more than just several, we will be getting consistency and sanity if #2 and #3 hold. However, if there are even a single violator in #2 and #3, we would need to fix them first before we can proceed. And the transition clock starts ticking after everything is fixed (if such a fix is indeed needed). As usual, I'd prefer to keep the clock running for at least 6 months, preferrably longer, and during that time, we may need the usual "You invoked me without any paths, but this command will start behaving differently in 1.X.0, you have been warned."

A command line option to explicitly ask full-tree can be added anytime without waiting for 1.7.0. I do not think it will be ready for 1.6.5 but we can always have 1.6.6 if needed.

Previous: Jeff KingNext: Clemens Buchacher
Message 9 of 35 in “[BUG] 'add -u' doesn't work from untracked subdir”
  1. SZEDER GáborSep 2, 2009
  2. Jeff KingSep 2, 2009
  3. Clemens BuchacherSep 4, 2009
  4. Jeff KingSep 5, 2009
  5. Junio C HamanoSep 5, 2009
  6. Jeff KingSep 5, 2009
  7. Junio C HamanoSep 5, 2009
  8. Jeff KingSep 5, 2009
  9. Junio C HamanoSep 5, 2009
  10. Clemens BuchacherSep 6, 2009
  11. Nanako ShiraishiSep 9, 2009
  12. Junio C HamanoSep 10, 2009
  13. Clemens BuchacherSep 10, 2009
  14. 1/2 grep: accept relative paths outside current working directoryClemens Buchacher, Sep 5, 2009
  15. 2/2 add 'scope' config optionClemens Buchacher, Sep 5, 2009
  16. 2/2 add 'scope' config optionClemens Buchacher, Sep 5, 2009
  17. Junio C HamanoSep 6, 2009
  18. grep: fix exit status if external_grep() returns errorClemens Buchacher, Sep 7, 2009
  19. Junio C HamanoSep 7, 2009
  20. Jeff KingSep 5, 2009
  21. Junio C HamanoSep 5, 2009
  22. Clemens BuchacherSep 5, 2009
  23. Junio C HamanoSep 5, 2009
  24. Jakub NarebskiSep 5, 2009
  25. Clemens BuchacherSep 5, 2009
  26. Junio C HamanoSep 5, 2009
  27. Matthieu MoySep 6, 2009
  28. Clemens BuchacherSep 6, 2009
  29. Matthieu MoySep 7, 2009
  30. SZEDER GáborSep 7, 2009
  31. Matthieu MoySep 7, 2009
  32. Nanako ShiraishiSep 7, 2009
  33. Junio C HamanoSep 7, 2009
  34. Clemens BuchacherSep 7, 2009
  35. SZEDER GáborSep 4, 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.