[Openvpn-devel] [PATCH v2] travis-ci: cleanup, refactor, upgrade ssl libraries

2018-05-27 Thread Ilya Shipitsin
Both "compiler" and "exclude" are redundant, so remove them. Add openssl-1.0.1u to build matrix. Enable explicit apt update (it was disabled by default in travis-ci). Signed-off-by: Ilya Shipitsin <chipits...@gmail.com> -- v2: Add openssl-1.0.1u to build matrix (than

[Openvpn-devel] [PATCH] travis-ci: cleanup and upgrade ssl libraries

2018-04-09 Thread Ilya Shipitsin
both "compiler" and "exclude" are redundant, so remove them Signed-off-by: Ilya Shipitsin <chipits...@gmail.com> --- .travis.yml | 13 - 1 file changed, 4 insertions(+), 9 deletions(-) diff --git a/.travis.yml b/.travis.yml index 51645924..1cbd96bc 1006

[Openvpn-devel] [PATCH] travis-ci: add pkcs11-helper patch identical to openvpn-build

2018-02-20 Thread Ilya Shipitsin
currently patch of implementing RFC7512 URI Scheme is added during building windows installer. The same patch was added to travis-ci cross builds Signed-off-by: Ilya Shipitsin <chipits...@gmail.com> --- .travis/build-deps.sh | 17 +- .travis/pkcs11-helper-001-RFC7512

[Openvpn-devel] [PATCH] travis-ci: modify openssl build script to support openssl-1.1.0

2018-01-14 Thread Ilya Shipitsin
no-multilib is only supported on openssl-1.0.X, do not use it if OPENSSL_VERSION is 1.1.0 Signed-off-by: Ilya Shipitsin <chipits...@gmail.com> --- .travis/build-deps.sh | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/.travis/build-deps.sh b/.travis/build-deps.sh

[Openvpn-devel] [PATCH v3] travis-ci: add brew cache, remove ccache

2018-01-04 Thread Ilya Shipitsin
1-2 minutes speedup osx builds by using brew cache. Also, ccache was removed for a while (builds fail after travis-ci upgraded clang to version 5.0.0) --- v2: this is a "v2" of previously issued "enable ccache for osx and mingw builds" patch. I decided not to enable ccache for mingw builds as it

[Openvpn-devel] [PATCH] travis-ci: speedup osx build by enabling brew cache

2018-01-04 Thread Ilya Shipitsin
1-2 minutes speedup by using brew cache, also ccache is no more disabled for osx build (even it does not speedup significantly, it simplifies the overall script) --- this is a "v2" of previously issued "enable ccache for osx and mingw builds" patch. I decided not to enable ccache for mingw builds

[Openvpn-devel] [PATCH] travis-ci: enable ccache for osx and mingw builds

2017-11-20 Thread Ilya Shipitsin
-- ccache was now tested by me and works for osx and mingw builds as well --- .travis.yml | 2 +- .travis/build-deps.sh | 14 ++ 2 files changed, 7 insertions(+), 9 deletions(-) diff --git a/.travis.yml b/.travis.yml index 366e6599..8efb1cbd 100644 --- a/.travis.yml +++

[Openvpn-devel] [PATCH 0/1] travis-ci: libressl support for osx

2017-08-13 Thread Ilya Shipitsin
Tunnelblick ships openvpn binary built with libressl (see https://github.com/Tunnelblick/Tunnelblick/issues/317 ) Ilya Shipitsin (1): travis-ci: add libressl build for osx .travis.yml| 5 + .travis/build-check.sh | 4 .travis/build-deps.sh | 31

[Openvpn-devel] [PATCH] v2 travis-ci: update pkcs11-helper to 1.22

2017-08-09 Thread Ilya Shipitsin
use pkcs11-helper from https://github.com/OpenSC/pkcs11-helper/ to match build process used in windows installer build Signed-off-by: Ilya Shipitsin <chipits...@gmail.com> --- v2: break lines up into 80-char, thanks Steffan Karger .travis.yml | 2 +- .travis/build-deps.

[Openvpn-devel] [PATCH 1/3] travis-ci: update pkcs11-helper to 1.22

2017-08-07 Thread Ilya Shipitsin
use pkcs11-helper from https://github.com/OpenSC/pkcs11-helper/ to match build process used in windows installer build --- .travis.yml | 2 +- .travis/build-deps.sh | 4 ++-- 2 files changed, 3 insertions(+), 3 deletions(-) diff --git a/.travis.yml b/.travis.yml index

[Openvpn-devel] [PATCH 2/3] travis-ci: update openssl to 1.0.2l, update mbedtls to 2.5.1

2017-08-07 Thread Ilya Shipitsin
--- .travis.yml | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/.travis.yml b/.travis.yml index fc98a4d0..0b531529 100644 --- a/.travis.yml +++ b/.travis.yml @@ -15,10 +15,10 @@ env: - TAP_WINDOWS_VERSION=9.21.2 - LZO_VERSION=2.10 -

[Openvpn-devel] [PATCH 3/3] add a test to "openvpn --help"

2017-08-07 Thread Ilya Shipitsin
inspired by https://www.mail-archive.com/openvpn-devel@lists.sourceforge.net/msg13808.html --- tests/Makefile.am | 2 +- tests/t_usage.sh | 29 + 2 files changed, 30 insertions(+), 1 deletion(-) create mode 100755 tests/t_usage.sh diff --git a/tests/Makefile.am

[Openvpn-devel] [PATCH] win32 build: check for ENETUNREACH

2017-07-09 Thread Ilya Shipitsin
Currently, we do not check for mingw-gcc version. For example, Debian 7 is shipped with 4.6.3, which does not know about ENETUNREACH --- this is for 2.4 and HEAD branches configure.ac | 14 ++ 1 file changed, 14 insertions(+) diff --git a/configure.ac b/configure.ac index

[Openvpn-devel] [PATCH] travis-ci: added gcc and clang openssl-1.1.0 builds

2017-06-19 Thread Ilya Shipitsin
openssl build script was modified according to official openssl manual: https://wiki.openssl.org/index.php/Compilation_and_Installation --- .travis.yml | 6 ++ .travis/build-deps.sh | 6 +++--- 2 files changed, 9 insertions(+), 3 deletions(-) diff --git a/.travis.yml b/.travis.yml

[Openvpn-devel] [PATCH] v4, travis-ci: add 2 mingw "build only" configurations

2017-05-05 Thread Ilya Shipitsin
Inspired by https://www.mail-archive.com/openvpn-devel@lists.sourceforge.net/msg13032.html build options are taken from regular windows installer builds Signed-off-by: Ilya Shipitsin <chipits...@gmail.com> --- v2: moved download/build dependencies into functions, changed cross build det

[Openvpn-devel] [PATCH] v3: resolve format string warnings

2017-03-29 Thread Ilya Shipitsin
[src/openvpn/options.c:1163]: (warning) %x in format string (no. 1) requires 'unsigned int *' but the argument type is 'signed int *'. [src/openvpn/route.c:3187]: (warning) %d in format string (no. 5) requires 'int *' but the argument type is 'unsigned int *'. Signed-off-by: Ilya Shipitsin

[Openvpn-devel] [PATCH] travis-ci: add 2 mingw "build only configurations"

2017-03-26 Thread Ilya Shipitsin
Inspired by https://www.mail-archive.com/openvpn-devel@lists.sourceforge.net/msg13032.html build options are taken from regular windows installer builds Signed-off-by: Ilya Shipitsin <chipits...@gmail.com> --- .travis.yml | 30 +--- .travis/build-deps.s

[Openvpn-devel] [PATCH] v2: resolve format string warnings

2017-03-25 Thread Ilya Shipitsin
options.c, thanks to Gert Doering Signed-off-by: Ilya Shipitsin <chipits...@gmail.com> --- src/openvpn/options.c | 4 ++-- src/openvpn/route.c | 2 +- 2 files changed, 3 insertions(+), 3 deletions(-) diff --git a/src/openvpn/options.c b/src/openvpn/options.c index dcb6ecf..6aa72dc 100644 -

[Openvpn-devel] [PATCH] resolve format string warnings

2017-03-25 Thread Ilya Shipitsin
[src/openvpn/options.c:1163]: (warning) %x in format string (no. 1) requires 'unsigned int *' but the argument type is 'signed int *'. [src/openvpn/route.c:3187]: (warning) %d in format string (no. 5) requires 'int *' but the argument type is 'unsigned int *'. Signed-off-by: Ilya Shipitsin

[Openvpn-devel] [PATCH] travis-ci: remove unused files

2017-03-05 Thread Ilya Shipitsin
Those files were commited by mistake. I implemented building dependencies in 4 separate scripts, later Steffan Karger combined all 4 scripts into "build-deps.sh". Signed-off-by: Ilya Shipitsin <chipits...@gmail.com> --- .travis/build-mbedtls-linux.sh | 9 - .travis/buil

[Openvpn-devel] [PATCH] travis-ci: add "make distcheck" to test scenario, V2

2017-02-25 Thread Ilya Shipitsin
in rare cases openvpn is built from tarball, it happens during "installer build" process. "make distcheck" helps to prevent problems during such builds. V2: limit "make distcheck" to one build configuration Signed-off-by: Ilya Shipitsin <chipits...@gmail.com&

[Openvpn-devel] [PATCH] travis-ci: add "make distcheck" to test scenario

2017-02-23 Thread Ilya Shipitsin
in rare cases openvpn is built from tarball, it happens during "installer build" process. "make distcheck" helps to prevent problems during such builds. Signed-off-by: Ilya Shipitsin <chipits...@gmail.com> --- .travis.yml | 1 + 1 file changed, 1 insertion(+)

[Openvpn-devel] [PATCH] v2, add PR template in order to simplify new developers cooperate properly if they open PR. discussed here: https://sourceforge.net/p/openvpn/mailman/message/35601310/

2017-02-05 Thread Ilya Shipitsin
v2: @mattock has changed links and provided an example of "git send-email" --- .github/PULL_REQUEST_TEMPLATE.md | 15 +++ 1 file changed, 15 insertions(+) create mode 100644 .github/PULL_REQUEST_TEMPLATE.md diff --git a/.github/PULL_REQUEST_TEMPLATE.md

[Openvpn-devel] [PATCH] Resolve several travis-ci issues

2017-01-28 Thread Ilya Shipitsin
MBEDTLS_VERSION, OPENSSL_VERSION were defined twice - in both .travis.yml and .travis/build-deps.sh files, the last one defined OPENSSL_VERSION via nonexistent OPENSSL_VERION variable, which lead us to use openssl-1.0.1 instead of openssl-1.0.2, I removed variable definition from build-deps.sh

[Openvpn-devel] [PATCH] Resolve several travis-ci issues

2017-01-28 Thread Ilya Shipitsin
MBEDTLS_VERSION, OPENSSL_VERSION were defined twice - in both .travis.yml and .travis/build-deps.sh files, the last one defined OPENSSL_VERSION via nonexistent OPENSSL_VERION variable, which lead us to use openssl-1.0.1 instead of openssl-1.0.2, I removed variable definition from build-deps.sh

[Openvpn-devel] [PATCH] Resolve several travis-ci issues

2017-01-28 Thread Ilya Shipitsin
MBEDTLS_VERSION, OPENSSL_VERSION were defined twice - in both .travis.yml and .travis/build-deps.sh files, the last one defined OPENSSL_VERSION via nonexistent OPENSSL_VERION variable, which lead us to use openssl-1.0.1 instead of openssl-1.0.2, I removed variable definition from build-deps.sh

[Openvpn-devel] [PATCH] Resolve several travis-ci issues

2017-01-27 Thread Ilya Shipitsin
MBEDTLS_VERSION, OPENSSL_VERSION were defined twice - in both .travis.yml and .travis/build-deps.sh files, the last one defined OPENSSL_VERSION via nonexistent OPENSSL_VERION variable, which lead us to use openssl-1.0.1 instead of openssl-1.0.2, I removed variable definition from build-deps.sh

[Openvpn-devel] [PATCH] cppcheck finding: add "ASSERT( maxoutput > 0 || separator != NULL )" to prevent possible null pointer derefence

2016-09-18 Thread Ilya Shipitsin
--- src/openvpn/buffer.c | 6 -- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/src/openvpn/buffer.c b/src/openvpn/buffer.c index 52c6ab9..57bded9 100644 --- a/src/openvpn/buffer.c +++ b/src/openvpn/buffer.c @@ -438,10 +438,12 @@ format_hex_ex (const uint8_t *data, int size,

[Openvpn-devel] [PATCH] enable "--disable-crypto" build configuration

2016-09-17 Thread Ilya Shipitsin
--- .travis.yml | 4 1 file changed, 4 deletions(-) diff --git a/.travis.yml b/.travis.yml index 452c48e..369db97 100644 --- a/.travis.yml +++ b/.travis.yml @@ -51,10 +51,6 @@ matrix: os: osx osx_image: xcode7.3 compiler: clang - allow_failures: -- env:

[Openvpn-devel] [PATCH] travis-ci refactor by Steffan Karger

2016-09-15 Thread Ilya Shipitsin
From: Steffan Karger Signed-off-by: Steffan Karger --- .travis.yml | 63 -- .travis/build-deps.sh | 85 +++ 2 files changed, 119 insertions(+), 29 deletions(-)

[Openvpn-devel] [PATCH] initial travis-ci support

2016-09-15 Thread Ilya Shipitsin
--- .travis.yml| 85 ++ .travis/build-mbedtls-linux.sh | 10 + .travis/build-mbedtls-osx.sh | 9 + .travis/build-openssl-linux.sh | 12 ++ .travis/build-openssl-osx.sh | 11 ++ 5 files changed, 127 insertions(+)

[Openvpn-devel] [PATCH] initial travis-ci support

2016-09-15 Thread Ilya Shipitsin
--- .travis.yml| 85 ++ .travis/build-mbedtls-linux.sh | 10 + .travis/build-mbedtls-osx.sh | 9 + .travis/build-openssl-linux.sh | 12 ++ .travis/build-openssl-osx.sh | 11 ++ 5 files changed, 127 insertions(+)