From mboxrd@z Thu Jan 1 00:00:00 1970 From: Thomas Petazzoni Date: Wed, 6 Jul 2016 21:56:07 +0200 Subject: [Buildroot] [PATCH v2] openblas: add an option to install tests on the target In-Reply-To: <1467717365-27849-1-git-send-email-Vincent.Riera@imgtec.com> References: <1467717365-27849-1-git-send-email-Vincent.Riera@imgtec.com> Message-ID: <20160706215607.785816dd@free-electrons.com> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: buildroot@busybox.net Hello, On Tue, 5 Jul 2016 12:16:05 +0100, Vicente Olivert Riera wrote: > +ifeq ($(BR2_PACKAGE_OPENBLAS_INSTALL_TESTS),y) > +define OPENBLAS_INSTALL_TESTS > + for i in test ctest utest; do \ > + mkdir -p $(TARGET_DIR)/usr/share/openblas/$$i && \ > + cd $(@D)/$$i && \ > + find . -type f -perm -a=x -exec $(INSTALL) -m 0755 {} $(TARGET_DIR)/usr/share/openblas/$$i/{} \; && \ > + find . -type f -name "?in*" -exec $(INSTALL) -m 0644 {} $(TARGET_DIR)/usr/share/openblas/$$i/{} \; && \ > + find . -type f -name "*.dat" -exec $(INSTALL) -m 0644 {} $(TARGET_DIR)/usr/share/openblas/$$i/{} \; ; \ > + done > +endef > +OPENBLAS_POST_INSTALL_TARGET_HOOKS += OPENBLAS_INSTALL_TESTS > +endif To be honest, I really dislike such complicated install procedures. Could you instead work with upstream to have them add an appropriate target that allows to install the test programs? Looking at the upstream Github repository, they look very active, so getting some feedback should be possible. Thanks, Thomas -- Thomas Petazzoni, CTO, Free Electrons Embedded Linux, Kernel and Android engineering http://free-electrons.com