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

Re: [Xen-devel] [PATCH] configure: do not require Bison



2012/2/29 Ian Campbell <Ian.Campbell@xxxxxxxxxx>:
> On Wed, 2012-02-29 at 14:51 +0000, Roger Pau Monnà wrote:
>> 2012/2/29 Ian Campbell <ian.campbell@xxxxxxxxxx>:
>> > I _think_ this isn't required because we also checkin the generated files. 
>> > A
>> > more complete fix might also allow the user to force regeneration but I 
>> > didn't
>> > both here.
>> >
>> > Signed-off-by: Ian Campbell <ian.campbell@xxxxxxxxxx>
>> > ---
>> > Âtools/configure  Â| Â559 
>> > +++++++++++++++++++++++++---------------------------
>> > Âtools/configure.ac | Â Â1 -
>>
>> You forget to remove:
>>
>> AC_ARG_VAR([BISON], [Path to Bison parser generator])
>>
>> from tools/configure.ac and also clean related variables from
>> config/Tools.mk.in.
>
> Will do, thanks.
>
>> ÂIf you can, could you use autoconf 2.68 (present
>> in Debian stable) to generate the new configure script?
>
> I have 2.67-2 from stable. I think the 2.68 came from
> 24940:be9d56fada86.
>
> Is there a way to make configure.ac require a certain version of
> autoconf so we can be sure these sorts of changes are explicit?

AC_PREREQ([2.67]) should probably be set to 2.68 if that's ok with
everyone. I would prefer to use a version as new as posible, to avoid
hitting autconf bugs.

>> ÂSo we don't
>> have to push so many unrelated changes to configure script and
>> downgrate its version?
>
>

_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxx
http://lists.xen.org/xen-devel

 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.