Re: OpenSSH 7.7 t1 script breakage

[Date Prev] [Date Next] [Thread Prev] [Thread Next] [Date Index] [Thread Index]

 



On Thu, Apr 12, 2018 at 10:29:13PM +0000, Josh Soref 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 ``.

I think you're tripping over the difference between make and shell
syntax here.  In make, $(PWD) is a variable reference, not command
substitution as in shell, and will work provided that a PWD environment
variable exists, which in practice will depend mainly on the calling
shell.  $(pwd) would only work if a pwd environment variable exists,
which is rare.

My preferred way to write this would be REGRESSTMP = $(CURDIR)/regress
(without the extraneous double-quotes, which are going to behave
confusingly when substituted into a double-quoted string further down).

-- 
Colin Watson                                       [cjwatson@xxxxxxxxxx]
_______________________________________________
openssh-unix-dev mailing list
openssh-unix-dev@xxxxxxxxxxx
https://lists.mindrot.org/mailman/listinfo/openssh-unix-dev



[Date Prev] [Date Next] [Thread Prev] [Thread Next] [Date Index] [Thread Index]

[Index of Archives]     [Linux ARM Kernel]     [Linux ARM]     [Linux Omap]     [Fedora ARM]     [IETF Annouce]     [Security]     [Bugtraq]     [Linux]     [Linux OMAP]     [Linux MIPS]     [ECOS]     [Asterisk Internet PBX]     [Linux API]

  Powered by Linux