All of lore.kernel.org
 help / color / mirror / Atom feed
* [LTP] [RFC PATCH 0/4] Travis enhancements
@ 2017-11-28 23:21 Petr Vorel
  2017-11-28 23:21 ` [LTP] [RFC PATCH 1/4] travis: Add build script Petr Vorel
                   ` (4 more replies)
  0 siblings, 5 replies; 17+ messages in thread
From: Petr Vorel @ 2017-11-28 23:21 UTC (permalink / raw)
  To: ltp

Hi,

I've added several travis enhancements, most notably added 32bit cross-compile
build and added more compilers (add gcc-7, clang-4.0, clang-5.0, remove clang-3.9).
Now there is 12 builds instead of 9, speed increased from ~13 min [1] to ~26 min [2],
which is maybe to much.  If we don't like it, we can decide which compilers to have.

BTW I have more plans: adding cross-compile arm build and out of tree build
(this one needs to be fixed).

[1] https://travis-ci.org/pevik-travis/ltp/builds/197584686
[2] https://travis-ci.org/pevik-travis/ltp/builds/308683533

Kind regards,
Petr

Petr Vorel (4):
  travis: Add build script
  travis: Add 32 cross-compile build + use build.sh
  travis: Add gcc-7, clang-4.0, clang-5.0, remove clang-3.9
  travis: Install all dependencies (headers and libraries)

 .travis.yml | 74 +++++++++++++++++++++++++++++++++++++++++++++++++++++++++----
 build.sh    | 68 ++++++++++++++++++++++++++++++++++++++++++++++++++++++++
 2 files changed, 138 insertions(+), 4 deletions(-)
 create mode 100755 build.sh

-- 
2.15.0


^ permalink raw reply	[flat|nested] 17+ messages in thread

* [LTP] [RFC PATCH 1/4] travis: Add build script
  2017-11-28 23:21 [LTP] [RFC PATCH 0/4] Travis enhancements Petr Vorel
@ 2017-11-28 23:21 ` Petr Vorel
  2017-11-29  8:13   ` Li Wang
  2017-11-28 23:21 ` [LTP] [RFC PATCH 2/4] travis: Add 32 cross-compile build + use build.sh Petr Vorel
                   ` (3 subsequent siblings)
  4 siblings, 1 reply; 17+ messages in thread
From: Petr Vorel @ 2017-11-28 23:21 UTC (permalink / raw)
  To: ltp

This script is to be used for travis build, but can be used for local
builds as well.

For usage run
./build.sh -h

Signed-off-by: Petr Vorel <pvorel@suse.cz>
---
 .travis.yml |  2 +-
 build.sh    | 68 +++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
 2 files changed, 69 insertions(+), 1 deletion(-)
 create mode 100755 build.sh

diff --git a/.travis.yml b/.travis.yml
index d937f9dcf..f2d51f131 100644
--- a/.travis.yml
+++ b/.travis.yml
@@ -72,4 +72,4 @@ notifications:
     email:
         secure: "b/xcA/K5OyQvPPnd0PRahTH5LJu8lgz8goGHvhXpHo+ZPsPgTDXNFo5cX9fSOkMuFKeoW8iGl0wOgK2+ptc8mbYDw277K4RFIHRHeV/KIoE1EzjQnEFiL8J0oHCAvDj12o0AXeriTyY9gICXKbR31Br6Zh5eKViDJe2OAGeHeDU="
 
-script: make autotools && ./configure --prefix $HOME/ltp --with-open-posix-testsuite --with-realtime-testsuite && make -j$(getconf _NPROCESSORS_ONLN) && make -j$(getconf _NPROCESSORS_ONLN) install
+script: ./build.sh
diff --git a/build.sh b/build.sh
new file mode 100755
index 000000000..3387c120d
--- /dev/null
+++ b/build.sh
@@ -0,0 +1,68 @@
+#!/bin/sh
+#
+# Copyright (c) 2017 Petr Vorel <pvorel@suse.cz>
+#
+# Script used for travis builds and for local test builds.
+
+set -e
+
+PREFIX="$HOME/ltp"
+CONFIGURE_OPTS="--with-open-posix-testsuite --with-realtime-testsuite --prefix=$PREFIX"
+
+build_32()
+{
+	echo "===== 32 bit in tree build into $PREFIX ====="
+	build CFLAGS="-m32" CXXFLAGS="-m32" LDFLAGS="-m32"
+}
+
+build_native()
+{
+	echo "===== native in tree build into $PREFIX ====="
+	build
+}
+
+build()
+{
+	echo "=== autotools ==="
+	make autotools
+
+	echo "=== configure ==="
+	if ! ./configure $CONFIGURE_OPTS $@; then
+		echo "configure failed, config log:"
+		cat config.log
+		exit 1
+	fi
+
+	echo "=== build ==="
+	make -j$(getconf _NPROCESSORS_ONLN)
+
+	echo "=== install ==="
+	make -j$(getconf _NPROCESSORS_ONLN) install
+}
+
+usage()
+{
+	cat << EOF
+Usage:
+$0 [ BUILD_TYPE ]
+$0 -h|--help|help
+
+Options:
+-h|--help|help  Print this help
+
+BUILD TYPES:
+32      32 bit build
+
+Default build is native build.
+EOF
+}
+
+case "$1" in
+	-h|--help|help) usage; exit 0;;
+	32) build="build_32";;
+	*) build="build_native";;
+esac
+
+$build
+
+# vim: set ft=sh ts=4 sts=4 sw=4 noet:
-- 
2.15.0


