[prev in list] [next in list] [prev in thread] [next in thread] 

List:       autoconf-bug
Subject:    Re: Improper substitution of @srcdir@ in VPATH template for "in-source" configure
From:       Eric Blake <eblake () redhat ! com>
Date:       2010-06-23 19:29:41
Message-ID: 4C2260A5.2090006 () redhat ! com
[Download RAW message or body]


On 06/23/2010 12:20 PM, Ralf Wildenhues wrote:
> Hi Keith,
> 
> * Keith Marshall wrote on Wed, Jun 23, 2010 at 10:58:01AM CEST:
>> Initially reported here:
>> http://thread.gmane.org/gmane.comp.gnu.mingw.user/33435/focus=33455
> 
> Thanks for the bug report.
> 
>>   $ cat Makefile.in
>>   # @configure_input@
>>   srcdir = @srcdir@
>>   VPATH = @srcdir@/src
>>   # EOF
> 
>>   $ grep VPATH Makefile
>>   VPATH = /src
>>
>> Note that this should be `VPATH = ./src', but the dot, which should 
>> have resulted from substitution of @srcdir@, has been elided.  Note 
>> however:
> 
> The patch below should fix this.  Tested on several systems.  OK?

Yes, that makes sense.

> 
> Space-separated VPATH entries are not portable to FreeBSD and several
> vendor makes.  I'm not quite sure if in practice we may run into make
> implementations any more that do not support
> - variable expansions at all in VPATH,
> - multiple entries in VPATH,
> - VPATH at all,
> 
> but the manual indicates there used to be such makes.  Eric, are you ok
> with the testsuite addition being a test balloon for this?  Otherwise,
> it needs a prerequisite test for the basic functionality to avoid
> spurious failures.

Hmm, adding a test that might fail on more exotic platforms will at
least remind us to patch things on those platforms, so I guess I'm okay
with it.  At any rate, the test passes on the most common open source
development setups, and we aren't violating the portability pitfalls
exposed by this test anywhere else.

-- 
Eric Blake   eblake@redhat.com    +1-801-349-2682
Libvirt virtualization library http://libvirt.org


["signature.asc" (application/pgp-signature)]

[prev in list] [next in list] [prev in thread] [next in thread]