OK, thanks. I'll update the patch to check for the additional location.

On the autoconf version, I think it's just the test suite that uses
some macros that were introduced in 2.64.

On Tue, Dec 17, 2013 at 10:14 AM, Madko <madk...@gmail.com> wrote:
> Typo was only in the mail, sorry about that Jesse. Configure tells nothing
> about checking if netdev_features_t is here. So I add ./boot.sh in the spec
> file just before the ./configure and it builds fine now. Just a problem on
> the autoconf version detection:
> configure.ac:15: error: Autoconf version 2.64 or higher is required
> On rhel6 we have 2.63 only, but I force it (sed the PREREQ to 2.63 in
> configure.ac) and it works fine too. Is there a good reason to prereq 2.64?
>
> Thanks for the patch
>
>
> 2013/12/17 Jesse Gross <je...@nicira.com>
>>
>> On Tue, Dec 17, 2013 at 1:59 AM, Madko <madk...@gmail.com> wrote:
>> > I try the patch on 2.0.0 tarball:
>> > http://ur1.ca/g7l1a
>> >
>> > still some erors about conflicting types for 'netdev_features_t'
>> >
>> > btw
>> >
>> >  OVS_GREP_IFELSE([$KSRC/include/linux/netdev_features.h],
>> > [netdev_features_t])
>> >
>> >
>> > I don't have $KSRC/include/linux/netdev_features.h, just a
>> > $KSRC/include/linux/netdevicde.h which defines u64 netdev_features_t. I
>> > try
>> > to change the OVS_GREP_IFELSE but no change...
>>
>> Hmm, can you see what the result of the configure test is (and double
>> check the spelling - you have a typo above). I don't have a CentOS 6.5
>> machine handy to test it on but it did compile fine on a new kernel
>> which defines this, which would indicate that the problem is with the
>> configure test.
>>
>> > Should I try with sources from git?
>>
>> The patch is against git master although it shouldn't really affect the
>> test.
>
>
>
>
> --
> Edouard Bourguignon
_______________________________________________
discuss mailing list
discuss@openvswitch.org
http://openvswitch.org/mailman/listinfo/discuss

Reply via email to