On Fri, 13 Nov 2020 21:12:19 -0000, GitLab Bridge on behalf of bcrocker wrote: > - arch="$(echo "$file" | sed "s/$PACKAGE_NAME-\([^-]*\).*.config/\1/")" > + arch="${file/$PACKAGE_NAME-/}" > + arch="${arch/-*/}" Just noting this is not equal. It doesn't seem to matter in practice, though, so go on with this, it's a good change. > - test ! -e $path && echo "$value" > $path && continue > + test ! -e "$path": && echo "$value" > "$path" && continue The colon seems to be a typo. This introduces a bug, the echo will always trigger. > - for f in $(ls -1 $WORK/kernel*.config) > + for f in "$WORK"/kernel*.config This behaves differently when there's no kernel*.config. Are you sure it cannot happen? It probably can't but please double check. > - make ARCH=$arch listnewconfig 2>/dev/null | grep -E 'CONFIG_' | cut -d'#' -f1 > $WORK/$config > + make ARCH="$arch" listnewconfig 2>/dev/null | grep -E 'CONFIG_' | cut -d'#' -f1 > $WORK/"$config" Curious, why the tool you're using is not flagging $WORK as a candidate for quotation? Jiri _______________________________________________ kernel mailing list -- kernel@xxxxxxxxxxxxxxxxxxxxxxx To unsubscribe send an email to kernel-leave@xxxxxxxxxxxxxxxxxxxxxxx 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@xxxxxxxxxxxxxxxxxxxxxxx