Hi Stephen, On Tue, May 30, 2017 at 05:31:51PM -0700, Stephen Hemminger wrote: > On Mon, 22 May 2017 16:27:53 +0300 > Baruch Siach <bar...@tkos.co.il> wrote: > > > Including libc headers first helps as a workaround to redefinition of struct > > ethhdr with a suitably patched musl libc that suppresses the kernel > > if_ether.h. > > > > Signed-off-by: Baruch Siach <bar...@tkos.co.il> > > --- > > ip/iplink_bridge.c | 2 +- > > 1 file changed, 1 insertion(+), 1 deletion(-) > > > > diff --git a/ip/iplink_bridge.c b/ip/iplink_bridge.c > > index 818b43c89b5b..cccdec1c203a 100644 > > --- a/ip/iplink_bridge.c > > +++ b/ip/iplink_bridge.c > > @@ -13,9 +13,9 @@ > > #include <stdlib.h> > > #include <string.h> > > #include <netinet/in.h> > > +#include <netinet/ether.h> > > #include <linux/if_link.h> > > #include <linux/if_bridge.h> > > -#include <netinet/ether.h> > > #include <net/if.h> > > > > #include "rt_names.h" > > Applied.
Thanks. > Next time please fix the Subject line. You implied two patches by using 1/2 > but only one was sent. You have seen and responded[1] to patch 2/2 in this series. Eventually that patch turned out to be redundant, so you committed cda81a4ea5e (include: remove no longer used iptables_common.h) instead. > It is easier for me if you include iproute2 in subject since then it goes > into a separate folder. > > Subject: [PATCH iproute2] ip: include libc headers first > > or if intended for net-next > > Subject: [PATCH iproute2 net-next] ip: add magic VRF support I will remember that for next time. [1] http://marc.info/?l=linux-netdev&m=149547389506859 baruch -- http://baruch.siach.name/blog/ ~. .~ Tk Open Systems =}------------------------------------------------ooO--U--Ooo------------{= - bar...@tkos.co.il - tel: +972.52.368.4656, http://www.tkos.co.il -