Thread (1 message) 1 message, 1 author, 2023-08-23

Re: [PATCH] submodule: deprecate --recurse-submodules=""

From: Junio C Hamano <hidden>
Date: 2023-08-23 20:27:01

Taylor Blau [off-list ref] writes:
So here we'd be doing the equivalent of

    *v = RECURSE_SUBMODULES_OFF;

when trying to parse `--recurse-submodules=""`. Should this advice
instead say "[...] Use --no-recurse-submodules without an argument,
which does the same thing"?
Sounds right.  So there is nothing to change here, I guess.

If --recurse-submodules="" does something people do not expect to
happen, an warning might be warranted, but I somehow do not think
that is the case here.  If we are not hurting anybody by accepting
that (possibly unusual) form, I do not think we would want to add an
extra warning, either.

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