Angel Pons has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/39123 )
Change subject: payloads/ext/tianocore/Makefile: Enable quiet mode ......................................................................
Patch Set 1:
(1 comment)
https://review.coreboot.org/c/coreboot/+/39123/1/payloads/external/tianocore... File payloads/external/tianocore/Makefile:
https://review.coreboot.org/c/coreboot/+/39123/1/payloads/external/tianocore... PS1, Line 54: BUILD_STR=-a IA32 -q -a X64 -t COREBOOT -p $(bootloader)/$(bootloader)Ia32X64.dsc -b $(BUILD_TYPE) $(TIMER) $(build_flavor)
Maybe use "-a IA32 -a X64 -q" […]
I get what you mean. I think it's better to factor out the common part of BUILD_STR anyway.
I'll put `-q` at the beginning, so that it appears in the same column on both lines.