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

2017-04-25 Thread Steffan Karger
Hi, On 25-04-17 20:51, Илья Шипицин wrote: > 2017-04-25 22:49 GMT+05:00 Steffan Karger >: > > On 25-04-17 09:50, Ilya Shipitsin wrote: > > Inspired by > https://www.mail-archive.com/openvpn-devel@lists.sourceforge.net/msg13032.html > >

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

2017-04-25 Thread Илья Шипицин
2017-04-25 22:49 GMT+05:00 Steffan Karger : > Hi, > > On 25-04-17 09:50, Ilya Shipitsin wrote: > > 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

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

2017-04-25 Thread Steffan Karger
Hi, On 25-04-17 09:50, Ilya Shipitsin wrote: > 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 > --- > v2: moved download/build dependencies into functi

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

2017-04-25 Thread Samuli Seppänen
-BEGIN PGP SIGNED MESSAGE- Hash: SHA256 On 25/04/2017 10:50, Ilya Shipitsin wrote: > Inspired by > https://www.mail-archive.com/openvpn-devel@lists.sourceforge.net/msg13 032.html > > build options are taken from regular windows installer builds > > Signed-off-by: Ilya Shipitsin --- v2:

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

2017-04-25 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 --- v2: moved download/build dependencies into functions, changed cross build detection from shell expansion ${CH

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

2017-04-10 Thread Samuli Seppänen
On 10/04/2017 00:44, David Sommerseth wrote: > On 09/04/17 15:31, Илья Шипицин wrote: >> >> how did you came to conclusion that it means poor code quality? >> is it written somewhere on the https://openvpn.net on how to write bash >> scripts ? > > Steffan have given a good reasoning, and I complet

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

2017-04-09 Thread David Sommerseth
On 09/04/17 15:31, Илья Шипицин wrote: > > how did you came to conclusion that it means poor code quality? > is it written somewhere on the https://openvpn.net on how to write bash > scripts ? Steffan have given a good reasoning, and I completely agree with him. Further, it is impossible to writ

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

2017-04-09 Thread Steffan Karger
Hi, [This will be my last on-list reply about this.] On 09-04-17 19:10, Илья Шипицин wrote: > > > 2017-04-09 21:58 GMT+05:00 Steffan Karger >: > > Hi, > > On 09-04-17 15:31, Илья Шипицин wrote: > > > > @@ -70,7 +84,6 @@ if [ "${TRAVIS_OS_NAME}" !

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

2017-04-09 Thread Илья Шипицин
2017-04-09 21:58 GMT+05:00 Steffan Karger : > Hi, > > On 09-04-17 15:31, Илья Шипицин wrote: > > > > @@ -70,7 +84,6 @@ if [ "${TRAVIS_OS_NAME}" != "osx" ]; then > > > > fi > > > > > > > > # Download and build crypto lib > > > > -mkdir -p download-cache > >

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

2017-04-09 Thread Steffan Karger
Hi, On 09-04-17 15:31, Илья Шипицин wrote: > > > @@ -70,7 +84,6 @@ if [ "${TRAVIS_OS_NAME}" != "osx" ]; then > > > fi > > > > > > # Download and build crypto lib > > > -mkdir -p download-cache > > > if [ "${SSLLIB}" = "openssl" ]; then > >

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

2017-04-09 Thread Илья Шипицин
2017-04-09 17:03 GMT+05:00 Steffan Karger : > Hi, > > On 09-04-17 12:54, Илья Шипицин wrote: > > > > 2017-04-09 13:44 GMT+05:00 Steffan Karger > >: > > > > On 26-03-17 13:21, Ilya Shipitsin wrote: > > > + TAP_CFLAGS="-I${PWD}/tap-windows-${TAP_WINDOWS_VERSIO

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

2017-04-09 Thread Steffan Karger
Hi, On 09-04-17 12:54, Илья Шипицин wrote: > > 2017-04-09 13:44 GMT+05:00 Steffan Karger >: > > On 26-03-17 13:21, Ilya Shipitsin wrote: > > + TAP_CFLAGS="-I${PWD}/tap-windows-${TAP_WINDOWS_VERSION}/include" > LZO_CFLAGS="-I${PREFIX}/include" LZO_LIBS=

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

2017-04-09 Thread Илья Шипицин
2017-04-09 13:44 GMT+05:00 Steffan Karger : > Hi, > > On 26-03-17 13:21, Ilya Shipitsin wrote: > > Inspired by https://www.mail-archive.com/openvpn-devel@lists. > sourceforge.net/msg13032.html > > build options are taken from regular windows installer builds > > Feature-ACK. Adding cross-compile

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

2017-04-09 Thread Steffan Karger
Hi, On 26-03-17 13:21, Ilya Shipitsin wrote: > Inspired by > https://www.mail-archive.com/openvpn-devel@lists.sourceforge.net/msg13032.html > build options are taken from regular windows installer builds Feature-ACK. Adding cross-compile builds for Windows on travis is a good plan. Initial fee

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

2017-03-28 Thread Steffan Karger
Hi, On 28 March 2017 at 08:50, Илья Шипицин wrote: > I opened https://github.com/OpenVPN/openvpn/pull/85 > for discussion > > thoughts ? I put the patch on my review list, but need to tackle other things first. Will get back to you. -Steffan ---

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

2017-03-27 Thread Илья Шипицин
I opened https://github.com/OpenVPN/openvpn/pull/85 for discussion thoughts ? 2017-03-26 16:21 GMT+05:00 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

[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 --- .travis.yml | 30 +--- .travis/build-deps.sh | 55