^ permalink raw reply related	[flat|nested] 17+ messages in thread

* [LTP] [RFC PATCH 2/4] travis: Add 32 cross-compile build + use build.sh
  2017-11-28 23:21 [LTP] [RFC PATCH 0/4] Travis enhancements Petr Vorel
  2017-11-28 23:21 ` [LTP] [RFC PATCH 1/4] travis: Add build script Petr Vorel
@ 2017-11-28 23:21 ` Petr Vorel
  2017-11-29 15:45   ` Cyril Hrubis
  2017-11-28 23:21 ` [LTP] [RFC PATCH 3/4] travis: Add gcc-7, clang-4.0, clang-5.0, remove clang-3.9 Petr Vorel
                   ` (2 subsequent siblings)
  4 siblings, 1 reply; 17+ messages in thread
From: Petr Vorel @ 2017-11-28 23:21 UTC (permalink / raw)
  To: ltp

Signed-off-by: Petr Vorel <pvorel@suse.cz>
---
 .travis.yml | 20 +++++++++++++++++++-
 1 file changed, 19 insertions(+), 1 deletion(-)

diff --git a/.travis.yml b/.travis.yml
index f2d51f131..347cae5e8 100644
--- a/.travis.yml
+++ b/.travis.yml
@@ -3,6 +3,7 @@ language: c
 matrix:
     include:
         - os: linux
+          env: BUILD="normal"
           compiler: gcc-4.6
           addons:
               apt:
@@ -10,6 +11,7 @@ matrix:
                   packages: ['gcc-4.6']
 
         - os: linux
+          env: BUILD="normal"
           compiler: gcc-4.7
           addons:
               apt:
@@ -17,6 +19,7 @@ matrix:
                   packages: ['gcc-4.7']
 
         - os: linux
+          env: BUILD="normal"
           compiler: gcc-4.8
           addons:
               apt:
@@ -24,6 +27,7 @@ matrix:
                   packages: ['gcc-4.8']
 
         - os: linux
+          env: BUILD="normal"
           compiler: gcc-4.9
           addons:
               apt:
@@ -31,6 +35,7 @@ matrix:
                   packages: ['gcc-4.9']
 
         - os: linux
+          env: BUILD="normal"
           compiler: gcc-5
           addons:
               apt:
@@ -38,6 +43,7 @@ matrix:
                   packages: ['gcc-5']
 
         - os: linux
+          env: BUILD="normal"
           compiler: gcc-6
           addons:
               apt:
@@ -45,6 +51,7 @@ matrix:
                   packages: ['gcc-6']
 
         - os: linux
+          env: BUILD="normal"
           compiler: clang-3.5
           addons:
               apt:
@@ -52,6 +59,7 @@ matrix:
                   packages: ['clang-3.5']
 
         - os: linux
+          env: BUILD="normal"
           compiler: clang-3.8
           addons:
               apt:
@@ -59,6 +67,7 @@ matrix:
                   packages: ['clang-3.8']
 
         - os: linux
+          env: BUILD="normal"
           compiler: clang-3.9
           addons:
               apt:
@@ -68,8 +77,17 @@ matrix:
                       - 'ubuntu-toolchain-r-test'
                   packages: ['clang-3.9']
 
+        - os: linux
+          env: BUILD="32"
+          compiler: gcc-6
+          addons:
+              apt:
+                  sources: ['ubuntu-toolchain-r-test']
+                  packages: ['gcc-6', 'gcc-6-multilib', 'linux-libc-dev:i386']
+
 notifications:
     email:
         secure: "b/xcA/K5OyQvPPnd0PRahTH5LJu8lgz8goGHvhXpHo+ZPsPgTDXNFo5cX9fSOkMuFKeoW8iGl0wOgK2+ptc8mbYDw277K4RFIHRHeV/KIoE1EzjQnEFiL8J0oHCAvDj12o0AXeriTyY9gICXKbR31Br6Zh5eKViDJe2OAGeHeDU="
 
-script: ./build.sh
+before_install:
+script: ./build.sh $BUILD
-- 
2.15.0


^ permalink raw reply related	[flat|nested] 17+ messages in thread

* [LTP] [RFC PATCH 3/4] travis: Add gcc-7, clang-4.0, clang-5.0, remove clang-3.9
  2017-11-28 23:21 [LTP] [RFC PATCH 0/4] Travis enhancements Petr Vorel
  2017-11-28 23:21 ` [LTP] [RFC PATCH 1/4] travis: Add build script Petr Vorel
  2017-11-28 23:21 ` [LTP] [RFC PATCH 2/4] travis: Add 32 cross-compile build + use build.sh Petr Vorel
@ 2017-11-28 23:21 ` Petr Vorel
  2017-11-28 23:21 ` [LTP] [RFC PATCH 4/4] travis: Install all dependencies (headers and libraries) Petr Vorel
  2017-11-29 15:44 ` [LTP] [RFC PATCH 0/4] Travis enhancements Cyril Hrubis
  4 siblings, 0 replies; 17+ messages in thread
From: Petr Vorel @ 2017-11-28 23:21 UTC (permalink / raw)
  To: ltp

Signed-off-by: Petr Vorel <pvorel@suse.cz>
---
 .travis.yml | 24 +++++++++++++++++++++---
 1 file changed, 21 insertions(+), 3 deletions(-)

diff --git a/.travis.yml b/.travis.yml
index 347cae5e8..2c186b71c 100644
--- a/.travis.yml
+++ b/.travis.yml
@@ -50,6 +50,14 @@ matrix:
                   sources: ['ubuntu-toolchain-r-test']
                   packages: ['gcc-6']
 
+        - os: linux
+          env: BUILD="normal"
+          compiler: gcc-7
+          addons:
+              apt:
+                  sources: ['ubuntu-toolchain-r-test']
+                  packages: ['gcc-7']
+
         - os: linux
           env: BUILD="normal"
           compiler: clang-3.5
@@ -68,14 +76,24 @@ matrix:
 
         - os: linux
           env: BUILD="normal"
-          compiler: clang-3.9
+          compiler: clang-4.0
           addons:
               apt:
                   sources:
-                      - sourceline: "deb http://apt.llvm.org/trusty/ llvm-toolchain-trusty-3.9 main"
+                      - sourceline: "deb http://apt.llvm.org/trusty/ llvm-toolchain-trusty-4.0 main"
                         key_url: "http://apt.llvm.org/llvm-snapshot.gpg.key"
                       - 'ubuntu-toolchain-r-test'
-                  packages: ['clang-3.9']
+                  packages: ['clang-4.0']
+
+        - os: linux
+          env: BUILD="normal"
+          compiler: clang-5.0
+          addons:
+              apt:
+                  sources:
+                      - sourceline: "deb http://apt.llvm.org/trusty/ llvm-toolchain-trusty-5.0 main"
+                        key_url: "http://apt.llvm.org/llvm-snapshot.gpg.key"
+                  packages: ['clang-5.0']
 
         - os: linux
           env: BUILD="32"
-- 
2.15.0


^ permalink raw reply related	[flat|nested] 17+ messages in thread

* [LTP] [RFC PATCH 4/4] travis: Install all dependencies (headers and libraries)
  2017-11-28 23:21 [LTP] [RFC PATCH 0/4] Travis enhancements Petr Vorel
                   ` (2 preceding siblings ...)
  2017-11-28 23:21 ` [LTP] [RFC PATCH 3/4] travis: Add gcc-7, clang-4.0, clang-5.0, remove clang-3.9 Petr Vorel
@ 2017-11-28 23:21 ` Petr Vorel
  2017-11-29 15:48   ` Cyril Hrubis
  2017-11-29 15:44 ` [LTP] [RFC PATCH 0/4] Travis enhancements Cyril Hrubis
  4 siblings, 1 reply; 17+ messages in thread
From: Petr Vorel @ 2017-11-28 23:21 UTC (permalink / raw)
  To: ltp

Before it was checked building the code without dependencies (i.e.
warning about missing libraries => testing autotools checks) which
we don't now, but it's better test building the code with installed
dependencies.

The only exception is selinux support on 32bit build as as
libsepol1-dev:i386 conflict with libsepol1-dev:amd64, thus
libselinux1-dev:i386 cannot be installed:
The following packages have unmet dependencies:
 libsepol1-dev : Conflicts: libsepol1-dev:i386 but 2.2-1ubuntu0.1 is to be installed
 libsepol1-dev:i386 : Conflicts: libsepol1-dev but 2.2-1ubuntu0.1 is to be installed
E: Unable to correct problems, you have held broken packages.

Signed-off-by: Petr Vorel <pvorel@suse.cz>
---
 .travis.yml | 30 ++++++++++++++++++++++++++++++
 1 file changed, 30 insertions(+)

diff --git a/.travis.yml b/.travis.yml
index 2c186b71c..7e4fdb2c3 100644
--- a/.travis.yml
+++ b/.travis.yml
@@ -108,4 +108,34 @@ notifications:
         secure: "b/xcA/K5OyQvPPnd0PRahTH5LJu8lgz8goGHvhXpHo+ZPsPgTDXNFo5cX9fSOkMuFKeoW8iGl0wOgK2+ptc8mbYDw277K4RFIHRHeV/KIoE1EzjQnEFiL8J0oHCAvDj12o0AXeriTyY9gICXKbR31Br6Zh5eKViDJe2OAGeHeDU="
 
 before_install:
+    - sudo apt install -f -y
+      libacl1-dev
+      libacl1:amd64
+      libacl1:i386
+      libaio-dev
+      libaio1:amd64
+      libaio1:i386
+      libcap-dev
+      libcap2:amd64
+      libcap2:i386
+      libc6
+      libc6-dev
+      libc6-dev-i386
+      libc6:i386
+      libkeyutils-dev
+      libkeyutils1:amd64
+      libkeyutils1:i386
+      libmm-dev
+      libnuma-dev
+      libnuma1:amd64
+      libnuma1:i386
+      libselinux1-dev:amd64
+      libssl-dev:amd64
+      libssl-dev:i386
+      libsepol1-dev:amd64
+      libssl-dev
+      libtirpc1:amd64
+      libtirpc1:i386
+      linux-libc-dev
+
 script: ./build.sh $BUILD
-- 
2.15.0


^ permalink raw reply related	[flat|nested] 17+ messages in thread

* [LTP] [RFC PATCH 1/4] travis: Add build script
  2017-11-28 23:21 ` [LTP] [RFC PATCH 1/4] travis: Add build script Petr Vorel
@ 2017-11-29  8:13   ` Li Wang
  2017-11-29  8:27     ` Petr Vorel
  0 siblings, 1 reply; 17+ messages in thread
From: Li Wang @ 2017-11-29  8:13 UTC (permalink / raw)
  To: ltp

On Wed, Nov 29, 2017 at 7:21 AM, Petr Vorel <pvorel@suse.cz> wrote:
> This script is to be used for travis build, but can be used for local
> builds as well.
>
> For usage run
> ./build.sh -h
>
> Signed-off-by: Petr Vorel <pvorel@suse.cz>
> ---
>  .travis.yml |  2 +-
>  build.sh    | 68 +++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
>  2 files changed, 69 insertions(+), 1 deletion(-)
>  create mode 100755 build.sh
>
> diff --git a/.travis.yml b/.travis.yml
> index d937f9dcf..f2d51f131 100644
> --- a/.travis.yml
> +++ b/.travis.yml
> @@ -72,4 +72,4 @@ notifications:
>      email:
>          secure: "b/xcA/K5OyQvPPnd0PRahTH5LJu8lgz8goGHvhXpHo+ZPsPgTDXNFo5cX9fSOkMuFKeoW8iGl0wOgK2+ptc8mbYDw277K4RFIHRHeV/KIoE1EzjQnEFiL8J0oHCAvDj12o0AXeriTyY9gICXKbR31Br6Zh5eKViDJe2OAGeHeDU="
>
> -script: make autotools && ./configure --prefix $HOME/ltp --with-open-posix-testsuite --with-realtime-testsuite && make -j$(getconf _NPROCESSORS_ONLN) && make -j$(getconf _NPROCESSORS_ONLN) install
> +script: ./build.sh
> diff --git a/build.sh b/build.sh
> new file mode 100755
> index 000000000..3387c120d
> --- /dev/null
> +++ b/build.sh
> @@ -0,0 +1,68 @@
> +#!/bin/sh
> +#
> +# Copyright (c) 2017 Petr Vorel <pvorel@suse.cz>
> +#
> +# Script used for travis builds and for local test builds.
> +
> +set -e
> +
> +PREFIX="$HOME/ltp"
> +CONFIGURE_OPTS="--with-open-posix-testsuite --with-realtime-testsuite --prefix=$PREFIX"
> +
> +build_32()
> +{
> +       echo "===== 32 bit in tree build into $PREFIX ====="
> +       build CFLAGS="-m32" CXXFLAGS="-m32" LDFLAGS="-m32"
> +}
> +
> +build_native()
> +{
> +       echo "===== native in tree build into $PREFIX ====="
> +       build
> +}
> +

How about adding a new function for ltp build out of tree?

build_out_of_tree()
{
       echo "===== build ltp out of tree ====="
...
}

> +build()
> +{
> +       echo "=== autotools ==="
> +       make autotools
> +
> +       echo "=== configure ==="
> +       if ! ./configure $CONFIGURE_OPTS $@; then
> +               echo "configure failed, config log:"
> +               cat config.log
> +               exit 1
> +       fi
> +
> +       echo "=== build ==="
> +       make -j$(getconf _NPROCESSORS_ONLN)
> +
> +       echo "=== install ==="
> +       make -j$(getconf _NPROCESSORS_ONLN) install
> +}
> +
> +usage()
> +{
> +       cat << EOF
> +Usage:
> +$0 [ BUILD_TYPE ]
> +$0 -h|--help|help
> +
> +Options:
> +-h|--help|help  Print this help
> +
> +BUILD TYPES:
> +32      32 bit build
> +
> +Default build is native build.
> +EOF
> +}
> +
> +case "$1" in
> +       -h|--help|help) usage; exit 0;;


            -o|--out) build="build_out_of_tree";;


> +       32) build="build_32";;
> +       *) build="build_native";;
> +esac
> +
> +$build
> +
> +# vim: set ft=sh ts=4 sts=4 sw=4 noet:
> --
> 2.15.0
>
>
> --
> Mailing list info: https://lists.linux.it/listinfo/ltp



-- 
Li Wang
liwang@redhat.com

^ permalink raw reply	[flat|nested] 17+ messages in thread

* [LTP] [RFC PATCH 1/4] travis: Add build script
  2017-11-29  8:13   ` Li Wang
@ 2017-11-29  8:27     ` Petr Vorel
  2017-11-29  8:41       ` Li Wang
  0 siblings, 1 reply; 17+ messages in thread
From: Petr Vorel @ 2017-11-29  8:27 UTC (permalink / raw)
  To: ltp

Hi Li,

> How about adding a new function for ltp build out of tree?

> build_out_of_tree()
> {
>        echo "===== build ltp out of tree ====="
> ...
> }
I'm planning it, but first we need to actually fix out of tree build.
Well, It's not broken, but it does not install all files (make install is
broken), so technically I could add it.

> > +case "$1" in
> > +       -h|--help|help) usage; exit 0;;


>             -o|--out) build="build_out_of_tree";;

> > +       32) build="build_32";;
> > +       *) build="build_native";;
I decided not to use getopts, as we'd have to use getopt from util-linux (getopts shell
builtin does not handle long opts.


Kind regards,
Petr


^ permalink raw reply	[flat|nested] 17+ messages in thread

* [LTP] [RFC PATCH 1/4] travis: Add build script
  2017-11-29  8:27     ` Petr Vorel
@ 2017-11-29  8:41       ` Li Wang
  2017-11-29 14:52         ` Petr Vorel
  0 siblings, 1 reply; 17+ messages in thread
From: Li Wang @ 2017-11-29  8:41 UTC (permalink / raw)
  To: ltp

On Wed, Nov 29, 2017 at 4:27 PM, Petr Vorel <pvorel@suse.cz> wrote:
> Hi Li,
>
>> How about adding a new function for ltp build out of tree?
>
>> build_out_of_tree()
>> {
>>        echo "===== build ltp out of tree ====="
>> ...
>> }
> I'm planning it, but first we need to actually fix out of tree build.
> Well, It's not broken, but it does not install all files (make install is
> broken), so technically I could add it.

Hmm, what kind of files will be missed?

I just tried your build.sh with this new function, but seems make
install successfully.

build_out_of_tree()
{
    echo "===== build ltp out of tree ====="
    TOP=$PWD
    TOP_SRCDIR=$TOP/../ltp
    SYSROOT=$TOP/../ltp-install
    TOP_BUILDDIR=$TOP/../ltp-build
    test -d "$TOP_BUILDDIR" || mkdir -p "$TOP_BUILDDIR"

    pushd $TOP_SRCDIR; make autotools; popd

    pushd "$TOP_BUILDDIR" && "$TOP_SRCDIR/configure"
    OUT_OF_BUILD_TREE_DIR=$TOP_BUILDDIR
    make \
        -C "$OUT_OF_BUILD_TREE_DIR" \
        -f "$TOP_SRCDIR/Makefile" \
        "top_srcdir=$TOP_SRCDIR" \
        "top_builddir=$OUT_OF_BUILD_TREE_DIR"

    make \
        -C "$OUT_OF_BUILD_TREE_DIR" \
        -f "$TOP_SRCDIR/Makefile" \
        "top_srcdir=$TOP_SRCDIR" \
        "top_builddir=$OUT_OF_BUILD_TREE_DIR" \
        "DESTDIR=$SYSROOT" \
        SKIP_IDCHECK=1 install
    popd
}


>
>> > +case "$1" in
>> > +       -h|--help|help) usage; exit 0;;
>
>
>>             -o|--out) build="build_out_of_tree";;
>
>> > +       32) build="build_32";;
>> > +       *) build="build_native";;
> I decided not to use getopts, as we'd have to use getopt from util-linux (getopts shell
> builtin does not handle long opts.
>
>
> Kind regards,
> Petr
>



-- 
Li Wang
liwang@redhat.com

^ permalink raw reply	[flat|nested] 17+ messages in thread

* [LTP] [RFC PATCH 1/4] travis: Add build script
  2017-11-29  8:41       ` Li Wang
@ 2017-11-29 14:52         ` Petr Vorel
  2017-11-29 15:42           ` Cyril Hrubis
  0 siblings, 1 reply; 17+ messages in thread
From: Petr Vorel @ 2017-11-29 14:52 UTC (permalink / raw)
  To: ltp

Hi Li,

> Hmm, what kind of files will be missed?
see bellow.

> I just tried your build.sh with this new function, but seems make
> install successfully.

> build_out_of_tree()
> {
>     echo "===== build ltp out of tree ====="
>     TOP=$PWD
>     TOP_SRCDIR=$TOP/../ltp
>     SYSROOT=$TOP/../ltp-install
>     TOP_BUILDDIR=$TOP/../ltp-build
>     test -d "$TOP_BUILDDIR" || mkdir -p "$TOP_BUILDDIR"

>     pushd $TOP_SRCDIR; make autotools; popd
pushd and popd are bashisms, which we don't want to use (we need to have POSIX compliant
shell scripts). I know, you're reading this INSTALL, which is IMHO too complicated (I'd
like to simplify this instructions, but that's another task).

>     pushd "$TOP_BUILDDIR" && "$TOP_SRCDIR/configure"
You're running configure without flags. For travis we run full build, e.g. with
--with-open-posix-testsuite --with-realtime-testsuite flags.
Try to run it at least with --with-open-posix-testsuite and you see these errors:
make -C "testcases" \
	-f "/home/pvorel/install/src/ltp.git/testcases/Makefile" install
make[1]: Entering directory '/home/pvorel/install/src/ltp-build/testcases'
make[2]: Entering directory '/home/pvorel/install/src/ltp-build/testcases/open_posix_testsuite'
/home/pvorel/install/src/ltp.git/testcases/open_posix_testsuite/Makefile:38: Makefile.linux: No such file or directory
make[2]: *** No rule to make target 'Makefile.linux'.  Stop.
make[2]: Leaving directory '/home/pvorel/install/src/ltp-build/testcases/open_posix_testsuite'
make[1]: *** [/home/pvorel/install/src/ltp.git/../ltp.git/include/mk/generic_trunk_target.inc:93: install] Error 2
make[1]: Leaving directory '/home/pvorel/install/src/ltp-build/testcases'
make: *** [/home/pvorel/install/src/ltp.git/../ltp.git/Makefile:142: testcases-install] Error 2
make: Leaving directory '/home/pvorel/install/src/ltp-build'

I've got some patches which fix error in installing, but some of the files don't get
installed (I don't remember which ones), that's why I haven't pushed the fixes yet.

>     OUT_OF_BUILD_TREE_DIR=$TOP_BUILDDIR
>     make \
>         -C "$OUT_OF_BUILD_TREE_DIR" \
>         -f "$TOP_SRCDIR/Makefile" \
>         "top_srcdir=$TOP_SRCDIR" \
>         "top_builddir=$OUT_OF_BUILD_TREE_DIR"

>     make \
>         -C "$OUT_OF_BUILD_TREE_DIR" \
>         -f "$TOP_SRCDIR/Makefile" \
>         "top_srcdir=$TOP_SRCDIR" \
>         "top_builddir=$OUT_OF_BUILD_TREE_DIR" \
>         "DESTDIR=$SYSROOT" \
>         SKIP_IDCHECK=1 install
>     popd
> }
Also -j flag for make is missing, which speeds build a lot (again, that's why I don't like
way in INSTALL).

Kind regards,
Petr

^ permalink raw reply	[flat|nested] 17+ messages in thread

* [LTP] [RFC PATCH 1/4] travis: Add build script
  2017-11-29 14:52         ` Petr Vorel
@ 2017-11-29 15:42           ` Cyril Hrubis
  2017-11-29 18:31             ` Petr Vorel
  0 siblings, 1 reply; 17+ messages in thread
From: Cyril Hrubis @ 2017-11-29 15:42 UTC (permalink / raw)
  To: ltp

Hi!
> You're running configure without flags. For travis we run full build, e.g. with
> --with-open-posix-testsuite --with-realtime-testsuite flags.

Can we, pretty please, have out-of-tree build without the realtime and
openposix testsuite for now. That way we will not break what is
supposedly working for now...

-- 
Cyril Hrubis
chrubis@suse.cz

^ permalink raw reply	[flat|nested] 17+ messages in thread

