imagebuilder: fix multiple issue with manifest and sign keys handling

Fix multiple issue with manifest handling where APK was hardcoded and fix a logic error where (TODO) APK _check_keys was called for the OPKG codepath instead of correctly calling for the APK codepath. Fixes: d788ab3 ("build: add APK package build capabilities") Signed-off-by: Christian Marangi <[email protected]>
lede-project · May 19, 2024 · 5d37d8d · 5d37d8d
1 parent 65c1f0d
commit 5d37d8d
Showing 1 changed file with 10 additions and 2 deletions.
diff --git a/target/imagebuilder/files/Makefile b/target/imagebuilder/files/Makefile
@@ -159,7 +159,11 @@ _call_manifest: FORCE
 	mkdir -p $(TARGET_DIR) $(BIN_DIR) $(TMP_DIR) $(DL_DIR)
 	$(MAKE) package_reload >/dev/null
 	$(MAKE) package_install >/dev/null
+ifeq ($(CONFIG_USE_APK),)
+	$(OPKG) list-installed $(if $(STRIP_ABI),--strip-abi)
+else
 	$(APK) list --quiet --manifest --no-network
+endif
 
 package_index: FORCE
 	@echo >&2
@@ -202,7 +206,11 @@ endif
 
 package_list: FORCE
 	@$(MAKE) -s package_reload
+ifeq ($(CONFIG_USE_APK),)
+	@$(OPKG) list --size 2>/dev/null
+else
 	@$(APK) list --size 2>/dev/null
+endif
 
 package_install: FORCE
 	@echo
@@ -277,8 +285,6 @@ endif
 
 _check_keys: FORCE
 ifeq ($(CONFIG_USE_APK),)
-	# TODO
-else
 ifneq ($(CONFIG_SIGNATURE_CHECK),)
 	@if [ ! -s $(BUILD_KEY) -o ! -s $(BUILD_KEY).pub ]; then \
 		echo Generate local signing keys... >&2; \
@@ -294,6 +300,8 @@ ifneq ($(CONFIG_SIGNATURE_CHECK),)
 			-s $(BUILD_KEY); \
 	fi
 endif
+else
+	# TODO
 endif
 
 image: