Re: [PATCH] t7500: fix tests with absolute path following ":(optional)" on Windows
- From
D. Ben Knoble <ben.knoble@gmail.com>
- Date
- Oct 20, 2025, 20:27 UTC
- Message-ID
- <CALnO6CBwn-NP-ZdoaeOD37_VM8N4D-KKopm3nnf4a9b+9OiizA@mail.gmail.com>
- In-Reply-To
- <5d780103-285b-4e6c-9b26-2a87609837cf@kdbg.org>
On Mon, Oct 20, 2025 at 1:32 PM Johannes Sixt <j6t@kdbg.org> wrote:
Show 12 quoted lines
> > Am 20.10.25 um 15:43 schrieb Ben Knoble: > > Going forward I will probably stick with using pwd, given the > > difference in platform behavior. > $(pwd) is usually safe, but not always. If we have to look at every > instance anyway, we can use $PWD for efficiency if it does not matter, > and $(pwd) only when it is necessary. > > > Is there a doc or test lint for that? If not, might be useful. > > If this were documented somewhere, would you have found it and obeyed > the recommendations?
Likely yes, but I'll admit to being the exception rather than the rule (I like to read). A lint is more valuable in that it can at least be run rather than searched for.
-- D. Ben Knoble