Thread (10 messages) flat view 10 messages, 3 authors, 2016-06-15

Re: [PATCH 1/2] require pathspec for "git add -u/-A"

From: Jeff King <hidden>
Date: 2016-06-15 22:56:22

Possibly related (same subject, not in this thread)

On Tue, Mar 12, 2013 at 02:58:44PM +0100, Matthieu Moy wrote:
quoted
I guess we already rejected the idea of being able to shut off the
warning and just get the new behavior, in favor of having people
specify it manually each time?
Somehow, but we may find a way to do so, as long as it temporary (i.e.
something that will have no effect after the transition period), and
that is is crystal clear that it's temporary.
Yeah, I think this is easy as long as it is "enable the new behavior
now" and not "toggle between new and old behavior". That is, a boolean
rather than a selector, with a note that it will go away at the behavior
switch.

The only downside I see is that a user may switch it on now, saying
"Yes, I understand and prefer the new behavior", but some script they
run might not expect it. We can warn against that in the documentation,
but that may or may not be enough.

Here's a series which does that; if it's the direction we want to go, I
think we'd want to rebase Junio's "now add -u is full-tree" patch on
top.

  [1/2]: t2200: check that "add -u" limits itself to subdirectory
  [2/2]: add: respect add.updateroot config option

-Peff
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help