From: Anthony PERARD <anthony.perard@vates.tech>
Date: Wed, 26 Mar 2025 14:28:25 +0000 (+0000)
Subject: automation: Avoid changing source files for randconfig tests
X-Git-Url: http://xenbits.xensource.com/gitweb?a=commitdiff_plain;h=e43d9621d8081a88397e2b49b08906c4f4acaa57;p=xen.git

automation: Avoid changing source files for randconfig tests

We should avoid changing files from the source tree if we don't intend
to commit the result.

We don't really need to check if $EXTRA_FIXED_RANDCONFIG is empty so
add it to the temporary file in all cases.

Signed-off-by: Anthony PERARD <anthony.perard@vates.tech>
Acked-by: Stefano Stabellini <sstabellini@kernel.org>
---

diff --git a/automation/scripts/build b/automation/scripts/build
index a348c781d1..cdb8cd7c72 100755
--- a/automation/scripts/build
+++ b/automation/scripts/build
@@ -12,12 +12,12 @@ $cc --version
 # random config or default config
 if [[ "${RANDCONFIG}" == "y" ]]; then
 
+    cp -f xen/tools/kconfig/allrandom.config xen/allrandom.config.tmp
+
     # Append job-specific fixed configuration
-    if [[ -n "${EXTRA_FIXED_RANDCONFIG}" ]]; then
-        echo "${EXTRA_FIXED_RANDCONFIG}" >> xen/tools/kconfig/allrandom.config
-    fi
+    echo "${EXTRA_FIXED_RANDCONFIG}" >> xen/allrandom.config.tmp
 
-    make -j$(nproc) -C xen KCONFIG_ALLCONFIG=tools/kconfig/allrandom.config randconfig
+    make -j$(nproc) -C xen KCONFIG_ALLCONFIG=allrandom.config.tmp randconfig
 
     # RANDCONFIG implies HYPERVISOR_ONLY
     HYPERVISOR_ONLY="y"