]> code.ossystems Code Review - openembedded-core.git/commitdiff
reproducible: Drop BUILD_REPRODUCIBLE_BINARIES variable
authorRichard Purdie <richard.purdie@linuxfoundation.org>
Thu, 14 Oct 2021 11:58:40 +0000 (12:58 +0100)
committerRichard Purdie <richard.purdie@linuxfoundation.org>
Sat, 16 Oct 2021 16:35:01 +0000 (17:35 +0100)
We want things to be reproduicble and the variable doesn't really change
much any more. Drop the remaining uses and make those code paths always
active.

Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
meta/classes/image-prelink.bbclass
meta/classes/image.bbclass
meta/conf/bitbake.conf
meta/lib/oe/sstatesig.py
meta/recipes-core/busybox/busybox.inc

index 0da094a5518bee6fb68a9ccea917bf76e17e0d99..8158eeaf4ca132fe7da8e16770dce992b93673fc 100644 (file)
@@ -46,17 +46,12 @@ prelink_image () {
        dynamic_loader=${@get_linuxloader(d)}
 
        # prelink!
-       if [ "${BUILD_REPRODUCIBLE_BINARIES}" = "1" ]; then
-               bbnote " prelink: BUILD_REPRODUCIBLE_BINARIES..."
-               if [ "$REPRODUCIBLE_TIMESTAMP_ROOTFS" = "" ]; then
-                       export PRELINK_TIMESTAMP=`git log -1 --pretty=%ct `
-               else
-                       export PRELINK_TIMESTAMP=$REPRODUCIBLE_TIMESTAMP_ROOTFS
-               fi
-               ${STAGING_SBINDIR_NATIVE}/prelink --root ${IMAGE_ROOTFS} -am -N -c ${sysconfdir}/prelink.conf --dynamic-linker $dynamic_loader
+       if [ "$REPRODUCIBLE_TIMESTAMP_ROOTFS" = "" ]; then
+               export PRELINK_TIMESTAMP=`git log -1 --pretty=%ct `
        else
-               ${STAGING_SBINDIR_NATIVE}/prelink --root ${IMAGE_ROOTFS} -amR -N -c ${sysconfdir}/prelink.conf --dynamic-linker $dynamic_loader
+               export PRELINK_TIMESTAMP=$REPRODUCIBLE_TIMESTAMP_ROOTFS
        fi
+       ${STAGING_SBINDIR_NATIVE}/prelink --root ${IMAGE_ROOTFS} -am -N -c ${sysconfdir}/prelink.conf --dynamic-linker $dynamic_loader
 
        # Remove the prelink.conf if we had to add it.
        if [ "$dummy_prelink_conf" = "true" ]; then
index c2f32320278ca2d5b2ef9fe70cf9b6926c5e622c..2fa69a40d10d4bc4d9dd4ede7872d2a2a879d2c5 100644 (file)
@@ -651,17 +651,15 @@ ROOTFS_PREPROCESS_COMMAND += "${@bb.utils.contains('DISTRO_FEATURES', 'usrmerge'
 POPULATE_SDK_PRE_TARGET_COMMAND += "${@bb.utils.contains('DISTRO_FEATURES', 'usrmerge', 'create_merged_usr_symlinks_sdk; ', '',d)}"
 
 reproducible_final_image_task () {
-    if [ "${BUILD_REPRODUCIBLE_BINARIES}" = "1" ]; then
+    if [ "$REPRODUCIBLE_TIMESTAMP_ROOTFS" = "" ]; then
+        REPRODUCIBLE_TIMESTAMP_ROOTFS=`git -C "${COREBASE}" log -1 --pretty=%ct 2>/dev/null` || true
         if [ "$REPRODUCIBLE_TIMESTAMP_ROOTFS" = "" ]; then
-            REPRODUCIBLE_TIMESTAMP_ROOTFS=`git -C "${COREBASE}" log -1 --pretty=%ct 2>/dev/null` || true
-            if [ "$REPRODUCIBLE_TIMESTAMP_ROOTFS" = "" ]; then
-                REPRODUCIBLE_TIMESTAMP_ROOTFS=`stat -c%Y ${@bb.utils.which(d.getVar("BBPATH"), "conf/bitbake.conf")}`
-            fi
+            REPRODUCIBLE_TIMESTAMP_ROOTFS=`stat -c%Y ${@bb.utils.which(d.getVar("BBPATH"), "conf/bitbake.conf")}`
         fi
-        # Set mtime of all files to a reproducible value
-        bbnote "reproducible_final_image_task: mtime set to $REPRODUCIBLE_TIMESTAMP_ROOTFS"
-        find  ${IMAGE_ROOTFS} -print0 | xargs -0 touch -h  --date=@$REPRODUCIBLE_TIMESTAMP_ROOTFS
     fi
+    # Set mtime of all files to a reproducible value
+    bbnote "reproducible_final_image_task: mtime set to $REPRODUCIBLE_TIMESTAMP_ROOTFS"
+    find  ${IMAGE_ROOTFS} -print0 | xargs -0 touch -h  --date=@$REPRODUCIBLE_TIMESTAMP_ROOTFS
 }
 
 systemd_preset_all () {
index 1db9b3fc051dfdd90a1ba78defbf9c184937b8ea..790f2f7a8c49f4e13d9841939c0694edecd5529d 100644 (file)
@@ -636,7 +636,6 @@ SDE_DIR = "${WORKDIR}/source-date-epoch"
 SDE_FILE = "${SDE_DIR}/__source_date_epoch.txt"
 SDE_DEPLOYDIR = "${WORKDIR}/deploy-source-date-epoch"
 
-BUILD_REPRODUCIBLE_BINARIES = "1"
 export PYTHONHASHSEED = "0"
 export PERL_HASH_SEED = "0"
 export SOURCE_DATE_EPOCH ?= "${@get_source_date_epoch_value(d)}"
index c2e3e2f4f58cace9f43e3f314dede96203cfa22f..038404e377a477f8eee65b06dd11644bfd0e6990 100644 (file)
@@ -489,7 +489,7 @@ def OEOuthashBasic(path, sigfile, task, d):
     include_timestamps = False
     include_root = True
     if task == "package":
-        include_timestamps = d.getVar('BUILD_REPRODUCIBLE_BINARIES') == '1'
+        include_timestamps = True
         include_root = False
     extra_content = d.getVar('HASHEQUIV_HASH_VERSION')
 
index 808c3dc700952c3bcbcd8d38077770d71aa33626..622325aabb333030016a7471de0c8dd4d4a414cc 100644 (file)
@@ -110,9 +110,8 @@ python () {
 }
 
 do_prepare_config () {
-       if [ "${BUILD_REPRODUCIBLE_BINARIES}" = "1" ]; then
-               export KCONFIG_NOTIMESTAMP=1
-       fi
+       export KCONFIG_NOTIMESTAMP=1
+
        sed -e '/CONFIG_STATIC/d' \
                < ${WORKDIR}/defconfig > ${S}/.config
        echo "# CONFIG_STATIC is not set" >> .config
@@ -143,9 +142,8 @@ do_configure () {
 
 do_compile() {
        unset CFLAGS CPPFLAGS CXXFLAGS LDFLAGS
-       if [ "${BUILD_REPRODUCIBLE_BINARIES}" = "1" ]; then
-               export KCONFIG_NOTIMESTAMP=1
-       fi
+       export KCONFIG_NOTIMESTAMP=1
+
        if [ "${BUSYBOX_SPLIT_SUID}" = "1" -a x`grep "CONFIG_FEATURE_INDIVIDUAL=y" .config` = x ]; then
                # split the .config into two parts, and make two busybox binaries
                if [ -e .config.orig ]; then