From e86af9a60a6165188ec069ea519eeadb1e2ad3cc Mon Sep 17 00:00:00 2001 From: Leah Rowe Date: Sun, 22 Oct 2023 12:31:55 +0100 Subject: 20231021hotfix: replace x_ with err in some places keymaps weren't being set in keymay.cfg of cbfs, due to use of x_ in the rom script, and x_ doesn't handle quotes or spaces in arguments well. i'm going to remove use of x_ and xx_ (it's in my todo), for next release. for now, hot patch the release. i've gone through and replaced use of x_ with || err, in some places. not just the keymap.cfg command, but others too. in case there are more issues we missed. this commit is being tagged "20231021fix" and i'm using this tag to re-build the 20231021 release. i'll just replace the tarballs in rsync and add errata to the news page announcing the release. all i did was break peoples umlauts, i didn't brick their machines fortunately! very minor bug. anyway, x_/xx_ is a great idea, but sh isn't really designed for that style of programming. i'll go back to using just || err in the next release. Signed-off-by: Leah Rowe --- script/build/grub | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) (limited to 'script/build/grub') diff --git a/script/build/grub b/script/build/grub index ba0845d3..31fc0084 100755 --- a/script/build/grub +++ b/script/build/grub @@ -26,7 +26,7 @@ handle_dependencies() [ -d "src/grub" ] || x_ ./update trees -f grub [ -f "src/grub/grub-mkstandalone" ] || build_grub_utils x_ mkdir -p "${elfdir}" - x_ rm -f "${elfdir}/"* + rm -f "${elfdir}/"* || err "!rm elf, handle_, ${elfdir}/" } build_grub_utils() @@ -37,7 +37,7 @@ build_grub_utils() x_ ./bootstrap --gnulib-srcdir=gnulib/ --no-git x_ ./autogen.sh x_ ./configure --with-platform=coreboot - x_ make -j$(nproc) FS_PAYLOAD_MODULES="" + make -j$(nproc) FS_PAYLOAD_MODULES="" || err "!mk grub utils" ) } @@ -47,8 +47,8 @@ build_keymap_configs() [ -f "${keylayoutfile}" ] || continue keymap="${keylayoutfile##${grubcfgsdir}/keymap/}" keymap="${keymap%.gkb}" - x_ printf "keymap %s\n" "${keymap}" > \ - "${elfdir}/keymap_${keymap}.cfg" + printf "keymap %s\n" "${keymap}" > \ + "${elfdir}/keymap_${keymap}.cfg" || err "!insert keymap" done } -- cgit v1.2.1