From: Justin M. Forbes <jfor...@fedoraproject.org>

Print arch with process_configs errors

Now that we are capable of processing multiple config files in parallel,
we do not know exactly which arch a failure is seen on. This is
important as frequently errors are specific to a single arch. This patch
just prints the arch before the variant on failure messages to help us
determine the best way to correct an issue.

Signed-off-by: Justin M. Forbes <jfor...@fedoraproject.org>

diff --git a/redhat/configs/process_configs.sh 
b/redhat/configs/process_configs.sh
index blahblah..blahblah 100755
--- a/redhat/configs/process_configs.sh
+++ b/redhat/configs/process_configs.sh
@@ -104,7 +104,7 @@ checkoptions()
 
                ! $checkoptions_error && return
 
-               sed -i "1s/^/Error: Mismatches found in configuration files for 
${variant}\n/" .mismatches${count}
+               sed -i "1s/^/Error: Mismatches found in configuration files for 
${arch} ${variant}\n/" .mismatches${count}
                [ "$CONTINUEONERROR" ] || exit 1
        else
                rm -f .mismatches${count}
@@ -253,7 +253,7 @@ function process_config()
        grep -E 'CONFIG_' .listnewconfig${count} > .newoptions${count}
        if test -n "$NEWOPTIONS" && test -s .newoptions${count}
        then
-               echo "Found unset config items in ${variant}, please set them 
to an appropriate value" >> .errors${count}
+               echo "Found unset config items in ${arch} ${variant}, please 
set them to an appropriate value" >> .errors${count}
                cat .newoptions${count} >> .errors${count}
                rm .newoptions${count}
                RETURNCODE=1
@@ -264,7 +264,7 @@ function process_config()
        grep -E 'config.*warning' .listnewconfig${count} > .warnings${count}
        if test -n "$CHECKWARNINGS" && test -s .warnings${count}
        then
-               echo "Found misconfigured config items in ${variant}, please 
set them to an appropriate value" >> .errors${count}
+               echo "Found misconfigured config items in ${arch} ${variant}, 
please set them to an appropriate value" >> .errors${count}
                cat .warnings${count} >> .errors${count}
                rm .warnings${count}
                [ "$CONTINUEONERROR" ] || exit 1

--
https://gitlab.com/cki-project/kernel-ark/-/merge_requests/1643
_______________________________________________
kernel mailing list -- kernel@lists.fedoraproject.org
To unsubscribe send an email to kernel-le...@lists.fedoraproject.org
Fedora Code of Conduct: 
https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/kernel@lists.fedoraproject.org
Do not reply to spam on the list, report it: 
https://pagure.io/fedora-infrastructure

Reply via email to