uboot-tools: dont build tools unconditionally
authorRobert Marko <[email protected]>
Sun, 23 Mar 2025 09:29:28 +0000 (10:29 +0100)
committerRobert Marko <[email protected]>
Mon, 24 Mar 2025 17:43:44 +0000 (18:43 +0100)
Currently, both envtools and the rest of U-Boot tools are being built
regardless if the dumpimage package has been selected.

This will fail if only envtools are selected since the rest of tools
require OpenSSL while envtools do not require them.

So, only build tools if dumpimage is selected.

Fixes: 46e376c93514 ("uboot-tools: migrate uboot-envtools to uboot-tools")
Fixes: #18327
Tested-by: Martin Blumenstingl <[email protected]>
Link: https://github.com/openwrt/openwrt/pull/18329
Signed-off-by: Robert Marko <[email protected]>
package/boot/uboot-tools/Makefile

index 54b4647d24831a712a7931bd62155b26faa11128..dd43c65e5cd77fc11ea439a9573e4424c37ccdb0 100644 (file)
@@ -59,8 +59,8 @@ define Package/uboot-envtools/conffiles
 endef
 
 define Build/Configure
-       $(MAKE) -C $(PKG_BUILD_DIR) tools-only_defconfig
-       $(MAKE) -C $(PKG_BUILD_DIR) syncconfig
+       $(call Build/Compile/Default,tools-only_defconfig)
+       $(call Build/Compile/Default,syncconfig)
        $(SED) 's/CONFIG_TOOLS_LIBCRYPTO=y/# CONFIG_TOOLS_LIBCRYPTO is not set/' $(PKG_BUILD_DIR)/.config
 endef
 
@@ -71,8 +71,14 @@ MAKE_FLAGS += \
        NO_PYTHON=1
 
 define Build/Compile
+
+ifneq ($(CONFIG_PACKAGE_uboot-envtools),)
        $(call Build/Compile/Default,envtools)
+endif
+ifneq ($(CONFIG_PACKAGE_dumpimage),)
        $(call Build/Compile/Default,cross_tools)
+endif
+
 endef
 
 define Package/dumpimage/install