Hey there.

I was trying to build ecore earlier today and apparently the libgcrypt
CFLAGS and linker flags were not being passed as expected (GnuTLS
support is enabled).

In the end, it looks like the code in ecore_check_options.m4 was failing
here:

  checking for libgcrypt-config... /usr/local/bin/libgcrypt-config
  checking for libgcrypt... yes
  ./configure: TLS_CFLAGS+= -I/usr/local/include: not found
  ./configure: TLS_LIBS+= -L/usr/local/lib -lgcrypt -lgpg-error: not
               found

The attached patch solves the issue by not using '+=' to append the
values to TLS_{CFLAGS,LIBS}. I thought of using AS_VAR_APPEND, but I
don't know if there are autoconf versions we are supposed to support
which do not have that macro.

The patch should also apply cleanly to the 1.0, 1.1 and 1.2
branches. Please let me know if there should be a NEWS/ChangeLog entry.

Index: m4/ecore_check_options.m4
===================================================================
--- m4/ecore_check_options.m4	(revision 70923)
+++ m4/ecore_check_options.m4	(working copy)
@@ -178,8 +178,8 @@
    if test "x$_ecore_have_gnutls" = "xyes";then
      AC_PATH_GENERIC([libgcrypt], [], [_ecore_have_gnutls="yes"], [_ecore_have_gnutls="no"])
         if test "x${_ecore_have_gnutls}" = "xyes" ; then
-           TLS_CFLAGS+=" ${LIBGCRYPT_CFLAGS}"
-           TLS_LIBS+=" ${LIBGCRYPT_LIBS}"
+           TLS_CFLAGS="${TLS_CFLAGS} ${LIBGCRYPT_CFLAGS}"
+           TLS_LIBS="${TLS_LIBS} ${LIBGCRYPT_LIBS}"
         fi
    fi
 
------------------------------------------------------------------------------
Live Security Virtual Conference
Exclusive live event will cover all the ways today's security and 
threat landscape has changed and how IT managers can respond. Discussions 
will include endpoint security, mobile security and the latest in malware 
threats. http://www.accelacomm.com/jaw/sfrnl04242012/114/50122263/
_______________________________________________
enlightenment-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/enlightenment-devel

Reply via email to