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

Re: [PATCH v13 00/11] Add interpret-trailers builtin

From: Christian Couder <hidden>
Date: 2016-06-15 23:02:31

From: Junio C Hamano <redacted>
Christian Couder [off-list ref] writes:
quoted
  '=' is always accepted as separator when parsing
  "--trailer '<token><sep><value>'" command line arguments, for
  compatibility with other git commands
Hmph.  Which of other commands take "--option foo=bar"?

Puzzled...
Most commands accept "--option=value". I know that it is different
from "--option foo=bar", but it could reduce user mistakes if we also
accept "--trailer foo=bar".

And if we accept it, then whatever the user has configured in the
"trailer.separators" option, "--trailer foo=bar" will always work and
add the default separator. So it is a generic way to add
"foo<separator> bar" to the trailers whatever the user want as a
separator.

But maybe I should reword the documentation to add this reason. (I did
not do that in the v14 series I just sent.)

Thanks,
Christian.
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help