diff options
author | Ævar Arnfjörð Bjarmason <avarab@gmail.com> | 2022-06-28 13:05:28 +0300 |
---|---|---|
committer | Junio C Hamano <gitster@pobox.com> | 2022-06-28 23:13:17 +0300 |
commit | 0d68ee723e54330138450f29e358e5ebe1a47aa0 (patch) | |
tree | f0dbe768348e496c05853b08c7ecfeac51d35d9e /git-submodule.sh | |
parent | d9c7f69aaa6b001949e9d2b693c22c595cc9d0d6 (diff) |
submodule update: remove "-v" option
In e84c3cf3dc3 (git-submodule.sh: accept verbose flag in cmd_update to
be non-quiet, 2018-08-14) the "git submodule update" sub-command was
made to understand "-v", but the option was never documented.
The only in-tree user has been this test added in
3ad0401e9e6 (submodule update: silence underlying merge/rebase with
"--quiet", 2020-09-30), it wasn't per-se testing --quiet, but fixing a
bug in e84c3cf3dc3: It used to set "GIT_QUIET=0" instead of unsetting
it on "-v", and thus we'd end up passing "--quiet" to "git
submodule--helper" on "-v", since the "--quiet" option was passed
using the ${parameter:+word} construct.
Furthermore, even if someone had used the "-v" option they'd only be
getting the default output. Our default in both git-submodule.sh and
"git submodule--helper" has been to be "verbose", so the only way this
option could have matter is if it were used as e.g.:
git submodule --quiet update -v [...]
I.e. to undo the effect of a previous "--quiet" on the command-line.
Signed-off-by: Ævar Arnfjörð Bjarmason <avarab@gmail.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
Diffstat (limited to 'git-submodule.sh')
-rwxr-xr-x | git-submodule.sh | 3 |
1 files changed, 0 insertions, 3 deletions
diff --git a/git-submodule.sh b/git-submodule.sh index 5b9683bf76..0df6b0fc97 100755 --- a/git-submodule.sh +++ b/git-submodule.sh @@ -241,9 +241,6 @@ cmd_update() -q|--quiet) GIT_QUIET=1 ;; - -v) - unset GIT_QUIET - ;; --progress) progress=1 ;; |