Message ID | 20170503123215.20544-1-peter@korsgaard.com |
---|---|
State | Accepted |
Commit | 40a5d253787039774992334050577363c09a7d94 |
Headers | show |
All, On Wed, May 3, 2017 at 7:32 AM, Peter Korsgaard <peter@korsgaard.com> wrote: > Adds overlay support. > > Drop the non-upstreamed 0002-extra-cflags.patch and instead simply pass the > needed CFLAGS as make arguments. Passing the full upstream warning flags > are not important in the Buildroot context, so this is simpler than > maintaining a patch. > > Cc: "Yann E. MORIN" <yann.morin.1998@free.fr> > Signed-off-by: Peter Korsgaard <peter@korsgaard.com> A dtc bump came up earlier this week on one of our projects starting to use overlays. :-) Thanks for the patch. Reviewed-by: Matt Weber <matthew.weber@rockwellcollins.com> > --- > package/dtc/0002-extra-cflags.patch | 32 -------------------------------- > package/dtc/dtc.hash | 2 +- > package/dtc/dtc.mk | 6 +++--- > 3 files changed, 4 insertions(+), 36 deletions(-) > delete mode 100644 package/dtc/0002-extra-cflags.patch > > diff --git a/package/dtc/0002-extra-cflags.patch b/package/dtc/0002-extra-cflags.patch > deleted file mode 100644 > index f070692c0..000000000 > --- a/package/dtc/0002-extra-cflags.patch > +++ /dev/null > @@ -1,32 +0,0 @@ > -Makefile: append the CFLAGS to existing ones > - > -Allow the user to pass custom CFLAGS (eg. optimisation flags). > - > -Do not use EXTRA_CFLAGS, append to existing CFLAGS with += (Arnout) > - > -Signed-off-by: "Yann E. MORIN" <yann.morin.1998@free.fr> > -Cc: Arnout Vandecappelle <arnout@mind.be> > - > ---- > -Patch not sent upstream. > - > -Although not specific to buildroot, I am not sure this is the best > -way to handle user-supplied CFLAGS. > - > -diff --git a/Makefile b/Makefile > -index 962f94eba661..bf6b317158cf 100644 > ---- a/Makefile > -+++ b/Makefile > -@@ -16,9 +16,10 @@ LOCAL_VERSION = > - CONFIG_LOCALVERSION = > - > - CPPFLAGS = -I libfdt -I . > --WARNINGS = -Werror -Wall -Wpointer-arith -Wcast-qual -Wnested-externs \ > -+WARNINGS = -Wall -Wpointer-arith -Wcast-qual -Wnested-externs \ > - -Wstrict-prototypes -Wmissing-prototypes -Wredundant-decls -Wshadow > --CFLAGS = -g -Os -fPIC -Werror $(WARNINGS) > -+CFLAGS ?= -g -Os > -+CFLAGS += -fPIC $(WARNINGS) > - > - BISON = bison > - LEX = flex > diff --git a/package/dtc/dtc.hash b/package/dtc/dtc.hash > index 8b3af331f..75c2e40ee 100644 > --- a/package/dtc/dtc.hash > +++ b/package/dtc/dtc.hash > @@ -1,2 +1,2 @@ > # from https://www.kernel.org/pub/software/utils/dtc/sha256sums.asc > -sha256 77992ad8eac7b68f553d0ba58e5b51604ac803d126196c99e3ae38aaae28bb94 dtc-1.4.1.tar.xz > +sha256 470731d5c015b160d26a96645dbb1c7337d6e7b8c98244612002b66bedf6cffb dtc-1.4.4.tar.xz > diff --git a/package/dtc/dtc.mk b/package/dtc/dtc.mk > index 5bb2ca26a..9d50e3596 100644 > --- a/package/dtc/dtc.mk > +++ b/package/dtc/dtc.mk > @@ -4,7 +4,7 @@ > # > ################################################################################ > > -DTC_VERSION = 1.4.1 > +DTC_VERSION = 1.4.4 > DTC_SOURCE = dtc-$(DTC_VERSION).tar.xz > DTC_SITE = https://www.kernel.org/pub/software/utils/dtc > DTC_LICENSE = GPL-2.0+ or BSD-2-Clause (library) > @@ -32,7 +32,7 @@ DTC_INSTALL_GOAL = install-lib > endif # $(BR2_PACKAGE_DTC_PROGRAMS) != y > > define DTC_BUILD_CMDS > - $(TARGET_CONFIGURE_OPTS) $(MAKE) -C $(@D) PREFIX=/usr > + $(TARGET_CONFIGURE_OPTS) $(MAKE) CFLAGS="$(TARGET_CFLAGS) -fPIC" -C $(@D) PREFIX=/usr > endef > > # For staging, only the library is needed > @@ -47,7 +47,7 @@ endef > > # host build > define HOST_DTC_BUILD_CMDS > - $(HOST_CONFIGURE_OPTS) $(MAKE) -C $(@D) PREFIX=$(HOST_DIR)/usr > + $(HOST_CONFIGURE_OPTS) $(MAKE) CFLAGS="$(HOST_CFLAGS) -fPIC" -C $(@D) PREFIX=$(HOST_DIR)/usr > endef > > define HOST_DTC_INSTALL_CMDS > -- > 2.11.0 > > _______________________________________________ > buildroot mailing list > buildroot@busybox.net > http://lists.busybox.net/mailman/listinfo/buildroot
>>>>> "Peter" == Peter Korsgaard <peter@korsgaard.com> writes: > Adds overlay support. > Drop the non-upstreamed 0002-extra-cflags.patch and instead simply pass the > needed CFLAGS as make arguments. Passing the full upstream warning flags > are not important in the Buildroot context, so this is simpler than > maintaining a patch. > Cc: "Yann E. MORIN" <yann.morin.1998@free.fr> > Signed-off-by: Peter Korsgaard <peter@korsgaard.com> Committed, thanks.
diff --git a/package/dtc/0002-extra-cflags.patch b/package/dtc/0002-extra-cflags.patch deleted file mode 100644 index f070692c0..000000000 --- a/package/dtc/0002-extra-cflags.patch +++ /dev/null @@ -1,32 +0,0 @@ -Makefile: append the CFLAGS to existing ones - -Allow the user to pass custom CFLAGS (eg. optimisation flags). - -Do not use EXTRA_CFLAGS, append to existing CFLAGS with += (Arnout) - -Signed-off-by: "Yann E. MORIN" <yann.morin.1998@free.fr> -Cc: Arnout Vandecappelle <arnout@mind.be> - ---- -Patch not sent upstream. - -Although not specific to buildroot, I am not sure this is the best -way to handle user-supplied CFLAGS. - -diff --git a/Makefile b/Makefile -index 962f94eba661..bf6b317158cf 100644 ---- a/Makefile -+++ b/Makefile -@@ -16,9 +16,10 @@ LOCAL_VERSION = - CONFIG_LOCALVERSION = - - CPPFLAGS = -I libfdt -I . --WARNINGS = -Werror -Wall -Wpointer-arith -Wcast-qual -Wnested-externs \ -+WARNINGS = -Wall -Wpointer-arith -Wcast-qual -Wnested-externs \ - -Wstrict-prototypes -Wmissing-prototypes -Wredundant-decls -Wshadow --CFLAGS = -g -Os -fPIC -Werror $(WARNINGS) -+CFLAGS ?= -g -Os -+CFLAGS += -fPIC $(WARNINGS) - - BISON = bison - LEX = flex diff --git a/package/dtc/dtc.hash b/package/dtc/dtc.hash index 8b3af331f..75c2e40ee 100644 --- a/package/dtc/dtc.hash +++ b/package/dtc/dtc.hash @@ -1,2 +1,2 @@ # from https://www.kernel.org/pub/software/utils/dtc/sha256sums.asc -sha256 77992ad8eac7b68f553d0ba58e5b51604ac803d126196c99e3ae38aaae28bb94 dtc-1.4.1.tar.xz +sha256 470731d5c015b160d26a96645dbb1c7337d6e7b8c98244612002b66bedf6cffb dtc-1.4.4.tar.xz diff --git a/package/dtc/dtc.mk b/package/dtc/dtc.mk index 5bb2ca26a..9d50e3596 100644 --- a/package/dtc/dtc.mk +++ b/package/dtc/dtc.mk @@ -4,7 +4,7 @@ # ################################################################################ -DTC_VERSION = 1.4.1 +DTC_VERSION = 1.4.4 DTC_SOURCE = dtc-$(DTC_VERSION).tar.xz DTC_SITE = https://www.kernel.org/pub/software/utils/dtc DTC_LICENSE = GPL-2.0+ or BSD-2-Clause (library) @@ -32,7 +32,7 @@ DTC_INSTALL_GOAL = install-lib endif # $(BR2_PACKAGE_DTC_PROGRAMS) != y define DTC_BUILD_CMDS - $(TARGET_CONFIGURE_OPTS) $(MAKE) -C $(@D) PREFIX=/usr + $(TARGET_CONFIGURE_OPTS) $(MAKE) CFLAGS="$(TARGET_CFLAGS) -fPIC" -C $(@D) PREFIX=/usr endef # For staging, only the library is needed @@ -47,7 +47,7 @@ endef # host build define HOST_DTC_BUILD_CMDS - $(HOST_CONFIGURE_OPTS) $(MAKE) -C $(@D) PREFIX=$(HOST_DIR)/usr + $(HOST_CONFIGURE_OPTS) $(MAKE) CFLAGS="$(HOST_CFLAGS) -fPIC" -C $(@D) PREFIX=$(HOST_DIR)/usr endef define HOST_DTC_INSTALL_CMDS
Adds overlay support. Drop the non-upstreamed 0002-extra-cflags.patch and instead simply pass the needed CFLAGS as make arguments. Passing the full upstream warning flags are not important in the Buildroot context, so this is simpler than maintaining a patch. Cc: "Yann E. MORIN" <yann.morin.1998@free.fr> Signed-off-by: Peter Korsgaard <peter@korsgaard.com> --- package/dtc/0002-extra-cflags.patch | 32 -------------------------------- package/dtc/dtc.hash | 2 +- package/dtc/dtc.mk | 6 +++--- 3 files changed, 4 insertions(+), 36 deletions(-) delete mode 100644 package/dtc/0002-extra-cflags.patch