Re: [Qemu-devel] [PATCH] rbd: don't link with -lcrypto

[Christian Brunner]

You are right, libcrypto can be omitted.

(Recent versions of ceph use crypto++ anyway.)

Thanks, Christian


Reviewed-by: Christian Brunner <address@hidden>


2011/3/7 Aurelien Jarno <address@hidden>:
> rbd support tries to both link with -lrados and -lcrypto. While the
> first one is of course necessary, the second is not necessary (only
> librados ifself needs to link with libcrypto).
>
> This fixes a licensing issue: qemu as a whole is GPL v2, and thus can't
> be linked with OpenSSL without an exception in the license, which seems
> difficult to get given the number of persons involved.
>
> Cc: Christian Brunner <address@hidden>
> Signed-off-by: Aurelien Jarno <address@hidden>
> ---
>  configure |    2 +-
>  1 files changed, 1 insertions(+), 1 deletions(-)
>
> diff --git a/configure b/configure
> index ef51a58..5932e39 100755
> --- a/configure
> +++ b/configure
> @@ -1801,7 +1801,7 @@ if test "$rbd" != "no" ; then
>  #include <rados/librados.h>
>  int main(void) { rados_initialize(0, NULL); return 0; }
>  EOF
> -  rbd_libs="-lrados -lcrypto"
> +  rbd_libs="-lrados"
>   if compile_prog "" "$rbd_libs" ; then
>     librados_too_old=no
>     cat > $TMPC <<EOF
> --
> 1.7.2.3
>
>
>