Re: [Openvpn-devel] [PATCH v3] travis-ci: add arm64, s390x builds.

2020-03-24 Thread Илья Шипицин 
sometimes, arm64 build fails

https://travis-ci.org/github/OpenVPN/openvpn/jobs/666389482?utm_medium=notification&utm_source=github_status


if that will become often, we will mark it as "allowed failures"



thanks to everyone, we have cmocka tests back!

вт, 24 мар. 2020 г. в 14:04, Илья Шипицин :

> I guess nobody yet reported that issue.
>
> Maybe, I'll report.
>
> вт, 24 мар. 2020 г. в 14:03, Lev Stipakov :
>
>> Yes, I agree that with name is looks much better.
>>
>> I wonder why displaying arch requires you to be logged in.
>>
>
___
Openvpn-devel mailing list
Openvpn-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/openvpn-devel

Re: [Openvpn-devel] [PATCH v3] travis-ci: add arm64, s390x builds.

2020-03-24 Thread Илья Шипицин 
I guess nobody yet reported that issue.

Maybe, I'll report.

вт, 24 мар. 2020 г. в 14:03, Lev Stipakov :

> Yes, I agree that with name is looks much better.
>
> I wonder why displaying arch requires you to be logged in.
>
___
Openvpn-devel mailing list
Openvpn-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/openvpn-devel

Re: [Openvpn-devel] [PATCH v3] travis-ci: add arm64, s390x builds.

2020-03-24 Thread Lev Stipakov 
Yes, I agree that with name is looks much better.

I wonder why displaying arch requires you to be logged in.
___
Openvpn-devel mailing list
Openvpn-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/openvpn-devel

Re: [Openvpn-devel] [PATCH v3] travis-ci: add arm64, s390x builds.

2020-03-24 Thread Lev Stipakov 
Hi,

Tested with my openvpn github fork - all jobs passed and overall
travis dashboard looks nice thanks to added "name" field.

It would be even nicer to have "arch" as part of name, because now it looks like

 gcc | openssl-1.1.1d
 gcc | openssl-1.1.1d

But this is already step forward and improvements could be done in a
separate patch.

Acked-by: Lev Stipakov 


___
Openvpn-devel mailing list
Openvpn-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/openvpn-devel

Re: [Openvpn-devel] [PATCH v3] travis-ci: add arm64, s390x builds.

2020-03-22 Thread Илья Шипицин 
sorry, I sent it twice.

the last one is good, please ignore previous "v2"

I've managed to resolve "travis_wait" issue. it was non trivial output
redirection  issue.
the rest is just fine.


patch itself is important it re-enables cmocka tests in travis (they are
not running now, after cmocka git submodule was removed)

вс, 22 мар. 2020 г. в 17:35, :

