* [LTP] [PATCH v1] travis: add ppc64le and s390x builds (issue: #615)
@ 2019-11-18 10:57 Clemens Famulla-Conrad
2019-11-18 14:54 ` Petr Vorel
0 siblings, 1 reply; 4+ messages in thread
From: Clemens Famulla-Conrad @ 2019-11-18 10:57 UTC (permalink / raw)
To: ltp
Travis started to support ppc64le and s390x
https://blog.travis-ci.com/2019-11-12-multi-cpu-architecture-ibm-power-ibm-z
This patch enable ppc64le and s390x build on debian:stable.
Remove `-i` switch from docker command as it isn't needed and caused
trouble running docker 'The input device is not a tty'.
Signed-off-by: Clemens Famulla-Conrad <cfamullaconrad@suse.de>
---
.travis.yml | 12 +++++++++++-
1 file changed, 11 insertions(+), 1 deletion(-)
diff --git a/.travis.yml b/.travis.yml
index b16a3872a..be1504b92 100644
--- a/.travis.yml
+++ b/.travis.yml
@@ -65,6 +65,16 @@ matrix:
env: DISTRO=centos:latest
compiler: gcc
+ - os: linux
+ arch: ppc64le
+ compiler: gcc
+ env: DISTRO=debian:stable
+
+ - os: linux
+ arch: s390x
+ compiler: gcc
+ env: DISTRO=debian:stable
+
before_install:
- DIR="/usr/src/ltp"
- printf "FROM $DISTRO\nRUN mkdir -p $DIR\nWORKDIR $DIR\nCOPY . $DIR\n" > Dockerfile
@@ -76,4 +86,4 @@ script:
- INSTALL="${INSTALL%%/*}"
- if [ ! "$TREE" ]; then TREE="in"; fi
- case $VARIANT in cross-compile*) BUILD="cross";; i386) BUILD="32";; *) BUILD="native";; esac
- - docker run -it ltp /bin/sh -c "cd travis && ./$INSTALL.sh && if [ \"$VARIANT\" ]; then ./$INSTALL.$VARIANT.sh; fi && ../build.sh -o $TREE -t $BUILD -c $CC"
+ - docker run -t ltp /bin/sh -c "cd travis && ./$INSTALL.sh && if [ \"$VARIANT\" ]; then ./$INSTALL.$VARIANT.sh; fi && ../build.sh -o $TREE -t $BUILD -c $CC"
--
2.16.4
^ permalink raw reply related [flat|nested] 4+ messages in thread
* [LTP] [PATCH v1] travis: add ppc64le and s390x builds (issue: #615)
2019-11-18 10:57 [LTP] [PATCH v1] travis: add ppc64le and s390x builds (issue: #615) Clemens Famulla-Conrad
@ 2019-11-18 14:54 ` Petr Vorel
2019-11-18 18:07 ` Clemens Famulla-Conrad
0 siblings, 1 reply; 4+ messages in thread
From: Petr Vorel @ 2019-11-18 14:54 UTC (permalink / raw)
To: ltp
Hi Clemens,
thanks a lot!
> Travis started to support ppc64le and s390x
> https://blog.travis-ci.com/2019-11-12-multi-cpu-architecture-ibm-power-ibm-z
> This patch enable ppc64le and s390x build on debian:stable.
> Remove `-i` switch from docker command as it isn't needed and caused
> trouble running docker 'The input device is not a tty'.
Interesting. Indeed, fails with -i. Don't remember, why I needed it.
LGTM, I'd just move these 2 jobs higher (above fedora), as these builds are not
usual so have higher priority to find a bug.
Change ppc64le to to debian:testing (we already have stable build for ppc64le
cross compilation).
Something like:
https://travis-ci.org/pevik/ltp/builds/613539454
https://github.com/pevik/ltp/commit/2b69957b5838eb9f27e65ba172a71c24f15af6bd
(No need to resend a patch, both can do during merge.)
Kind regards,
Petr
^ permalink raw reply [flat|nested] 4+ messages in thread
* [LTP] [PATCH v1] travis: add ppc64le and s390x builds (issue: #615)
2019-11-18 14:54 ` Petr Vorel
@ 2019-11-18 18:07 ` Clemens Famulla-Conrad
2019-11-18 20:22 ` Petr Vorel
0 siblings, 1 reply; 4+ messages in thread
From: Clemens Famulla-Conrad @ 2019-11-18 18:07 UTC (permalink / raw)
To: ltp
Hi Petr,
On Mon, 2019-11-18 at 15:54 +0100, Petr Vorel wrote:
<snip>
>
> LGTM, I'd just move these 2 jobs higher (above fedora), as these
> builds are not
> usual so have higher priority to find a bug.
> Change ppc64le to to debian:testing (we already have stable build for
> ppc64le
> cross compilation).
Makes sense
>
> Something like:
> https://travis-ci.org/pevik/ltp/builds/613539454
> https://github.com/pevik/ltp/commit/2b69957b5838eb9f27e65ba172a71c24f
> 15af6bd
> (No need to resend a patch, both can do during merge.)
Thanks a lot
looks good to me as well.
^ permalink raw reply [flat|nested] 4+ messages in thread
* [LTP] [PATCH v1] travis: add ppc64le and s390x builds (issue: #615)
2019-11-18 18:07 ` Clemens Famulla-Conrad
@ 2019-11-18 20:22 ` Petr Vorel
0 siblings, 0 replies; 4+ messages in thread
From: Petr Vorel @ 2019-11-18 20:22 UTC (permalink / raw)
To: ltp
Hi Clements,
> > LGTM, I'd just move these 2 jobs higher (above fedora), as these
> > builds are not
> > usual so have higher priority to find a bug.
> > Change ppc64le to to debian:testing (we already have stable build for
> > ppc64le
> > cross compilation).
> Makes sense
> > Something like:
> > https://travis-ci.org/pevik/ltp/builds/613539454
> > https://github.com/pevik/ltp/commit/2b69957b5838eb9f27e65ba172a71c24f
> > 15af6bd
> > (No need to resend a patch, both can do during merge.)
> Thanks a lot
> looks good to me as well.
Thanks for your ack, merged with slightly changed commit message.
Kind regards,
Petr
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2019-11-18 20:22 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2019-11-18 10:57 [LTP] [PATCH v1] travis: add ppc64le and s390x builds (issue: #615) Clemens Famulla-Conrad
2019-11-18 14:54 ` Petr Vorel
2019-11-18 18:07 ` Clemens Famulla-Conrad
2019-11-18 20:22 ` 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.