Skip to contentSkip to conversation

eigen: add an option to install unsupported modules

PATCH New Davide Viti 2 participants 2 messages Mar 30, 2014 – Apr 3, 2014

lore
Share & tools
lore

Davide Viti

author
#0 lore raw

PATCH v6 1/1 eigen: add an option to install unsupported modules

Signed-off-by Davide Viti <[email protected]>

---
Dear Thomas,
here is a new version of the patch

Changes v5 -> v6:
- Make sure "unsupported" dir is created before copying any file (suggested by Thomas)
- install only header files in staging (suggested by Thomas)
Changes v4 -> v5:
- fix indentation (spaces > tabs)

regards,

Davide

 package/eigen/Config.in | 8 ++++++++
 package/eigen/eigen.mk  | 8 ++++++++
 2 files changed, 16 insertions(+)
package/eigen/Config.in +8 -0
index e94f9a3..03d65b1 100644
@@ -13,5 +13,13 @@ config BR2_PACKAGE_EIGEN
 
 	  http://eigen.tuxfamily.org/
 
+if BR2_PACKAGE_EIGEN
+
+config BR2_PACKAGE_EIGEN_UNSUPPORTED_MODULES
+	bool "unsupported modules"
+	help
+	  Install eigen unsupported modules
+endif
+
 comment "eigen needs a toolchain w/ C++"
 	depends on !BR2_INSTALL_LIBSTDCPP
package/eigen/eigen.mk +8 -0
index 9a5ce7d..7fd3187 100644
@@ -14,12 +14,20 @@ EIGEN_INSTALL_STAGING = YES
 EIGEN_INSTALL_TARGET = NO
 EIGEN_DEST_DIR = $(STAGING_DIR)/usr/include/eigen3
 
+ifeq ($(BR2_PACKAGE_EIGEN_UNSUPPORTED_MODULES),y)
+define EIGEN_INSTALL_UNSUPPORTED_MODULES_CMDS
+	mkdir -p $(EIGEN_DEST_DIR)/unsupported
+	cp -a $(@D)/unsupported/Eigen $(EIGEN_DEST_DIR)/unsupported
+endef
+endif
+
 # This package only consists of headers that need to be
 # copied over to the sysroot for compile time use
 define EIGEN_INSTALL_STAGING_CMDS
 	$(RM) -r $(EIGEN_DEST_DIR)
 	mkdir -p $(EIGEN_DEST_DIR)
 	cp -a $(@D)/Eigen $(EIGEN_DEST_DIR)
+	$(EIGEN_INSTALL_UNSUPPORTED_MODULES_CMDS)
 endef
 
 $(eval $(generic-package))
Signature · 1 line
1.9.1
Applied
#1 lore raw

PATCH v6 1/1 eigen: add an option to install unsupported modules

Dear Davide Viti,

Quoted 1 line · Davide Viti
Signed-off-by Davide Viti <[email protected]>

Applied, thanks.

Thomas

Signature · 3 lines
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com