This is the mail archive of the gdb-patches@sourceware.org mailing list for the GDB project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: obtaining configure args from config.status


* Jan Kratochvil wrote on Tue, Sep 08, 2009 at 06:28:26PM CEST:
> On Mon, 31 Aug 2009 22:29:39 +0200, Ralf Wildenhues wrote:
> > +	     eval $(SHELL) "\"\$$rootsrc/testsuite/configure\" $$configargs" \
> > +			   "\"--srcdir=\$$rootsrc/testsuite\"" \
> 
> do I miss anything if here would be just such simple quoting?
> 
> 	     eval $(SHELL) $${rootsrc}/testsuite/configure "$$configargs" \
> 			   --srcdir=\$${rootsrc}/testsuite \
> 
> Other parts of Makefile.in already cannot cope with spaces in $srcdir.

True.  The quoting also avoids globbing, though, so I've become used to
using it consistently, also because many other packages do cope with
spaces in the absolute directory name, as long as $srcdir is relative
and doesn't contain any spaces.  But hey, if it's desired, I'll gladly
change the patch as above.

Thanks,
Ralf


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]