diff options
| author | Leah Rowe <leah@libreboot.org> | 2025-08-01 05:27:33 +0100 |
|---|---|---|
| committer | Leah Rowe <leah@libreboot.org> | 2025-08-01 05:32:25 +0100 |
| commit | a499d5bba2f9f35dd10c0bab4b9c2313f744da33 (patch) | |
| tree | e23a4ba0a3959345f33dfc8bd184d0afa3b75ee1 /include/rom.sh | |
| parent | 84a1ff85b0706d7eb47118dd37f0f16443d48108 (diff) | |
rom.sh: Don't run mkhelpers if release=n
This fixes the following error on ./mk release:
cp: cannot stat 'elf/coreboot/default/d510mo/libgfxinit_txtmode/coreboot.rom': No such file or directory
I recently re-wrote the handling of coreboot images, and
I overlooked this entirely. When a given target specifies
release=n, it has to be skipped, so builds are not done.
The "release" variable is already checked, in mkcoreboottar.
Let's also put it in the other mkhelper functions, to make sure
there are no errors.
Signed-off-by: Leah Rowe <leah@libreboot.org>
Diffstat (limited to 'include/rom.sh')
| -rw-r--r-- | include/rom.sh | 5 |
1 files changed, 5 insertions, 0 deletions
diff --git a/include/rom.sh b/include/rom.sh index 57948ec4..edfd81e0 100644 --- a/include/rom.sh +++ b/include/rom.sh @@ -45,6 +45,8 @@ mkpayload_grub() corebootpremake() { + [ "$XBMK_RELEASE" = "y" ] && [ "$release" = "n" ] && return 0; : + [ -n "$mode" ] || [ ! -f "$srcdir/.config" ] || $dry printf \ "CONFIG_CCACHE=y\n" >> "$srcdir/.config" || err "$srcdir: !cook"; : fx_ check_coreboot_util printf "cbfstool\nifdtool\n" @@ -76,11 +78,14 @@ check_coreboot_util() coreboot_pad_one_byte() { + [ "$XBMK_RELEASE" = "y" ] && [ "$release" = "n" ] && return 0; : $dry pad_one_byte "$srcdir/build/coreboot.rom" } mkcorebootbin() { + [ "$XBMK_RELEASE" = "y" ] && [ "$release" = "n" ] && return 0; : + $dry fx_ check_coreboot_util printf "cbfstool\nifdtool\n" for y in "$target_dir/config"/*; do |