Skip to content

Commit

Permalink
Fix build reproducibility in Make 3.82
Browse files Browse the repository at this point in the history
Make 3.82 no longer sort the result of wildcards (see
http:https://comments.gmane.org/gmane.comp.gnu.make.bugs/4260). This may break
build reproducibility.

This patch sort results of wildcards to ensure reproducibility.

Signed-off-by: Jérôme Pouiller <[email protected]>
Acked-by: Arnout Vandecappelle (Essensium/Mind) <[email protected]>
Signed-off-by: Peter Korsgaard <[email protected]>
  • Loading branch information
jerome-pouiller authored and jacmet committed Sep 6, 2013
1 parent 055f1c0 commit 741cbcc
Show file tree
Hide file tree
Showing 14 changed files with 14 additions and 14 deletions.
2 changes: 1 addition & 1 deletion Makefile
Original file line number Diff line number Diff line change
Expand Up @@ -333,7 +333,7 @@ ifneq ($(PACKAGE_OVERRIDE_FILE),)
-include $(PACKAGE_OVERRIDE_FILE)
endif

include package/*/*.mk
include $(sort $(wildcard package/*/*.mk))

include boot/common.mk
include linux/linux.mk
Expand Down
2 changes: 1 addition & 1 deletion boot/common.mk
Original file line number Diff line number Diff line change
@@ -1 +1 @@
include boot/*/*.mk
include $(sort $(wildcard boot/*/*.mk))
2 changes: 1 addition & 1 deletion docs/manual/manual.mk
Original file line number Diff line number Diff line change
Expand Up @@ -53,5 +53,5 @@ $(1)-clean:
.PHONY: $(1) $(1)-clean manual-update-lists
endef

MANUAL_SOURCES = $(wildcard docs/manual/*.txt) $(wildcard docs/images/*)
MANUAL_SOURCES = $(sort $(wildcard docs/manual/*.txt) $(wildcard docs/images/*))
$(eval $(call GENDOC,manual))
2 changes: 1 addition & 1 deletion fs/common.mk
Original file line number Diff line number Diff line change
Expand Up @@ -94,4 +94,4 @@ define ROOTFS_TARGET
$(call ROOTFS_TARGET_INTERNAL,$(1),$(call UPPERCASE,$(1)))
endef

include fs/*/*.mk
include $(sort $(wildcard fs/*/*.mk))
2 changes: 1 addition & 1 deletion linux/linux.mk
Original file line number Diff line number Diff line change
Expand Up @@ -281,7 +281,7 @@ define LINUX_INSTALL_TARGET_CMDS
$(LINUX_INSTALL_HOST_TOOLS)
endef

include linux/linux-ext-*.mk
include $(sort $(wildcard linux/linux-ext-*.mk))

$(eval $(generic-package))

Expand Down
2 changes: 1 addition & 1 deletion package/efl/efl.mk
Original file line number Diff line number Diff line change
@@ -1,3 +1,3 @@
EFL_VERSION = 1.7.7

include package/efl/*/*.mk
include $(sort $(wildcard package/efl/*/*.mk))
2 changes: 1 addition & 1 deletion package/freescale-imx/freescale-imx.mk
Original file line number Diff line number Diff line change
Expand Up @@ -9,5 +9,5 @@ FREESCALE_IMX_VERSION = 1.1.0
# No official download site from freescale, just this mirror
FREESCALE_IMX_MIRROR_SITE = http:https://download.ossystems.com.br/bsp/freescale/source

include package/freescale-imx/*/*.mk
include $(sort $(wildcard package/freescale-imx/*/*.mk))

2 changes: 1 addition & 1 deletion package/gcc/gcc.mk
Original file line number Diff line number Diff line change
Expand Up @@ -220,4 +220,4 @@ HOST_GCC_COMMON_CONF_OPT += \
--with-long-double-128
endif

include package/gcc/*/*.mk
include $(sort $(wildcard package/gcc/*/*.mk))
2 changes: 1 addition & 1 deletion package/gtk2-themes/gtk2-themes.mk
Original file line number Diff line number Diff line change
@@ -1 +1 @@
include package/gtk2-themes/*/*.mk
include $(sort $(wildcard package/gtk2-themes/*/*.mk))
2 changes: 1 addition & 1 deletion package/matchbox/matchbox.mk
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
ifeq ($(BR2_PACKAGE_MATCHBOX),y)
include package/matchbox/*/*.mk
include $(sort $(wildcard package/matchbox/*/*.mk))
TARGETS+=matchbox-lib matchbox-wm
endif
2 changes: 1 addition & 1 deletion package/opengl/opengl.mk
Original file line number Diff line number Diff line change
@@ -1 +1 @@
include package/opengl/*/*.mk
include $(sort $(wildcard package/opengl/*/*.mk))
2 changes: 1 addition & 1 deletion package/qt5/qt5.mk
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
QT5_VERSION = 5.0.2
QT5_SITE = http:https://download.qt-project.org/archive/qt/5.0/$(QT5_VERSION)/submodules/
include package/qt5/*/*.mk
include $(sort $(wildcard package/qt5/*/*.mk))

define QT5_LA_PRL_FILES_FIXUP
for i in $$(find $(STAGING_DIR)/usr/lib* -name "libQt5*.la"); do \
Expand Down
2 changes: 1 addition & 1 deletion package/x11r7/x11r7.mk
Original file line number Diff line number Diff line change
@@ -1 +1 @@
include package/x11r7/*/*.mk
include $(sort $(wildcard package/x11r7/*/*.mk))
2 changes: 1 addition & 1 deletion support/dependencies/dependencies.mk
Original file line number Diff line number Diff line change
Expand Up @@ -14,7 +14,7 @@ DEPENDENCIES_HOST_PREREQ :=
define suitable-host-package
$(shell support/dependencies/check-host-$(1).sh $(2))
endef
-include support/dependencies/check-host-*.mk
-include $(sort $(wildcard support/dependencies/check-host-*.mk))

ifeq ($(BR2_STRIP_sstrip),y)
DEPENDENCIES_HOST_PREREQ+=host-sstrip
Expand Down

0 comments on commit 741cbcc

Please sign in to comment.