mdadm: convert to generic-package
authorThomas Petazzoni <thomas.petazzoni@bootlin.com>
Sun, 14 Oct 2018 20:00:54 +0000 (22:00 +0200)
committerPeter Korsgaard <peter@korsgaard.com>
Mon, 15 Oct 2018 05:20:18 +0000 (07:20 +0200)
There is absolutely no reason for mdadm.mk to use autotools-package:
this package does not have any configure script at all, and its
Makefile is not generated using automake.

Therefore, convert it to use the generic-package
infrastructure. Compared to the previous code, we are now using
$(TARGET_CONFIGURE_OPTS), which passes our CPPFLAGS. This overrides
the CPPFLAGS from mdadm's Makefile, so we repeat the only CPPFLAGS
flag passed in the Makefile, -DBINDIR.

Signed-off-by: Thomas Petazzoni <thomas.petazzoni@bootlin.com>
Signed-off-by: Peter Korsgaard <peter@korsgaard.com>
package/mdadm/mdadm.mk

index 207b0be60dbf0364e7879abc4b9a4fe9d4e8afd4..6efade981d9fd0cfb34e6d27a3f4b3f571968886 100644 (file)
@@ -10,14 +10,19 @@ MDADM_SITE = $(BR2_KERNEL_MIRROR)/linux/utils/raid/mdadm
 MDADM_LICENSE = GPL-2.0+
 MDADM_LICENSE_FILES = COPYING
 
-MDADM_MAKE_OPTS = \
-       CFLAGS="$(TARGET_CFLAGS) -DNO_COROSYNC -DNO_DLM" CC="$(TARGET_CC)" CHECK_RUN_DIR=0 -C $(MDADM_DIR) mdadm
-
-MDADM_INSTALL_TARGET_OPTS = \
-       DESTDIR=$(TARGET_DIR)/usr -C $(MDADM_DIR) install-mdadm
+define MDADM_BUILD_CMDS
+       $(TARGET_MAKE_ENV) $(MAKE) -C $(@D) \
+               $(TARGET_CONFIGURE_OPTS) \
+               CFLAGS="$(TARGET_CFLAGS) -DNO_COROSYNC -DNO_DLM" \
+               CPPFLAGS="$(TARGET_CPPFLAGS) -DBINDIR=\\\"/usr/sbin\\\"" \
+               CHECK_RUN_DIR=0 \
+               mdadm
+endef
 
-define MDADM_CONFIGURE_CMDS
-       # Do nothing
+define MDADM_INSTALL_TARGET_CMDS
+       $(TARGET_MAKE_ENV) $(MAKE) -C $(@D) \
+               DESTDIR=$(TARGET_DIR)/usr \
+               install-mdadm
 endef
 
-$(eval $(autotools-package))
+$(eval $(generic-package))