> From: Ilya Shipitsin 
>
> as described on https://docs.travis-ci.com/user/multi-cpu-architectures
> travis-ci
> now supports amd64, ppcle, arm64, s390 architectures. Add arm64 and s390x.
>
> travis-ci images were upgraded to bionic.
>
> "sudo" is deprecated, let us remove it, also "matrix" is deprecated in
> favour of "jobs".
>
> LD_LIBRARY_PATH was replaced by using "rpath" in LDFLAGS, which is more
> elegant way of linking.
>
> also, dependencies were upgraded to the latest versions.
>
> travis_wait was added for long openssl builds.
>
> cmocka was added to linux and osx builds.
> ---
> v3 resolved travis_wait output redirection issue, now it works as
> expected. I had to specify "names" for jobs,
> without names travis puts secure variable as job name
>
> v2 rebased against proper commit
>
>
>  .travis.yml   | 87 +--
>  .travis/build-check.sh| 10 +
>  .travis/build-deps.sh | 10 +++--
>  .travis/run-build-deps.sh | 10 -
>  4 files changed, 62 insertions(+), 55 deletions(-)
>  delete mode 100755 .travis/run-build-deps.sh
>
> diff --git a/.travis.yml b/.travis.yml
> index 40296d87..925d09ea 100644
> --- a/.travis.yml
> +++ b/.travis.yml
> @@ -1,5 +1,4 @@
> -sudo: required
> -dist: xenial
> +dist: bionic
>
>  os: linux
>
> @@ -11,86 +10,111 @@ env:
>  - PREFIX="${HOME}/opt"
>  - TAP_WINDOWS_VERSION=9.23.3
>  - LZO_VERSION=2.10
> -- PKCS11_HELPER_VERSION=1.25.1
> -- MBEDTLS_VERSION=2.16.1
> +- PKCS11_HELPER_VERSION=1.26
> +- MBEDTLS_VERSION=2.16.4
>  - MBEDTLS_CFLAGS="-I${PREFIX}/include"
>  - MBEDTLS_LIBS="-L${PREFIX}/lib -lmbedtls -lmbedx509 -lmbedcrypto"
> -- OPENSSL_VERSION=1.0.2s
> +- OPENSSL_VERSION=1.0.2u
>  - OPENSSL_CFLAGS="-I${PREFIX}/include"
>  - OPENSSL_LIBS="-L${PREFIX}/lib -lssl -lcrypto"
>  # The next declaration is the encrypted COVERITY_SCAN_TOKEN, created
>  #   via the "travis encrypt" command using the project repo's public
> key
>  - secure:
> "l9mSnEW4LJqjxftH5i1NdDaYfGmQB1mPXnSB3DXnsjzkCWZ+yJLfBemfQ0tx/wS7chBYxqUaUIMT0hw4zJVp/LANFJo2vfh//ymTS6h0uApRY1ofg9Pp1BFcV1laG6/u8pwSZ2EBy/GhCd3DS436oE8sYBRaFM9FU62L/oeQBfJ7r4ID/0eB1b8bqlbD4paty9MHui2P8EZJwR+KAD84prtfpZOcrSMxPh9OUhJxzxUvvVoP4s4+lZ5Kgg1bBQ3yzKGDqe8VOgK2BWCEuezqhMMc8oeKmAe7CUkoz5gsGYH++k3I9XzP9Z4xeJKoQnC/82qi4xkJmlaOxdionej9bHIcjfRt7D8j1J0U+wOj4p8VrDy7yHaxuN2fi0K5MGa/CaXQSrkna8dePniCng+xQ2MY/zxuRX2gA6xPNLUyQLU9LqIug7wj4z84Hk9iWib4L20MoPjeEo+vAUNq8FtjOPxMuHNpv4iGGx6kgJm7RXl5vC5hxfK6MprrnYe2U5Mcd8jpzagKBaKHL3zV2FxX9k0jRO9Mccz7M2WnaV0MQ6zcngzTN4+s0kCjhfGKd2F2ANT2Gkhj3Me36eNHfuE0dBbvYCMh4b3Mgd7b/OuXwQWdJ8PjJ1WHXjSOw5sHw1suaV6cEO2Meyz5j1tOkyOi0M9QF+LFenQ9vLH4sBCww8U="
>
> -matrix:
> +jobs:
>include:
> -- env:
> +- name: cl
> +  env:
>- SSLLIB="openssl"
>- OPENSSL_VERSION="1.1.1d"
>- P7Z="c:\Program Files\7-Zip\7z.exe"
>- CC="cl"
>os: windows
>compiler: cl
> -- env: SSLLIB="openssl" RUN_COVERITY="1"
> +- name: Coverity scan
> +  env: SSLLIB="openssl" RUN_COVERITY="1"
>os: linux
>compiler: gcc
> -- env: SSLLIB="openssl" OPENSSL_VERSION="1.0.1u"
> +- name: gcc | openssl-1.0.1u
> +  env: SSLLIB="openssl" OPENSSL_VERSION="1.0.1u"
>os: linux
>compiler: gcc
> -- env: SSLLIB="openssl" OPENSSL_VERSION="1.1.1c"
> +- name: gcc | openssl-1.1.1d
> +  env: SSLLIB="openssl" OPENSSL_VERSION="1.1.1d"
>os: linux
> +  arch: amd64
>compiler: gcc
> -- env: SSLLIB="openssl" OPENSSL_VERSION="1.1.1c" LABEL="linux-ppc64le"
> -  os: linux-ppc64le
> +- name: gcc | openssl-1.1.1d
> +  env: SSLLIB="openssl" OPENSSL_VERSION="1.1.1d"
> +  os: linux
> +  arch: ppc64le
> +  compiler: gcc
> +- name: gcc | openssl-1.1.1d
> +  env: SSLLIB="openssl" OPENSSL_VERSION="1.1.1d"
> +  os: linux
> +  arch: arm64
> +  compiler: gcc
> +- name: gcc | openssl-1.1.1d
> +  env: SSLLIB="openssl" OPENSSL_VERSION="1.1.1d"
> +  os: linux
> +  arch: s390x
>compiler: gcc
> -- env: SSLLIB="openssl" EXTRA_CONFIG="--enable-iproute2"
> +- name: gcc | openssl-1.0.2u | iproute2
> +  env: SSLLIB="openssl" EXTRA_CONFIG="--enable-iproute2"
>os: linux
>compiler: gcc
> -- env: SSLLIB="openssl" CFLAGS="-fsanitize=address" CC=clang-9
> +- name: clang+asan | openssl-1.0.2u
> +  env: SSLLIB="openssl" CFLAGS="-fsanitize=address" CC=clang-9
>os: linux
>compiler: clang
> -- env: SSLLIB="openssl" OPENSSL_VERSION="1.1.1c" CC=clang-9
> +- name: clang | op

