[Buildroot] [PATCH 1/2] libnet:new package
Thomas Petazzoni
thomas.petazzoni at free-electrons.com
Wed Nov 11 14:05:28 UTC 2015
Arnout,
I'm just seeing your review right now. Did you mark the patches as
Changes Requested? I did not mark them as Accepted as far as I
remember, even if I applied them.
On Wed, 11 Nov 2015 00:17:56 +0100, Arnout Vandecappelle wrote:
> We now require .hash files for github packages as well, since github nowadays
> provides stable tarballs.
This package is not fetched from github. But I've added the hash
nonetheless.
> > +LIBNET_VERSION = 1.1.6
> > +LIBNET_SITE = http://sourceforge.net/projects/libnet-dev/files
> > +LIBNET_INSTALL_STAGING = YES
> > +LIBNET_LICENSE = BSD-2c
>
> There are also a few BSD-3c files, so I think this should be BSD-2c, BSD-3c.
I made a follow-up commit to fix this.
> The configure script tries to build and run a program to test for ETH_P_ALL
> availability in the AC_LIBNET_CHECK_PF_PACKET function. This will obviously fail
> when cross-compiling, maybe worse when hostarch == targetarch. So I think it's
> best to explicitly pass libnet_cv_have_packet_socket=yes
I did a test build here for ARM, and did not encounter any issues.
Which configuration did you test? Or is it just by inspection of
configure.ac?
Thanks,
Thomas
--
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com
More information about the buildroot
mailing list