On Thu, Apr 12, 2018 at 6:29 PM, Josh Soref <jsoref@xxxxxxxxx> wrote: > Randall S. Becker <rsbecker@xxxxxxxxxxxxx> 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. _______________________________________________ openssh-unix-dev mailing list openssh-unix-dev@xxxxxxxxxxx https://lists.mindrot.org/mailman/listinfo/openssh-unix-dev