[Openvpn-devel] [PATCH v3] travis-ci: add arm64, s390x builds.

2020-03-22 Thread chipitsine 
From: Ilya Shipitsin 

as described on https://docs.travis-ci.com/user/multi-cpu-architectures 
travis-ci
now supports amd64, ppcle, arm64, s390 architectures. Add arm64 and s390x.

travis-ci images were upgraded to bionic.

"sudo" is deprecated, let us remove it, also "matrix" is deprecated in favour 
of "jobs".

LD_LIBRARY_PATH was replaced by using "rpath" in LDFLAGS, which is more elegant 
way of linking.

also, dependencies were upgraded to the latest versions.

travis_wait was added for long openssl builds.

cmocka was added to linux and osx builds.
---
v3 resolved travis_wait output redirection issue, now it works as expected. I 
had to specify "names" for jobs,
without names travis puts secure variable as job name

v2 rebased against proper commit


 .travis.yml   | 87 +--
 .travis/build-check.sh| 10 +
 .travis/build-deps.sh | 10 +++--
 .travis/run-build-deps.sh | 10 -
 4 files changed, 62 insertions(+), 55 deletions(-)
 delete mode 100755 .travis/run-build-deps.sh

diff --git a/.travis.yml b/.travis.yml
index 40296d87..925d09ea 100644
--- a/.travis.yml
+++ b/.travis.yml
@@ -1,5 +1,4 @@
-sudo: required
-dist: xenial
+dist: bionic
 
 os: linux
 
@@ -11,86 +10,111 @@ env:
 - PREFIX="${HOME}/opt"
 - TAP_WINDOWS_VERSION=9.23.3
 - LZO_VERSION=2.10
-- PKCS11_HELPER_VERSION=1.25.1
-- MBEDTLS_VERSION=2.16.1
+- PKCS11_HELPER_VERSION=1.26
+- MBEDTLS_VERSION=2.16.4
 - MBEDTLS_CFLAGS="-I${PREFIX}/include"
 - MBEDTLS_LIBS="-L${PREFIX}/lib -lmbedtls -lmbedx509 -lmbedcrypto"
-- OPENSSL_VERSION=1.0.2s
+- OPENSSL_VERSION=1.0.2u
 - OPENSSL_CFLAGS="-I${PREFIX}/include"
 - OPENSSL_LIBS="-L${PREFIX}/lib -lssl -lcrypto"
 # The next declaration is the encrypted COVERITY_SCAN_TOKEN, created
 #   via the "travis encrypt" command using the project repo's public key
 - secure: 
