2016-09-17 16:17 GMT+05:00 Steffan Karger <stef...@karger.me>:

> Hi,
>
> On 17 September 2016 at 12:54, Ilya Shipitsin <chipits...@gmail.com>
> wrote:
> > ---
> >  .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: SSLLIB="openssl" EXTRA_CONFIG="--disable-crypto"
> > -      os: linux
> > -      compiler: clang
> >    exclude:
> >      - compiler: gcc
> >
> > --
> > 2.5.5
>
> Thanks.  The patch is good, so ACK.
>
> But the commit messages should describe why this change is needed.  In
> this case:
>
> Previously, 'make test' failed for --disable-crypto builds.  Since
> that is not fixed, we should no longer accept --disable-crypto builds
> to fail 'make test' on travis.
>
> Could you please include such descriptions in future patches?
>

ok, I'm going to play with travis-ci + valgrind soon.

or, some powershell for windows testing (it's on my list)



>
> -Steffan
>
------------------------------------------------------------------------------
_______________________________________________
Openvpn-devel mailing list
Openvpn-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/openvpn-devel

Reply via email to