Re: obtaining configure args from config.status

Ralf Wildenhues
2009-09-08T18:51:57Z

* Tom Tromey wrote on Tue, Sep 08, 2009 at 08:38:11PM CEST:
> >>>>> "Ralf" == Ralf Wildenhues writes:
>
> >> eval $(SHELL) $${rootsrc}/testsuite/configure "$$configargs" \
> >> --srcdir=\$${rootsrc}/testsuite \
> >> Other parts of Makefile.in already cannot cope with spaces in $srcdir.
>
> Ralf> True. The quoting also avoids globbing, though, so I've become used to
> Ralf> using it consistently, also because many other packages do cope with
> Ralf> spaces in the absolute directory name, as long as $srcdir is relative
> Ralf> and doesn't contain any spaces. But hey, if it's desired, I'll gladly
> Ralf> change the patch as above.
>
> If it doesn't hurt anything (other than readability), it is ok to leave
> it as you have it.

Yes, it should not hurt. Thanks for the review, applied.