"l9mSnEW4LJqjxftH5i1NdDaYfGmQB1mPXnSB3DXnsjzkCWZ+yJLfBemfQ0tx/wS7chBYxqUaUIMT0hw4zJVp/LANFJo2vfh//ymTS6h0uApRY1ofg9Pp1BFcV1laG6/u8pwSZ2EBy/GhCd3DS436oE8sYBRaFM9FU62L/oeQBfJ7r4ID/0eB1b8bqlbD4paty9MHui2P8EZJwR+KAD84prtfpZOcrSMxPh9OUhJxzxUvvVoP4s4+lZ5Kgg1bBQ3yzKGDqe8VOgK2BWCEuezqhMMc8oeKmAe7CUkoz5gsGYH++k3I9XzP9Z4xeJKoQnC/82qi4xkJmlaOxdionej9bHIcjfRt7D8j1J0U+wOj4p8VrDy7yHaxuN2fi0K5MGa/CaXQSrkna8dePniCng+xQ2MY/zxuRX2gA6xPNLUyQLU9LqIug7wj4z84Hk9iWib4L20MoPjeEo+vAUNq8FtjOPxMuHNpv4iGGx6kgJm7RXl5vC5hxfK6MprrnYe2U5Mcd8jpzagKBaKHL3zV2FxX9k0jRO9Mccz7M2WnaV0MQ6zcngzTN4+s0kCjhfGKd2F2ANT2Gkhj3Me36eNHfuE0dBbvYCMh4b3Mgd7b/OuXwQWdJ8PjJ1WHXjSOw5sHw1suaV6cEO2Meyz5j1tOkyOi0M9QF+LFenQ9vLH4sBCww8U="
 
-matrix:
+jobs:
   include:
-- env:
+- name: cl
+  env:
   - SSLLIB="openssl"
   - OPENSSL_VERSION="1.1.1d"
   - P7Z="c:\Program Files\7-Zip\7z.exe"
   - CC="cl"
   os: windows
   compiler: cl
-- env: SSLLIB="openssl" RUN_COVERITY="1"
+- name: Coverity scan
+  env: SSLLIB="openssl" RUN_COVERITY="1"
   os: linux
   compiler: gcc
-- env: SSLLIB="openssl" OPENSSL_VERSION="1.0.1u"
+- name: gcc | openssl-1.0.1u
+  env: SSLLIB="openssl" OPENSSL_VERSION="1.0.1u"
   os: linux
   compiler: gcc
-- env: SSLLIB="openssl" OPENSSL_VERSION="1.1.1c"
+- name: gcc | openssl-1.1.1d
+  env: SSLLIB="openssl" OPENSSL_VERSION="1.1.1d"
   os: linux
+  arch: amd64
   compiler: gcc
-- env: SSLLIB="openssl" OPENSSL_VERSION="1.1.1c" LABEL="linux-ppc64le"
-  os: linux-ppc64le
+- name: gcc | openssl-1.1.1d
+  env: SSLLIB="openssl" OPENSSL_VERSION="1.1.1d"
+  os: linux
+  arch: ppc64le
+  compiler: gcc
+- name: gcc | openssl-1.1.1d
+  env: SSLLIB="openssl" OPENSSL_VERSION="1.1.1d"
+  os: linux
+  arch: arm64
+  compiler: gcc
+- name: gcc | openssl-1.1.1d
+  env: SSLLIB="openssl" OPENSSL_VERSION="1.1.1d"
+  os: linux
+  arch: s390x
   compiler: gcc
-- env: SSLLIB="openssl" EXTRA_CONFIG="--enable-iproute2"
+- name: gcc | openssl-1.0.2u | iproute2
+  env: SSLLIB="openssl" EXTRA_CONFIG="--enable-iproute2"
   os: linux
   compiler: gcc
-- env: SSLLIB="openssl" CFLAGS="-fsanitize=address" CC=clang-9
+- name: clang+asan | openssl-1.0.2u
+  env: SSLLIB="openssl" CFLAGS="-fsanitize=address" CC=clang-9
   os: linux
   compiler: clang
-- env: SSLLIB="openssl" OPENSSL_VERSION="1.1.1c" CC=clang-9
+- name: clang | openssl-1.1.1d
+  env: SSLLIB="openssl" OPENSSL_VERSION="1.1.1d" CC=clang-9
   os: linux
   compiler: clang
-- env: SSLLIB="mbedtls"
+- name: gcc | mbedtls
+  env: SSLLIB="mbedtls"
   os: linux
   compiler: gcc
-- env: SSLLIB="mbedtls" CFLAGS="-fsanitize=address" CC=clang-9
+- name: clang+asan | mbedtls
+  env: SSLLIB="mbedtls" CFLAGS="-fsanitize=address" CC=clang-9
   os: linux
   compiler: clang
-- env: SSLLIB="openssl"
+- name: clang | openssl-1.0.2u
+  env: SSLLIB="openssl"
   os: osx
   compiler: