Re: [PATCH v2 7/9] help: error if [-a|-g|-c] and [-i|-m|-w] are combined

From: Junio C Hamano <gitster@pobox.com>
Date: Feb 23, 2022, 22:16 UTC
Message-ID: <xmqqv8x5qlea.fsf@gitster.g>
In-Reply-To: <patch-v2-7.9-992ee6580ac-20220221T193708Z-avarab@gmail.com>

Ævar Arnfjörð Bjarmason  <avarab@gmail.com> writes:

Show 8 quoted lines

> Add more sanity checking to "git help" usage by erroring out if these
> man viewer options are combined with incompatible command-modes that
> will never use these documentation viewers.
>
> This continues the work started in d35d03cf93e (help: simplify by
> moving to OPT_CMDMODE(), 2021-09-22) of adding more sanity checking to
> "git help". Doing this allows us to clarify the "SYNOPSIS" in the
> documentation, and the "git help -h" output.

While this is not wrong per-se, "git help --all --web" in the future that pops the same information up in your running browser would not be a bad thing to have.

That use of --web is certainly not in line with --man and --info, though (at that point we'd be saying "show in browser or show in terminal").

So I guess this is OK at least for now.