Re: [PATCH 3/5] make git-bugreport a builtin
- From
Johannes Schindelin <johannes.schindelin@gmx.de>
- Date
- Aug 14, 2020, 14:25 UTC
- Message-ID
- <nycvar.QRO.7.76.6.2008141623530.54@tvgsbejvaqbjf.bet>
- In-Reply-To
- <xmqqr1sa1iym.fsf@gitster.c.googlers.com>
Hi Junio,
On Thu, 13 Aug 2020, Junio C Hamano wrote:
Show 22 quoted lines
> Junio C Hamano <gitster@pobox.com> writes: > > > Jeff King <peff@peff.net> writes: > > > >> macros). That could change eventually of course, but it's not like > >> switching back to a stand-alone at that point is a big deal. > > > > If it is not a big deal, then I am fine, but at the same time it > > becomes unclear why we can say "we can use X and Y niceties only > > available for builtins". > > Actually, there is another issue that is potentially a lot worse. > > "git bugreport" not being built-in would mean that the compiled-in > git-version MUST not be relied upon (it merely is a version of the > source "git bugreport" came from, it does not necessarily match the > main "git" binary when we are dealing with a confused user with > GIT_EXEC_PATH problem), but has to be asked to "git" binary on the > $PATH externally. That needs to be done for any non-builtin > binaries (e.g. when user is having issue with git-remote--curl") > anyway, keeping "git bugreport" out of builtin would force us the > necessary discipline from early on.
Since `git bugreport` requires specific functionality from, say, `git-remote--curl` we would already have a bit of trouble in case of version skew there.
But yes, I am glad that we finally can at least rely on the Git version `git bugreport` reports.
Ciao, Dscho