08/11/2019 17:15, Stephen Hemminger: > On Wed, 6 Nov 2019 13:09:43 +0100 > Thomas Monjalon <tho...@monjalon.net> wrote: > > > The constants like AF_INET are in sys/socket.h in FreeBSD. > > The #ifdef macro __FreeBSD__ is replaced with RTE_EXEC_ENV_FREEBSD > > in order to be consistent across DPDK files, and allow to grep > > for EXEC_ENV among other benefits. > > > > Signed-off-by: Thomas Monjalon <tho...@monjalon.net> > > --- > > -#ifdef __FreeBSD__ > > +#ifdef RTE_EXEC_ENV_FREEBSD > > #include <sys/socket.h> > > #endif > > Why not always include sys/socket.h having an extra include > is much less of a nuisance than a silly #ifdef.
Yes, good suggestion. David already merged the patch. Do you want to propose a patch on top?