* [LTP] [RFC PATCH 0/4] Travis enhancements
  2017-11-28 23:21 [LTP] [RFC PATCH 0/4] Travis enhancements Petr Vorel
                   ` (3 preceding siblings ...)
  2017-11-28 23:21 ` [LTP] [RFC PATCH 4/4] travis: Install all dependencies (headers and libraries) Petr Vorel
@ 2017-11-29 15:44 ` Cyril Hrubis
  2017-11-29 19:07   ` Petr Vorel
  4 siblings, 1 reply; 17+ messages in thread
From: Cyril Hrubis @ 2017-11-29 15:44 UTC (permalink / raw)
  To: ltp

Hi!
> I've added several travis enhancements, most notably added 32bit cross-compile
> build and added more compilers (add gcc-7, clang-4.0, clang-5.0, remove clang-3.9).
> Now there is 12 builds instead of 9, speed increased from ~13 min [1] to ~26 min [2],
> which is maybe to much.  If we don't like it, we can decide which compilers to have.

Well we can cut a few compiler versions, perhaps just keep the latest
for each major number, that should catch most of the problems I guess.

-- 
Cyril Hrubis
chrubis@suse.cz

^ permalink raw reply	[flat|nested] 17+ messages in thread

* [LTP] [RFC PATCH 2/4] travis: Add 32 cross-compile build + use build.sh
  2017-11-28 23:21 ` [LTP] [RFC PATCH 2/4] travis: Add 32 cross-compile build + use build.sh Petr Vorel
@ 2017-11-29 15:45   ` Cyril Hrubis
  0 siblings, 0 replies; 17+ messages in thread
From: Cyril Hrubis @ 2017-11-29 15:45 UTC (permalink / raw)
  To: ltp

Hi!
> +        - os: linux
> +          env: BUILD="32"
> +          compiler: gcc-6
> +          addons:
> +              apt:
> +                  sources: ['ubuntu-toolchain-r-test']
> +                  packages: ['gcc-6', 'gcc-6-multilib', 'linux-libc-dev:i386']

Can we enable this for one old compiler as well? I would expect that
apart from generic 32bit compilation failures we will end up with
problems on older 32bit distros as well.

-- 
Cyril Hrubis
chrubis@suse.cz

^ permalink raw reply	[flat|nested] 17+ messages in thread

* [LTP] [RFC PATCH 4/4] travis: Install all dependencies (headers and libraries)
  2017-11-28 23:21 ` [LTP] [RFC PATCH 4/4] travis: Install all dependencies (headers and libraries) Petr Vorel
@ 2017-11-29 15:48   ` Cyril Hrubis
  2017-11-29 18:59     ` Petr Vorel
  0 siblings, 1 reply; 17+ messages in thread
From: Cyril Hrubis @ 2017-11-29 15:48 UTC (permalink / raw)
  To: ltp

Hi!
>  before_install:
> +    - sudo apt install -f -y
> +      libacl1-dev
> +      libacl1:amd64
> +      libacl1:i386
> +      libaio-dev
> +      libaio1:amd64
> +      libaio1:i386
> +      libcap-dev
> +      libcap2:amd64
> +      libcap2:i386
> +      libc6
> +      libc6-dev
> +      libc6-dev-i386
> +      libc6:i386
> +      libkeyutils-dev
> +      libkeyutils1:amd64
> +      libkeyutils1:i386
> +      libmm-dev
> +      libnuma-dev
> +      libnuma1:amd64
> +      libnuma1:i386
> +      libselinux1-dev:amd64
> +      libssl-dev:amd64
> +      libssl-dev:i386
> +      libsepol1-dev:amd64
> +      libssl-dev
> +      libtirpc1:amd64
> +      libtirpc1:i386
> +      linux-libc-dev

This runs before each compilation, right? Can we have one build
just with the basic C devel packages as well, since that have found
most bugs in stubs and fallback code so far.

-- 
Cyril Hrubis
chrubis@suse.cz

^ permalink raw reply	[flat|nested] 17+ messages in thread

* [LTP] [RFC PATCH 1/4] travis: Add build script
  2017-11-29 15:42           ` Cyril Hrubis
@ 2017-11-29 18:31             ` Petr Vorel
  2017-11-30  2:09               ` Li Wang
  0 siblings, 1 reply; 17+ messages in thread
From: Petr Vorel @ 2017-11-29 18:31 UTC (permalink / raw)
  To: ltp

Hi Li, Cyril,

> > You're running configure without flags. For travis we run full build, e.g. with
> > --with-open-posix-testsuite --with-realtime-testsuite flags.

> Can we, pretty please, have out-of-tree build without the realtime and
> openposix testsuite for now. That way we will not break what is
> supposedly working for now...
Sure

If you don't mind Li, I'll propose in v2 mine version of out-of-tree function and not yours due things I've noted. But thanks a lot for your input!


Kind regards,
Petr

