From patchwork Thu Aug 29 10:11:11 2024 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Richard Purdie X-Patchwork-Id: 48448 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from aws-us-west-2-korg-lkml-1.web.codeaurora.org (localhost.localdomain [127.0.0.1]) by smtp.lore.kernel.org (Postfix) with ESMTP id 99040C8302B for ; Thu, 29 Aug 2024 10:11:23 +0000 (UTC) Received: from mail-wr1-f51.google.com (mail-wr1-f51.google.com [209.85.221.51]) by mx.groups.io with SMTP id smtpd.web10.9433.1724926275887568390 for ; Thu, 29 Aug 2024 03:11:16 -0700 Authentication-Results: mx.groups.io; dkim=pass header.i=@linuxfoundation.org header.s=google header.b=Yefyxg/r; spf=pass (domain: linuxfoundation.org, ip: 209.85.221.51, mailfrom: richard.purdie@linuxfoundation.org) Received: by mail-wr1-f51.google.com with SMTP id ffacd0b85a97d-371a9bea8d4so294300f8f.2 for ; Thu, 29 Aug 2024 03:11:15 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linuxfoundation.org; s=google; t=1724926274; x=1725531074; darn=lists.openembedded.org; h=content-transfer-encoding:mime-version:message-id:date:subject:to :from:from:to:cc:subject:date:message-id:reply-to; bh=ThGPrA8k0ycfA9kTYIOSZQaWWZL7X8SJieZqJvpQ5ls=; b=Yefyxg/rwS8d+W02m80gd8TKL3sVC7X739S0ijHOFA26Dc0vXuhSHu5s6aOuNd0CAC 5NH7DqJJSSqSKeCWJvTGSh/edQN2fxWHfawBlG2yDv6GayM0dTQ3bpCkAu2Ux40Vxo4M HxoHNjrmuw18v+gMZYzRv9p/3jLyunBtWZd50= X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1724926274; x=1725531074; h=content-transfer-encoding:mime-version:message-id:date:subject:to :from:x-gm-message-state:from:to:cc:subject:date:message-id:reply-to; bh=ThGPrA8k0ycfA9kTYIOSZQaWWZL7X8SJieZqJvpQ5ls=; b=BnXATrhqF1/e1XvHdyR9A0OKdc88fCMtqTDci+bceZwY314izxk2UZtu9lNQ3S690f vAimn5cKaM3lm06p62KLWm2UZWJMympyiawIip5yOpwr2AZ0kkfS2QHFwWW9C80nPyW+ uLCHQRzFitkw+eFI0VewOQazh4yb2t4enD08p7NfHXmS9NUPgsyziDGSCccUqVacmE95 wUt5FJqrpBYddna4dL/FyDD9oqoVZnoxLSRCDQWS17QNuuRF5ZrCz2MDLijy/bnv/kbB NDC1u45HuiiDh9vzXVaDTqFH+KN4EuWRsNbBSz3weDeGr79CurSo8dUyKM5h8ldWRa99 uVHQ== X-Gm-Message-State: AOJu0YwDwZfXAsEPdDKCvJ3aYHzwb8LlOJWJ7hIqiQOvAAUwTHNs0YFx YZw0WXfEtcrWiU1AbR+BNuhOjnxepT5n7JHR8qMocgJk9FsJYIRVodW2Rurc45DOTxS10XXPIXn H X-Google-Smtp-Source: AGHT+IH3Zd+4jcu4Cw2WAUR3emmm+wyi/PnOA3LkiWibok+vjxtROZneL1S/Jrcd/Yiy0MLI7Rjmdw== X-Received: by 2002:adf:e882:0:b0:374:93ca:b1de with SMTP id ffacd0b85a97d-3749b586c35mr1677236f8f.52.1724926273441; Thu, 29 Aug 2024 03:11:13 -0700 (PDT) Received: from max.int.rpsys.net ([2001:8b0:aba:5f3c:19db:d42a:7902:cfcf]) by smtp.gmail.com with ESMTPSA id ffacd0b85a97d-3749ee99d6dsm1009420f8f.48.2024.08.29.03.11.12 for (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Thu, 29 Aug 2024 03:11:13 -0700 (PDT) From: Richard Purdie To: openembedded-core@lists.openembedded.org Subject: [PATCH 1/2] lz4: Fix static library reproducibility issue Date: Thu, 29 Aug 2024 11:11:11 +0100 Message-ID: <20240829101112.3986061-1-richard.purdie@linuxfoundation.org> X-Mailer: git-send-email 2.43.0 MIME-Version: 1.0 List-Id: X-Webhook-Received: from li982-79.members.linode.com [45.33.32.79] by aws-us-west-2-korg-lkml-1.web.codeaurora.org with HTTPS for ; Thu, 29 Aug 2024 10:11:23 -0000 X-Groupsio-URL: https://lists.openembedded.org/g/openembedded-core/message/203908 The autobuilder was seeing an intermittent reproducbility issue in lz4 with regard to symbol ordering in the static library. Add a patch to fix this which has been submitted upstream. Fix the SRC_URI whitespace and cleanup the patch directory naming whilst here. Signed-off-by: Richard Purdie --- .../lz4/lz4/reproducibility.patch | 33 +++++++++++++++++++ .../lz4/{files => lz4}/run-ptest | 0 meta/recipes-support/lz4/lz4_1.10.0.bb | 4 +-- 3 files changed, 35 insertions(+), 2 deletions(-) create mode 100644 meta/recipes-support/lz4/lz4/reproducibility.patch rename meta/recipes-support/lz4/{files => lz4}/run-ptest (100%) diff --git a/meta/recipes-support/lz4/lz4/reproducibility.patch b/meta/recipes-support/lz4/lz4/reproducibility.patch new file mode 100644 index 00000000000..c138986da89 --- /dev/null +++ b/meta/recipes-support/lz4/lz4/reproducibility.patch @@ -0,0 +1,33 @@ +lib/Makefile: Fix static library reproducibility + +The static library contents varies depending of the order of the object files on disk meaning +it isn't reproducible. + +To avoid this, use the SRCFILES values which are already sorted, mapped to the object names +instead. + + +Upstream-Status: Submitted [https://github.com/lz4/lz4/pull/1497] +Signed-off-by: Richard Purdie + +Index: git/lib/Makefile +=================================================================== +--- git.orig/lib/Makefile ++++ git/lib/Makefile +@@ -55,6 +55,7 @@ CFLAGS = $(DEBUGFLAGS) $(USERCFLAGS) + ALLFLAGS = $(CFLAGS) $(CPPFLAGS) $(LDFLAGS) + + SRCFILES := $(sort $(wildcard *.c)) ++OBJFILES = $(SRCFILES:.c=.o) + + include ../Makefile.inc + +@@ -102,7 +103,7 @@ liblz4.a: $(SRCFILES) + ifeq ($(BUILD_STATIC),yes) # can be disabled on command line + @echo compiling static library + $(COMPILE.c) $^ +- $(AR) rcs $@ *.o ++ $(AR) rcs $@ $(OBJFILES) + endif + + ifeq ($(WINBASED),yes) diff --git a/meta/recipes-support/lz4/files/run-ptest b/meta/recipes-support/lz4/lz4/run-ptest similarity index 100% rename from meta/recipes-support/lz4/files/run-ptest rename to meta/recipes-support/lz4/lz4/run-ptest diff --git a/meta/recipes-support/lz4/lz4_1.10.0.bb b/meta/recipes-support/lz4/lz4_1.10.0.bb index 40768fb2925..3b98adf42e5 100644 --- a/meta/recipes-support/lz4/lz4_1.10.0.bb +++ b/meta/recipes-support/lz4/lz4_1.10.0.bb @@ -13,8 +13,8 @@ PE = "1" SRCREV = "ebb370ca83af193212df4dcbadcc5d87bc0de2f0" SRC_URI = "git://github.com/lz4/lz4.git;branch=release;protocol=https \ - file://run-ptest \ - " + file://reproducibility.patch \ + file://run-ptest" UPSTREAM_CHECK_GITTAGREGEX = "v(?P.*)" S = "${WORKDIR}/git" From patchwork Thu Aug 29 10:11:12 2024 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Richard Purdie X-Patchwork-Id: 48447 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from aws-us-west-2-korg-lkml-1.web.codeaurora.org (localhost.localdomain [127.0.0.1]) by smtp.lore.kernel.org (Postfix) with ESMTP id 99F86C8302C for ; Thu, 29 Aug 2024 10:11:23 +0000 (UTC) Received: from mail-wr1-f46.google.com (mail-wr1-f46.google.com [209.85.221.46]) by mx.groups.io with SMTP id smtpd.web10.9435.1724926276086272468 for ; Thu, 29 Aug 2024 03:11:16 -0700 Authentication-Results: mx.groups.io; dkim=pass header.i=@linuxfoundation.org header.s=google header.b=eArYLroA; spf=pass (domain: linuxfoundation.org, ip: 209.85.221.46, mailfrom: richard.purdie@linuxfoundation.org) Received: by mail-wr1-f46.google.com with SMTP id ffacd0b85a97d-371bb8322b2so247412f8f.0 for ; Thu, 29 Aug 2024 03:11:15 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linuxfoundation.org; s=google; t=1724926274; x=1725531074; darn=lists.openembedded.org; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:to:from:from:to:cc:subject:date:message-id :reply-to; bh=0pISXm/ULhFkrm9Td6kkiUbUIlP8d34SSJDji1jIFTc=; b=eArYLroAIFlXocM0cjM8dgfENOfiARN7ZtRJeYltOXU28yqZtucVz6DKo7J2dK3s+3 BJD/8VVlP0wCqS1z1icOZQQt64ku6X/isxV5do9RPex/71h7AjBFnK27uCkfK5kybVct gAWmtCHxsfhw3mV1HmQ4njfjWAWksApSVAlIQ= X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1724926274; x=1725531074; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:to:from:x-gm-message-state:from:to:cc :subject:date:message-id:reply-to; bh=0pISXm/ULhFkrm9Td6kkiUbUIlP8d34SSJDji1jIFTc=; b=Rxu9FMXk5SSkunlxUIqY6czm1B/iwc2V84FfCuK7skamUNgDHDaX+/z7q7IYe+C749 tpFuKnwMPpGqz3GGAG07Ozbzqi5b4cmrTb+DXF6Qa9euJFs00oNE3ufPjoX6xoMDI4Uu uIz3ftxce6DezYw/dk4bSdrumqPSiqp40iy9qYi3ERi0XuGFiJ7hXETd/TYcH2AyVZYt GAwewfknYJRWLZMOyV84KUFrQ7EEbc19w3u6JGEs4xw8c2Uxso2P1M0/e/vXQ6C9IS/u b9Wshh+2e18kqYqUhUUDpUo6xb30dlCtO8bo/wsKayT8bnxnE3rruo4NIol+Lnncd/QV TInw== X-Gm-Message-State: AOJu0Yz1S8NQxTWZUhhSSsz4Cg1inSyCOYs8HBphygFEk76WVbhD+LVi bJY7YdSwbos0MeqaFj3dzsLoBtBVlyWjUmwmEnuJ2uF380AFwhqAnanxnq4wDX4X5L7PO/Y5Wvc b X-Google-Smtp-Source: AGHT+IG9jVL88lMjh3avmACD2/2LJ7FVYhYjyS80dLGSsRjIzKNDKy0SGhdPtkyQRTEEa8c2CP1F8A== X-Received: by 2002:a5d:4d0c:0:b0:360:9500:9bbb with SMTP id ffacd0b85a97d-374a012b767mr652776f8f.12.1724926274092; Thu, 29 Aug 2024 03:11:14 -0700 (PDT) Received: from max.int.rpsys.net ([2001:8b0:aba:5f3c:19db:d42a:7902:cfcf]) by smtp.gmail.com with ESMTPSA id ffacd0b85a97d-3749ee99d6dsm1009420f8f.48.2024.08.29.03.11.13 for (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Thu, 29 Aug 2024 03:11:13 -0700 (PDT) From: Richard Purdie To: openembedded-core@lists.openembedded.org Subject: [PATCH 2/2] lz4: Disable static libraries again Date: Thu, 29 Aug 2024 11:11:12 +0100 Message-ID: <20240829101112.3986061-2-richard.purdie@linuxfoundation.org> X-Mailer: git-send-email 2.43.0 In-Reply-To: <20240829101112.3986061-1-richard.purdie@linuxfoundation.org> References: <20240829101112.3986061-1-richard.purdie@linuxfoundation.org> MIME-Version: 1.0 List-Id: X-Webhook-Received: from li982-79.members.linode.com [45.33.32.79] by aws-us-west-2-korg-lkml-1.web.codeaurora.org with HTTPS for ; Thu, 29 Aug 2024 10:11:23 -0000 X-Groupsio-URL: https://lists.openembedded.org/g/openembedded-core/message/203909 This option was incorrectly removed in the recent upgrade. We don't use/need the static library so disable the build of it to save time/disk usage. Signed-off-by: Richard Purdie --- meta/recipes-support/lz4/lz4_1.10.0.bb | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/meta/recipes-support/lz4/lz4_1.10.0.bb b/meta/recipes-support/lz4/lz4_1.10.0.bb index 3b98adf42e5..f77a2e7bb95 100644 --- a/meta/recipes-support/lz4/lz4_1.10.0.bb +++ b/meta/recipes-support/lz4/lz4_1.10.0.bb @@ -23,7 +23,7 @@ inherit ptest CVE_STATUS[CVE-2014-4715] = "fixed-version: Fixed in r118, which is larger than the current version." -EXTRA_OEMAKE = "DESTDIR=${D}" +EXTRA_OEMAKE = "DESTDIR=${D} BUILD_STATIC=no" do_install() { oe_runmake install