Re: [PATCH] help: do not expect built-in commands to be hardlinked
- From
Junio C Hamano <gitster@pobox.com>
- Date
- Oct 7, 2020, 22:24 UTC
- Message-ID
- <xmqqh7r5znbo.fsf@gitster.c.googlers.com>
- In-Reply-To
- <nycvar.QRO.7.76.6.2010072349280.50@tvgsbejvaqbjf.bet>
Johannes Schindelin <Johannes.Schindelin@gmx.de> writes:
> Now, what I would have expected you to say when you found the `uniq()` > function is: Johannes, why don't you call `QSORT(); uniq()` after the call > to `load_builtin_commands()`? After all, `exec_path` and `env_path` might > both be `NULL`...
Nah, you are expecting too much out of me. I didn't ask because I knew we didn't need to, and I didn't particularly care if you lucked out or if you had the same understanding as I had how you arrived at the right solution ;-)