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