^ permalink raw reply	[flat|nested] 17+ messages in thread

* [LTP] [RFC PATCH 4/4] travis: Install all dependencies (headers and libraries)
  2017-11-29 15:48   ` Cyril Hrubis
@ 2017-11-29 18:59     ` Petr Vorel
  0 siblings, 0 replies; 17+ messages in thread
From: Petr Vorel @ 2017-11-29 18:59 UTC (permalink / raw)
  To: ltp

Hi Cyril,

> >  before_install:
> > +    - sudo apt install -f -y
> > +      libacl1-dev
...

> This runs before each compilation, right?
Yes.

> Can we have one build just with the basic C devel packages as well, since that have
> found most bugs in stubs and fallback code so far.
Sure. I just need to add this code into separate shell script and add more variables
into matrix.

Kind regards,
Petr

^ permalink raw reply	[flat|nested] 17+ messages in thread

* [LTP] [RFC PATCH 0/4] Travis enhancements
  2017-11-29 15:44 ` [LTP] [RFC PATCH 0/4] Travis enhancements Cyril Hrubis
@ 2017-11-29 19:07   ` Petr Vorel
  0 siblings, 0 replies; 17+ messages in thread
From: Petr Vorel @ 2017-11-29 19:07 UTC (permalink / raw)
  To: ltp

Hi Cyril,

> > I've added several travis enhancements, most notably added 32bit cross-compile
> > build and added more compilers (add gcc-7, clang-4.0, clang-5.0, remove clang-3.9).
> > Now there is 12 builds instead of 9, speed increased from ~13 min [1] to ~26 min [2],
> > which is maybe to much.  If we don't like it, we can decide which compilers to have.

> Well we can cut a few compiler versions, perhaps just keep the latest
> for each major number, that should catch most of the problems I guess.
Thanks for an input! OK, I'll drop gcc 4.6, 4.7, 4.8 and clang 3.5, 3.8.
So we'll have gcc 4.9, 5.x, 6.x, 7.x and clang 3.9, 4.0 and 5.0
Unfortunately I haven't manage to add gcc 8.

Kind regards,
Petr

^ permalink raw reply	[flat|nested] 17+ messages in thread

* [LTP] [RFC PATCH 1/4] travis: Add build script
  2017-11-29 18:31             ` Petr Vorel
@ 2017-11-30  2:09               ` Li Wang
  0 siblings, 0 replies; 17+ messages in thread
From: Li Wang @ 2017-11-30  2:09 UTC (permalink / raw)
  To: ltp

On Thu, Nov 30, 2017 at 2:31 AM, Petr Vorel <pvorel@suse.cz> wrote:
> Hi Li, Cyril,
>
>> > You're running configure without flags. For travis we run full build, e.g. with
>> > --with-open-posix-testsuite --with-realtime-testsuite flags.
>
>> Can we, pretty please, have out-of-tree build without the realtime and
>> openposix testsuite for now. That way we will not break what is
>> supposedly working for now...
> Sure
>
> If you don't mind Li, I'll propose in v2 mine version of out-of-tree function and not yours due things I've noted. But thanks a lot for your input!

Of course not, feel free to use yours.That's just a simple draft I gave above.


-- 
Li Wang
liwang@redhat.com

^ permalink raw reply	[flat|nested] 17+ messages in thread

end of thread, other threads:[~2017-11-30  2:09 UTC | newest]

Thread overview: 17+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2017-11-28 23:21 [LTP] [RFC PATCH 0/4] Travis enhancements Petr Vorel
2017-11-28 23:21 ` [LTP] [RFC PATCH 1/4] travis: Add build script Petr Vorel
2017-11-29  8:13   ` Li Wang
2017-11-29  8:27     ` Petr Vorel
2017-11-29  8:41       ` Li Wang
2017-11-29 14:52         ` Petr Vorel
2017-11-29 15:42           ` Cyril Hrubis
2017-11-29 18:31             ` Petr Vorel
2017-11-30  2:09               ` Li Wang
2017-11-28 23:21 ` [LTP] [RFC PATCH 2/4] travis: Add 32 cross-compile build + use build.sh Petr Vorel
2017-11-29 15:45   ` Cyril Hrubis
2017-11-28 23:21 ` [LTP] [RFC PATCH 3/4] travis: Add gcc-7, clang-4.0, clang-5.0, remove clang-3.9 Petr Vorel
2017-11-28 23:21 ` [LTP] [RFC PATCH 4/4] travis: Install all dependencies (headers and libraries) Petr Vorel
2017-11-29 15:48   ` Cyril Hrubis
2017-11-29 18:59     ` Petr Vorel
2017-11-29 15:44 ` [LTP] [RFC PATCH 0/4] Travis enhancements Cyril Hrubis
2017-11-29 19:07   ` Petr Vorel

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.