linux: ensure correct make targets are used for cuImage/simpleImage.<dtb>
PATCH 1/1 linux: ensure correct make targets are used for cuImage/simpleImage.<dtb>
Fixes https://gitlab.com/buildroot.org/buildroot/-/jobs/66561794
LINUX_DTS_NAME may end up with a leading space because of the += logic, and may contain multiple dts files - Neither of which works when we construct the {cu,simple}Image.$(LINUX_DTS_NAME) make target name.
Fix it by using the first word in the variable.
--- linux/linux.mk | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-)
linux/linux.mk +2 -2
@@ -155,9 +155,9 @@ LINUX_IMAGE_NAME = zImage.epapr else ifeq ($(BR2_LINUX_KERNEL_APPENDED_ZIMAGE),y) LINUX_IMAGE_NAME = zImage else ifeq ($(BR2_LINUX_KERNEL_CUIMAGE),y) -LINUX_IMAGE_NAME = cuImage.$(LINUX_DTS_NAME) +LINUX_IMAGE_NAME = cuImage.$(firstword $(LINUX_DTS_NAME)) else ifeq ($(BR2_LINUX_KERNEL_SIMPLEIMAGE),y) -LINUX_IMAGE_NAME = simpleImage.$(LINUX_DTS_NAME) +LINUX_IMAGE_NAME = simpleImage.$(firstword $(LINUX_DTS_NAME)) else ifeq ($(BR2_LINUX_KERNEL_IMAGE),y) LINUX_IMAGE_NAME = Image else ifeq ($(BR2_LINUX_KERNEL_LINUX_BIN),y)
Signature · 1 line
2.11.0
PATCH 1/1 linux: ensure correct make targets are used for cuImage/simpleImage.<dtb>
Quoted 10 lines
Quoted 1 line
Quoted 1 line
Quoted 1 line
Quoted 1 line
"Peter" == Peter Korsgaard <[email protected]> writes:
Fixes https://gitlab.com/buildroot.org/buildroot/-/jobs/66561794
LINUX_DTS_NAMEmay end up with a leading space because of the += logic, and may contain multiple dts files - Neither of which works when we construct the {cu,simple}Image.$(LINUX_DTS_NAME) make target name.Fix it by using the first word in the variable.
Signed-off-by Peter Korsgaard <[email protected]>
Committed, thanks.
Signature · 1 line
Bye, Peter Korsgaard
PATCH 1/1 linux: ensure correct make targets are used for cuImage/simpleImage.<dtb>
In reply to Peter KorsgaardQuoted 10 lines
Quoted 1 line
Quoted 1 line
Quoted 1 line
Quoted 1 line
"Peter" == Peter Korsgaard <[email protected]> writes:
Fixes https://gitlab.com/buildroot.org/buildroot/-/jobs/66561794
LINUX_DTS_NAMEmay end up with a leading space because of the += logic, and may contain multiple dts files - Neither of which works when we construct the {cu,simple}Image.$(LINUX_DTS_NAME) make target name.Fix it by using the first word in the variable.
Signed-off-by Peter Korsgaard <[email protected]>
Committed to 2018.02.x, thanks.
Signature · 1 line
Bye, Peter Korsgaard