OpenSSH 7.7 t1 script breakage
Nico Kadel-Garcia
nkadel at gmail.com
Fri Apr 13 11:41:04 AEST 2018
On Thu, Apr 12, 2018 at 6:29 PM, Josh Soref <jsoref at gmail.com> wrote:
> Randall S. Becker <rsbecker at nexbridge.com> wrote:
>
>>
>> -REGRESSTMP = "$(PWD)/regress"
>> +REGRESSTMP = `pwd`
>>
>> tests interop-tests t-exec unit: regress-prep regress-binaries $(TARGETS)
>>
>
> It looks like the problem is that pwd is in uppercase, not so much the
> distinction between $() and ``.
PWD is a commonly set environment variable, built into shells such as
bash, Changing it from "$(PWD)/regress" to `pwd` is begging for issues
compiling it in Windows environments, where spaces and Unicode may be
in the working directory name.. It's also no longer setting the
directory to the "/regress" subdirectory, which is begging for
confusion if done uncautiously.
More information about the openssh-unix-dev
mailing list