Message ID | 1429266405-9217-1-git-send-email-gustavo@zacarias.com.ar |
---|---|
State | Accepted |
Headers | show |
gustavo, All, On 2015-04-17 07:26 -0300, Gustavo Zacarias spake thusly: > Revert a 3.2.26 patch that breaks the bmon build (among possibly > others). Fixes: > http://autobuild.buildroot.net/results/296/296bcaf4fd1fadcab71881d6d00a8763bfa91a41/ I wonder if that should not be bmon that should be patched to include that header wherever necessary, instead of reverting a commit from libnl? Regards, Yann E. MORIN. > Signed-off-by: Gustavo Zacarias <gustavo@zacarias.com.ar> > --- > ...remove-unnecessary-include-of-private-lin.patch | 44 ++++++++++++++++++++++ > 1 file changed, 44 insertions(+) > create mode 100644 package/libnl/0001-Revert-route-remove-unnecessary-include-of-private-lin.patch > > diff --git a/package/libnl/0001-Revert-route-remove-unnecessary-include-of-private-lin.patch b/package/libnl/0001-Revert-route-remove-unnecessary-include-of-private-lin.patch > new file mode 100644 > index 0000000..3520899 > --- /dev/null > +++ b/package/libnl/0001-Revert-route-remove-unnecessary-include-of-private-lin.patch > @@ -0,0 +1,44 @@ > +From 61310ce3f2b6c5c21be8c788848f083e4c58d138 Mon Sep 17 00:00:00 2001 > +From: Gustavo Zacarias <gustavo@zacarias.com.ar> > +Date: Fri, 17 Apr 2015 07:20:48 -0300 > +Subject: [PATCH] Revert "route: remove unnecessary include of private > + linux/if.h" > + > +It breaks rtnl_link_get_flags() checks against IFF_* defines that aren't > +defined. > + > +This reverts commit 50a76998ac36ace3716d3c979b352fac73cfc80a. > + > +Signed-off-by: Gustavo Zacarias <gustavo@zacarias.com.ar> > +--- > + include/netlink/route/link.h | 1 + > + src/nl-link-set.c | 1 - > + 2 files changed, 1 insertion(+), 1 deletion(-) > + > +diff --git a/include/netlink/route/link.h b/include/netlink/route/link.h > +index 8f74f42..a7aa88b 100644 > +--- a/include/netlink/route/link.h > ++++ b/include/netlink/route/link.h > +@@ -15,6 +15,7 @@ > + #include <netlink/netlink.h> > + #include <netlink/cache.h> > + #include <netlink/addr.h> > ++#include <linux/if.h> > + #include <sys/types.h> > + > + #ifdef __cplusplus > +diff --git a/src/nl-link-set.c b/src/nl-link-set.c > +index 6a6bac8..bbb60f9 100644 > +--- a/src/nl-link-set.c > ++++ b/src/nl-link-set.c > +@@ -9,7 +9,6 @@ > + * Copyright (c) 2003-2010 Thomas Graf <tgraf@suug.ch> > + */ > + > +-#include <linux/if.h> > + #include <netlink/cli/utils.h> > + #include <netlink/cli/link.h> > + > +-- > +2.0.5 > + > -- > 2.0.5 > > _______________________________________________ > buildroot mailing list > buildroot@busybox.net > http://lists.busybox.net/mailman/listinfo/buildroot
Dear Gustavo Zacarias, On Fri, 17 Apr 2015 07:26:44 -0300, Gustavo Zacarias wrote: > Revert a 3.2.26 patch that breaks the bmon build (among possibly > others). Fixes: > http://autobuild.buildroot.net/results/296/296bcaf4fd1fadcab71881d6d00a8763bfa91a41/ > > Signed-off-by: Gustavo Zacarias <gustavo@zacarias.com.ar> > --- > ...remove-unnecessary-include-of-private-lin.patch | 44 ++++++++++++++++++++++ > 1 file changed, 44 insertions(+) > create mode 100644 package/libnl/0001-Revert-route-remove-unnecessary-include-of-private-lin.patch Both applied, thanks. However, for the libnl patch, can it be fixed upstream? Or should it really be fixed in bmon? Thanks, Thomas
diff --git a/package/libnl/0001-Revert-route-remove-unnecessary-include-of-private-lin.patch b/package/libnl/0001-Revert-route-remove-unnecessary-include-of-private-lin.patch new file mode 100644 index 0000000..3520899 --- /dev/null +++ b/package/libnl/0001-Revert-route-remove-unnecessary-include-of-private-lin.patch @@ -0,0 +1,44 @@ +From 61310ce3f2b6c5c21be8c788848f083e4c58d138 Mon Sep 17 00:00:00 2001 +From: Gustavo Zacarias <gustavo@zacarias.com.ar> +Date: Fri, 17 Apr 2015 07:20:48 -0300 +Subject: [PATCH] Revert "route: remove unnecessary include of private + linux/if.h" + +It breaks rtnl_link_get_flags() checks against IFF_* defines that aren't +defined. + +This reverts commit 50a76998ac36ace3716d3c979b352fac73cfc80a. + +Signed-off-by: Gustavo Zacarias <gustavo@zacarias.com.ar> +--- + include/netlink/route/link.h | 1 + + src/nl-link-set.c | 1 - + 2 files changed, 1 insertion(+), 1 deletion(-) + +diff --git a/include/netlink/route/link.h b/include/netlink/route/link.h +index 8f74f42..a7aa88b 100644 +--- a/include/netlink/route/link.h ++++ b/include/netlink/route/link.h +@@ -15,6 +15,7 @@ + #include <netlink/netlink.h> + #include <netlink/cache.h> + #include <netlink/addr.h> ++#include <linux/if.h> + #include <sys/types.h> + + #ifdef __cplusplus +diff --git a/src/nl-link-set.c b/src/nl-link-set.c +index 6a6bac8..bbb60f9 100644 +--- a/src/nl-link-set.c ++++ b/src/nl-link-set.c +@@ -9,7 +9,6 @@ + * Copyright (c) 2003-2010 Thomas Graf <tgraf@suug.ch> + */ + +-#include <linux/if.h> + #include <netlink/cli/utils.h> + #include <netlink/cli/link.h> + +-- +2.0.5 +
Revert a 3.2.26 patch that breaks the bmon build (among possibly others). Fixes: http://autobuild.buildroot.net/results/296/296bcaf4fd1fadcab71881d6d00a8763bfa91a41/ Signed-off-by: Gustavo Zacarias <gustavo@zacarias.com.ar> --- ...remove-unnecessary-include-of-private-lin.patch | 44 ++++++++++++++++++++++ 1 file changed, 44 insertions(+) create mode 100644 package/libnl/0001-Revert-route-remove-unnecessary-include-of-private-lin.patch