utils: lpac: fix passing CMAKE options
authorRobert Marko <robert.marko@sartura.hr>
Wed, 3 Jul 2024 11:43:16 +0000 (13:43 +0200)
committerRobert Marko <robimarko@gmail.com>
Wed, 3 Jul 2024 18:19:12 +0000 (20:19 +0200)
Turns out that having a comment for QMI over QRTR in the CMAKE_OPTIONS will
drop anything after it, so lets move the comment above CMAKE_OPTIONS.

Fixes: 34f9d96b4cc3 ("lpac: make APDU backends configurable")
Signed-off-by: Robert Marko <robert.marko@sartura.hr>
utils/lpac/Makefile

index ddb0f18151a107a25d318affd590645439677b67..86da6abe03ba9d73ca61bda5481186c336b93f85 100644 (file)
@@ -4,7 +4,7 @@ include $(TOPDIR)/rules.mk
 
 PKG_NAME:=lpac
 PKG_VERSION:=2.0.2
-PKG_RELEASE:=2
+PKG_RELEASE:=3
 
 PKG_SOURCE:=$(PKG_NAME)-$(PKG_VERSION).tar.gz
 PKG_SOURCE_URL:=https://codeload.github.com/estkme-group/lpac/tar.gz/refs/tags/v$(PKG_VERSION)?
@@ -45,10 +45,10 @@ endef
 
 TARGET_CFLAGS += $(FPIC)
 
+# libqmi 1.35.4 or newer is required for QMI over QRTR
 CMAKE_OPTIONS += \
   -DLPAC_WITH_APDU_PCSC=$(if $(CONFIG_LPAC_WITH_PCSC),ON,OFF) \
   -DLPAC_WITH_APDU_AT=$(if $(CONFIG_LPAC_WITH_AT),ON,OFF) \
-  # libqmi 1.35.4 or newer is required for this
   -DLPAC_WITH_APDU_QMI_QRTR=OFF
 
 define Package/lpac/install