Attention is currently required from: Ashish Kumar Mishra, Nico Huber, Saurabh Mishra.
Hello Ashish Kumar Mishra, Nico Huber, Saurabh Mishra, build bot (Jenkins),
I'd like you to reexamine a change. Please visit
https://review.coreboot.org/c/coreboot/+/83887?usp=email
to look at the new patch set (#3).
Change subject: Fix no-op for empty arg in Makefile for Bash 5.2.21 ......................................................................
Fix no-op for empty arg in Makefile for Bash 5.2.21
This change addresses an issue in Ubuntu 24.04 with GNU Bash version 5.2.21, where the Makefile failed with the error:
/bin/bash: -c: option requires an argument make: *** [Makefile.mk:1261: build/coreboot.pre] Error 2
The error was due to an empty third argument `$(3)` in a conditional `$(if ...)` statement. The original line did not have an explicit no-op for the case when `$(3)` is empty, leading to an incomplete command.
By adding a colon `:` as a no-op command in the else-part of the `$(if ...)`, we ensure execution of a harmless command when `$(3)` is empty, preventing the Makefile from failing due to a missing argument.
This fix ensures compatibility with the newer Bash version and prevents the Makefile from breaking under the specified conditions.
Change-Id: I276e89792779832edf325d4f781aa8df101e9091 Signed-off-by: Manigandan, Balaji balaji.manigandan@intel.com --- M Makefile.mk 1 file changed, 1 insertion(+), 1 deletion(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/87/83887/3