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?

-Steffan

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

Reply via email to