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

2017-01-30 Thread Antonio Quartulli
On Mon, Jan 30, 2017 at 02:28:22PM +0200, Samuli Seppänen wrote: > On 29/01/2017 21:43, David Sommerseth wrote: > > On 29/01/17 19:13, Gert Doering wrote: > >> > >>> Git can do this for you by specifying "-v X" (i.e. -v 2" to the git > >>> format-patch > >>> command. > >> > >> I've learned somethi

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

2017-01-30 Thread Samuli Seppänen
On 29/01/2017 21:43, David Sommerseth wrote: > On 29/01/17 19:13, Gert Doering wrote: >> >>> Git can do this for you by specifying "-v X" (i.e. -v 2" to the git >>> format-patch >>> command. >> >> I've learned something new today :-) (but I shouldn't be surprised: git >> is known to do everything

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

2017-01-29 Thread David Sommerseth
On 29/01/17 19:13, Gert Doering wrote: > >> Git can do this for you by specifying "-v X" (i.e. -v 2" to the git >> format-patch >> command. > > I've learned something new today :-) (but I shouldn't be surprised: git > is known to do everything to ease lazy committers' work :-) ) Weee! I realize

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

2017-01-29 Thread Gert Doering
Hi, On Sun, Jan 29, 2017 at 11:43:04PM +0800, Antonio Quartulli wrote: > Not sure what the maintainers think about this, but I stumbled on the wrong > version a couple of times today before realizing that the newer was another > one. Either "[PATCH v2]" or at least mention "v2: " in the commit m

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

2017-01-29 Thread Selva Nair
On Sun, Jan 29, 2017 at 1:58 AM, Ilya Shipitsin wrote: > 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 >

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

2017-01-29 Thread Antonio Quartulli
Hi Ilya, my 2-meta-cents here: when sending multiple versions of the same patch to the mailing list, I'd recommend to change the subject to "[PATCHvX]" where X is the patch version. This way it is easier to identify which patch comes before and which after. Not sure what the maintainers think ab

[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 "c

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

2017-01-28 Thread Илья Шипицин
2017-01-29 1:29 GMT+05:00 Selva Nair : > Hi, > > On Sat, Jan 28, 2017 at 3:06 AM, Ilya Shipitsin > wrote: > >> 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 >>

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

2017-01-28 Thread Selva Nair
Hi, On Sat, Jan 28, 2017 at 3:06 AM, Ilya Shipitsin wrote: > 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

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

2017-01-28 Thread Илья Шипицин
forgot to mention, this patch might be applied to both master and release/2.4 branches 2017-01-28 13:07 GMT+05:00 Илья Шипицин : > this one is final v2 patch > > 2017-01-28 13:06 GMT+05:00 Ilya Shipitsin : > >> MBEDTLS_VERSION, OPENSSL_VERSION were defined twice - in both >> .travis.yml and .tr

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

2017-01-28 Thread Илья Шипицин
this one is final v2 patch 2017-01-28 13:06 GMT+05:00 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.

[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 "

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

2017-01-28 Thread Илья Шипицин
sorry, I sent wrong patch. please ignore it 2017-01-28 13:01 GMT+05:00 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

[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 "

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

2017-01-27 Thread Илья Шипицин
please, ignore it. I will send v2 soon Пт, 27 янв. 2017 г. в 18:53, 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

[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 "