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