diff options
author | Elijah Newren <newren@gmail.com> | 2018-09-05 10:25:50 -0700 |
---|---|---|
committer | Junio C Hamano <gitster@pobox.com> | 2018-09-12 15:17:17 -0700 |
commit | d345e9fbe75c8bec6469253f0ee7e34228f41917 (patch) | |
tree | 619ff5285ab156d9d495ae6635ff8c0ab7173318 /t/t4002-diff-basic.sh | |
parent | update-ref: fix type of update_flags variable to match its usage (diff) | |
download | tgif-d345e9fbe75c8bec6469253f0ee7e34228f41917.tar.xz |
update-ref: allow --no-deref with --stdin
If passed both --no-deref and --stdin, update-ref would error out with a
general usage message that did not at all suggest these options were
incompatible. The manpage for update-ref did suggest through its
synopsis line that --no-deref and --stdin were incompatible, but it sadly
also incorrectly suggested that -d and --no-deref were incompatible. So
the help around the --no-deref option is buggy in a few ways.
The --stdin option did provide a different mechanism for avoiding
dereferencing symbolic-refs: adding a line reading
option no-deref
before every other directive in the input. (Technically, if the user
wants to do the extra work of first determining which refs they want to
update or delete are symbolic, then they only need to put the extra
"option no-deref" lines before the updates of those refs. But in some
cases, that's more work than just adding the "option no-deref" before
every other directive.)
It's easier to allow the user to just pass --no-deref along with --stdin
in order to tell update-ref that the user doesn't want any symbolic ref
to be dereferenced. It also makes the update-ref documentation simpler.
Implement that, and update the documentation to match.
Signed-off-by: Elijah Newren <newren@gmail.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
Diffstat (limited to 't/t4002-diff-basic.sh')
0 files changed, 0 insertions, 0 deletions