In message <[EMAIL PROTECTED]> on Tue, 2 Nov 2004 20:56:14 +0000 (GMT), Chris 
Pitchford <[EMAIL PROTECTED]> said:

openssl> On Tue, 2 Nov 2004, Richard Levitte - VMS Whacker wrote:
openssl> 
openssl> > In message <[EMAIL PROTECTED]> on Thu, 28 Oct 2004 17:51:48 -0400, Carson 
Gaspar <[EMAIL PROTECTED]> said:
openssl> >
openssl> > carson> --On Thursday, October 28, 2004 15:13:14 +0100 Robert Bannocks
openssl> > carson> <[EMAIL PROTECTED]> wrote:
openssl> > carson>
openssl> > carson> > I get this too if you quote the $(EXHEADER) to be "$(EXHEADER)" 
in the
openssl> > carson> > install section of the files listed below it compiles and 
installs ok.
openssl> > carson>
openssl> > carson> This will break if EXHEADER is _not_ empty. The following
openssl> > carson> snippet should work in all cases (assuming I got my backslash
openssl> > carson> escaping right...):
openssl> > carson>
openssl> > carson> install:
openssl> > carson>      @if test -n "$(EXHEADER)"; then \
openssl> > carson>              headerlist="$(EXHEADER)"; \
openssl> > carson>              for i in $$headerlist; do \
openssl> > carson>                      (cp $$i 
$(INSTALL_PREFIX)$(INSTALLTOP)/include/openssl/$$i; \
openssl> > carson>                      chmod 644 
$(INSTALL_PREFIX)$(INSTALLTOP)/include/openssl/$$i ); \
openssl> > carson>              done; \
openssl> > carson>      fi
openssl> >
openssl> > In this case, is the test strictly necessary?
openssl> 
openssl> The test is not required if an interim variable is used (such as
openssl> $headerlist). It would seem like a good work around.

Thanks.  I just tested with such a change, and things went very
smoothly.  Another test with that test removed, and I'll be ready to
commit.

Cheers,
Richard

-- 
Richard Levitte                         [EMAIL PROTECTED]
                                        http://richard.levitte.org/

"When I became a man I put away childish things, including
 the fear of childishness and the desire to be very grown up."
                                                -- C.S. Lewis
______________________________________________________________________
OpenSSL Project                                 http://www.openssl.org
User Support Mailing List                    [EMAIL PROTECTED]
Automated List Manager                           [EMAIL PROTECTED]

Reply via email to