[ovs-dev] [PATCH] configure: Enable OpenSSL support by default.

Justin Pettit jpettit at nicira.com
Wed Aug 25 22:36:23 UTC 2010


On Aug 20, 2010, at 11:26 AM, Ben Pfaff wrote:

> -     [AC_HELP_STRING([--enable-ssl], 
> -                     [Enable ssl support (requires libssl)])],
> +     [AC_HELP_STRING([--disable-ssl], [Disable OpenSSL support])],
>      [case "${enableval}" in
>         (yes) ssl=true ;;
>         (no)  ssl=false ;;
>         (*) AC_MSG_ERROR([bad value ${enableval} for --enable-ssl]) ;;

Did you want to change this message to "disable-ssl"?  Shouldn't the results of those "(yes)" and "(no)" cases be reversed in the case of disabling something?

My tree shows that "debian/rules" still passes in "--enable-ssl".  Did you want to remove that?

This is a great change to have.  Thanks.

--Justin






More information about the dev mailing list