Thread (1 message) 1 message, 1 author, 2016-06-15

Re: [PATCH 1.8.0] add: make "add -u" update full tree without pathspec

From: Matthieu Moy <hidden>
Date: 2016-06-15 22:50:41

Junio C Hamano [off-list ref] writes:
Matthieu Moy [off-list ref] writes:
quoted
But "update" should mean "replace git add -u with git add -u .", which
is the portable way to do the same.
quoted
so the introduction of the configuration becomes a flag-day event.
Hmph...
The introduction of the config variable is a non-event if you already
use the portable . notation.
I think you got this part wrong.  Until now, there was no "portability" to
worry about when using "git add -u".  "git add -u" used to be _guaranteed_
to be relative to the cwd, with or without ".".  Our update will _break_
that expectation and suddenly make "." "the portable notation".

In other words, it is far from a non-event.
There is a "if" in my sentence, and I do claim that if you verify the
condition behind the "if", it is a non-event.

I did not claim that the whole change was a non-event, just that the
introduction of a config variable was.
We still have to say "We are sorry, we will be breaking your scripts.
Please add an extra dot at the end".
Absolutely. My claim is just that adding the extra dot is the way to fix
script wrt the future change, or at least a better way than relying on a
config variable.

-- 
Matthieu Moy
http://www-verimag.imag.fr/~moy/
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help