diff mbox series

build broken with --enable-gcrypt

Message ID 046eb29c-9c68-1baf-846e-decd65f567e0@linaro.org
State New
Headers show
Series build broken with --enable-gcrypt | expand

Commit Message

Richard Henderson Aug. 27, 2020, 9:18 p.m. UTC 
The meson conversion seems to have not been properly accomplished for the
gcrypt option: we fail to link against -lgcrypt.

I tried aping the nettle bits, like the following, but it has no effect.  I
don't understand how the "nettle" token works, honestly, since it doesn't
appear to be used at all.


r~

Comments

Daniel P. Berrangé Aug. 28, 2020, 9:02 a.m. UTC | #1 
On Thu, Aug 27, 2020 at 02:18:46PM -0700, Richard Henderson wrote:
> The meson conversion seems to have not been properly accomplished for the

> gcrypt option: we fail to link against -lgcrypt.

> 

> I tried aping the nettle bits, like the following, but it has no effect.  I

> don't understand how the "nettle" token works, honestly, since it doesn't

> appear to be used at all.


Hmm, a red flag that we're missing CI coverage of the different
crypto backends.  I'll look into fixing this and the meson issue.


> diff --git a/configure b/configure

> index b1e11397a8..4fd50c0275 100755

> --- a/configure

> +++ b/configure

> @@ -6972,6 +6972,8 @@ if test "$gcrypt" = "yes" ; then

>    if test "$gcrypt_hmac" = "yes" ; then

>      echo "CONFIG_GCRYPT_HMAC=y" >> $config_host_mak

>    fi

> +  echo "GCRYPT_CFLAGS=$gcrypt_cflags" >> $config_host_mak

> +  echo "GCRYPT_LIBS=$gcrypt_libs" >> $config_host_mak

>  fi

>  if test "$nettle" = "yes" ; then

>    echo "CONFIG_NETTLE=y" >> $config_host_mak

> diff --git a/meson.build b/meson.build

> index f0fe5f8799..d45c548191 100644

> --- a/meson.build

> +++ b/meson.build

> @@ -108,6 +108,11 @@ if 'CONFIG_NETTLE' in config_host

>    nettle = declare_dependency(compile_args: config_host['NETTLE_CFLAGS'].split(),

>                                link_args: config_host['NETTLE_LIBS'].split())

>  endif

> +gcrypt = not_found

> +if 'CONFIG_GCRYPT' in config_host

> +  gcrypt = declare_dependency(compile_args: config_host['GCRYPT_CFLAGS'].split(),

> +                              link_args: config_host['GCRYPT_LIBS'].split())

> +endif

>  gnutls = not_found

>  if 'CONFIG_GNUTLS' in config_host

>    gnutls = declare_dependency(compile_args: config_host['GNUTLS_CFLAGS'].split(),



Regards,
Daniel
-- 
|: https://berrange.com      -o-    https://www.flickr.com/photos/dberrange :|
|: https://libvirt.org         -o-            https://fstop138.berrange.com :|
|: https://entangle-photo.org    -o-    https://www.instagram.com/dberrange :|
diff mbox series

Patch

diff --git a/configure b/configure
index b1e11397a8..4fd50c0275 100755
--- a/configure
+++ b/configure
@@ -6972,6 +6972,8 @@  if test "$gcrypt" = "yes" ; then
   if test "$gcrypt_hmac" = "yes" ; then
     echo "CONFIG_GCRYPT_HMAC=y" >> $config_host_mak
   fi
+  echo "GCRYPT_CFLAGS=$gcrypt_cflags" >> $config_host_mak
+  echo "GCRYPT_LIBS=$gcrypt_libs" >> $config_host_mak
 fi
 if test "$nettle" = "yes" ; then
   echo "CONFIG_NETTLE=y" >> $config_host_mak
diff --git a/meson.build b/meson.build
index f0fe5f8799..d45c548191 100644
--- a/meson.build
+++ b/meson.build
@@ -108,6 +108,11 @@  if 'CONFIG_NETTLE' in config_host
   nettle = declare_dependency(compile_args: config_host['NETTLE_CFLAGS'].split(),
                               link_args: config_host['NETTLE_LIBS'].split())
 endif
+gcrypt = not_found
+if 'CONFIG_GCRYPT' in config_host
+  gcrypt = declare_dependency(compile_args: config_host['GCRYPT_CFLAGS'].split(),
+                              link_args: config_host['GCRYPT_LIBS'].split())
+endif
 gnutls = not_found
 if 'CONFIG_GNUTLS' in config_host
   gnutls = declare_dependency(compile_args: config_host['GNUTLS_CFLAGS'].split(),