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

Re: [PATCH] Add format-patch option --no-name-prefix.

From: Junio C Hamano <hidden>
Date: 2016-06-15 22:44:00

Linus Torvalds [off-list ref] writes:
On Tue, 18 Dec 2007, Pascal Obry wrote:
quoted
Johannes Sixt a écrit :
quoted
that we need another diff option for it. Ok, on my keyboard 0 is typed
with the right hand, and 1 with the left hand, but... ??
Because you just did not read my follow-up message :)

I need this has I do not have the way to change the server applying the
patch. So nothing wrong with my hands or fingers :)
....
So I think you'd need separate arguments for the from/to prefixes, and not 
try to shoehorn it into one argument. With possibly some simple form to 
say "no prefix". So maybe something like

  --src-prefix=<string>		// default "a/"
  --dst-prefix=<string>		// default "b/"
  --no-prefix			// shorthand for --src-prefix="" --dst-prefix=""

would work for everybody?
One worry I have is that "diff --git" is validated more strictly than
other diffs by "git-apply", and patches generated with arbitrary prefix
would break it.  It might make sense to drop " --git" from the patch
header if we allow a/ and b/ to be changed inconsistently.
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help