diff mbox series

[meta-arago,master] coremark: Clean up buildpaths QA error

Message ID 20251201184100.1372032-1-reatmon@ti.com
State Accepted
Delegated to: Ryan Eatmon
Headers show
Series [meta-arago,master] coremark: Clean up buildpaths QA error | expand

Commit Message

Ryan Eatmon Dec. 1, 2025, 6:41 p.m. UTC 
The coremark binary includes a copy of the compiler flags in the output
which breaks the reproducible build tenet of Yocto.  This patch simply
replaces COMPILER_FLAGS with "".

Signed-off-by: Ryan Eatmon <reatmon@ti.com>
---
 ..._main-Remove-printing-compiler-flags.patch | 41 +++++++++++++++++++
 .../coremark/coremark_git.bb                  |  5 ++-
 2 files changed, 45 insertions(+), 1 deletion(-)
 create mode 100644 meta-arago-test/recipes-benchmark/coremark/coremark/0001-core_main-Remove-printing-compiler-flags.patch

Comments

PRC Automation Dec. 1, 2025, 6:45 p.m. UTC | #1 
meta-arago / na / 20251201184100.1372032-1-reatmon

PRC Results: FAIL

=========================================================
  check-yocto-patches: PASS
=========================================================
Patches
----------------------------------------
All patches passed



=========================================================
  apply-yocto-patch: FAIL
=========================================================
master
=====================
Summary:
- Patch Series: [meta-arago][master][PATCH] coremark: Clean up buildpaths QA error
- Submitter: From: Ryan Eatmon <reatmon@ti.com>
+From: Ryan Eatmon <reatmon@ti.com>
- Date: Date: Mon, 1 Dec 2025 12:41:00 -0600
+Date: Mon, 1 Dec 2025 12:21:23 -0600
- Num Patches: 1
- Mailing List (public inbox) Commit SHA: 170e989365362f8ba8a6ac2bd3f0ac92d01f8759

Applied to:
- Repository: lcpd-prc-meta-arago
- Base Branch: master
- Commit Author: LCPD Automation Script <lcpdbld@list.ti.com>
- Commit Subject: CI/CD Auto-Merger: cicd.master.202511180100
- Commit SHA: 82a437373896b01cd89d456dcc9cb28521b64ac7

Patches
----------------------------------------
FAIL - [meta-arago][master][PATCH] coremark: Clean up buildpaths QA error
    .git/rebase-apply/patch:51: trailing whitespace.
    -- 
    .git/rebase-apply/patch:53: new blank line at EOF.
    +
    error: meta-arago-test/recipes-benchmark/coremark/coremark_git.bb: does not exist in index
    Applying: coremark: Clean up buildpaths QA error
    Patch failed at 0001 coremark: Clean up buildpaths QA error
diff mbox series

Patch

diff --git a/meta-arago-test/recipes-benchmark/coremark/coremark/0001-core_main-Remove-printing-compiler-flags.patch b/meta-arago-test/recipes-benchmark/coremark/coremark/0001-core_main-Remove-printing-compiler-flags.patch
new file mode 100644
index 00000000..ef7c78b2
--- /dev/null
+++ b/meta-arago-test/recipes-benchmark/coremark/coremark/0001-core_main-Remove-printing-compiler-flags.patch
@@ -0,0 +1,41 @@ 
+From 0af4e2298ccacc8afc0294b7a731f0e866155427 Mon Sep 17 00:00:00 2001
+From: Ryan Eatmon <reatmon@ti.com>
+Date: Mon, 1 Dec 2025 12:21:23 -0600
+Subject: [PATCH] core_main: Remove printing compiler flags
+
+Reproducible builds do not like including anything that references the
+build environment on the host machine.  This would include printing out
+the COMPILER_FLAGS macro.  Substitute it with "".
+
+Upstream-Status: Inappropriate [OE-Specific]
+
+Signed-off-by: Ryan Eatmon <reatmon@ti.com>
+---
+ core_main.c | 4 ++--
+ 1 file changed, 2 insertions(+), 2 deletions(-)
+
+diff --git a/core_main.c b/core_main.c
+index a4beeb6..f0ab87f 100644
+--- a/core_main.c
++++ b/core_main.c
+@@ -380,7 +380,7 @@ for (i = 0; i < MULTITHREAD; i++)
+     ee_printf("Iterations       : %lu\n",
+               (long unsigned)default_num_contexts * results[0].iterations);
+     ee_printf("Compiler version : %s\n", COMPILER_VERSION);
+-    ee_printf("Compiler flags   : %s\n", COMPILER_FLAGS);
++    ee_printf("Compiler flags   : %s\n", ""); //Reproducible build fix: COMPILER_FLAGS);
+ #if (MULTITHREAD > 1)
+     ee_printf("Parallel %s : %d\n", PARALLEL_METHOD, default_num_contexts);
+ #endif
+@@ -410,7 +410,7 @@ for (i = 0; i < MULTITHREAD; i++)
+                       default_num_contexts * results[0].iterations
+                           / time_in_secs(total_time),
+                       COMPILER_VERSION,
+-                      COMPILER_FLAGS);
++                      ""); //Reproducible build fix: COMPILER_FLAGS);
+ #if defined(MEM_LOCATION) && !defined(MEM_LOCATION_UNSPEC)
+             ee_printf(" / %s", MEM_LOCATION);
+ #else
+-- 
+2.43.0
+
diff --git a/meta-arago-test/recipes-benchmark/coremark/coremark_git.bb b/meta-arago-test/recipes-benchmark/coremark/coremark_git.bb
index 1e002c06..9d770c44 100644
--- a/meta-arago-test/recipes-benchmark/coremark/coremark_git.bb
+++ b/meta-arago-test/recipes-benchmark/coremark/coremark_git.bb
@@ -3,7 +3,10 @@  SUMMARY = "The benchmark utility to measure the performance of CPU"
 LICENSE = "Coremark"
 LIC_FILES_CHKSUM = "file://LICENSE.md;md5=0a18b17ae63deaa8a595035f668aebe1"
 
-SRC_URI = "git://github.com/eembc/coremark.git;branch=main;protocol=https"
+SRC_URI = " \
+    git://github.com/eembc/coremark.git;branch=main;protocol=https \
+    file://0001-core_main-Remove-printing-compiler-flags.patch \
+"
 PV = "1.01+git"
 SRCREV = "1f483d5b8316753a742cbf5590caf5bd0a4e4777"