]> code.ossystems Code Review - openembedded-core.git/commitdiff
kbd: Fix build reproducibility issue
authorRichard Purdie <richard.purdie@linuxfoundation.org>
Mon, 27 Jan 2020 16:32:59 +0000 (16:32 +0000)
committerRichard Purdie <richard.purdie@linuxfoundation.org>
Mon, 27 Jan 2020 16:44:04 +0000 (16:44 +0000)
Our CFLAGS were being lost which was breaking reproducibile builds due to
losee of the debug prefix remapping flags. Fix this.

Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
meta/recipes-core/kbd/kbd/fix_cflags.patch [new file with mode: 0644]
meta/recipes-core/kbd/kbd_2.2.0.bb

diff --git a/meta/recipes-core/kbd/kbd/fix_cflags.patch b/meta/recipes-core/kbd/kbd/fix_cflags.patch
new file mode 100644 (file)
index 0000000..3722096
--- /dev/null
@@ -0,0 +1,25 @@
+We need to ensure our CFLAGS are preserved as well as whatever tweak configure
+tries to make. Without these, the debug prefix changes get lost and we lose
+build reproducibility, likely with other side effects.
+
+Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
+Upstream-Status: Pending
+2020/1/27
+
+Index: kbd-2.2.0/configure.ac
+===================================================================
+--- kbd-2.2.0.orig/configure.ac
++++ kbd-2.2.0/configure.ac
+@@ -72,9 +72,9 @@ if test "$enable_code_coverage" = yes; t
+ fi
+ case "$GCC,$ac_cv_prog_cc_g" in
+-      yes,yes) CFLAGS="-g $CC_O_LEVEL $FORTIFY_SOURCE" ;;
+-      yes,)    CFLAGS="$CC_O_LEVEL $FORTIFY_SOURCE" ;;
+-         ,yes) CFLAGS="-g" ;;
++      yes,yes) CFLAGS="-g $CC_O_LEVEL $FORTIFY_SOURCE $CFLAGS" ;;
++      yes,)    CFLAGS="$CC_O_LEVEL $FORTIFY_SOURCE $CFLAGS" ;;
++         ,yes) CFLAGS="-g $CFLAGS" ;;
+ esac
+ CC_CHECK_CFLAGS_APPEND([\
index e13cea763469d2a07a4ab3e6d706bd413b708ef4..e5700ff57f9eae8e612f841207937f32ad23124a 100644 (file)
@@ -17,6 +17,7 @@ SRC_URI = "${KERNELORG_MIRROR}/linux/utils/${BPN}/${BP}.tar.xz \
            ${@bb.utils.contains('DISTRO_FEATURES', 'ptest', 'file://set-proper-path-of-resources.patch', '', d)} \
            file://0001-analyze.l-add-missing-string-format.patch \
            file://0001-Use-DATADIR-and-append-i386-to-fix-libkbdfile-test08.patch \
+           file://fix_cflags.patch \
            "
 
 SRC_URI[md5sum] = "d1d7ae0b5fb875dc082731e09cd0c8bc"