[Buildroot] [PATCH 1/3] ltp-testsuite: change install prefix
Erico Nunes
nunes.erico at gmail.com
Sat Oct 1 11:50:22 UTC 2016
ltp-testsuite installs some of its files directly at its specified
prefix, which by default in Buildroot is /usr for autotools packages.
This is currently leaving scripts and its internal directories directly
installed in /usr, such as in:
# ls /usr
IDcheck.sh lib64 runltplite.sh share
Version libexec runtest testcases
bin runalltests.sh sbin testscripts
lib runltp scenario_groups ver_linux
It also seems to be not feasible to try to install all of its testcases
to standard locations such as /usr/bin as may contain hundreds of
binaries and may not be able to find them all if we change their install
locations. Therefore, it is better if ltp-testsuite installs its tree to
its own self-contained subdirectory.
Upstream instructions recommend that the install path be /opt/ltp,
however it seems that installing things to /opt is not a very common
Buildroot practice.
The proposal then is to install it to /usr/lib/ltp-testsuite and so it
can be run directly from there.
Signed-off-by: Erico Nunes <nunes.erico at gmail.com>
---
The /usr/lib/ltp-testsuite suggestion came from a brief discussion that
I had on IRC about where would be the least bad place to install it.
Alternatives could be /usr/share/ltp-testsuite or even /opt/ltp if we
were to strictly follow the upstream documentation.
It doesn't matter too much really as long as it doesn't mess /usr up
anymore.
---
package/ltp-testsuite/ltp-testsuite.mk | 4 ++++
1 file changed, 4 insertions(+)
diff --git a/package/ltp-testsuite/ltp-testsuite.mk b/package/ltp-testsuite/ltp-testsuite.mk
index cf48072..e97659e 100644
--- a/package/ltp-testsuite/ltp-testsuite.mk
+++ b/package/ltp-testsuite/ltp-testsuite.mk
@@ -21,6 +21,10 @@ else
LTP_TESTSUITE_CONF_OPTS += --without-modules
endif
+# We change the prefix to a custom one, otherwise we get scripts and
+# directories directly in /usr, such as /usr/runalltests.sh
+LTP_TESTSUITE_CONF_OPTS += --prefix=/usr/lib/ltp-testsuite
+
# Needs libcap with file attrs which needs attr, so both required
ifeq ($(BR2_PACKAGE_LIBCAP)$(BR2_PACKAGE_ATTR),yy)
LTP_TESTSUITE_DEPENDENCIES += libcap
--
2.7.4
More information about the buildroot
mailing list