[OE-core] [oe-core][PATCH] v2 cups: Shouldn't link against libgcrypt without using gcrypt functions
akuster808
akuster808 at gmail.com
Mon Jun 9 21:06:34 UTC 2014
Saul,
Thanks. I have booked marked
http://openembedded.org/wiki/Commit_Patch_Message_Guidelines
and maybe I will to remember to review that.
- Armin
On 06/09/2014 01:43 PM, Saul Wold wrote:
> On 06/09/2014 01:26 PM, Armin Kuster wrote:
>> backported patch from cups.org to address the cryto dependency issue
>> I am seeing.
>>
>> http://www.cups.org/str.php?L4399
>>
>> Signed-off-by: Armin Kuster <akuster at mvista.com>
>> ---
>> meta/recipes-extended/cups/cups.inc | 1 +
>> .../cups/cups/cups-no-gcrypt.patch | 40
>> ++++++++++++++++++++++
>> 2 files changed, 41 insertions(+)
>> create mode 100644
>> meta/recipes-extended/cups/cups/cups-no-gcrypt.patch
>>
>> diff --git a/meta/recipes-extended/cups/cups.inc
>> b/meta/recipes-extended/cups/cups.inc
>> index fd88552..3b8027b 100644
>> --- a/meta/recipes-extended/cups/cups.inc
>> +++ b/meta/recipes-extended/cups/cups.inc
>> @@ -8,6 +8,7 @@ SRC_URI =
>> "http://www.cups.org/software/${PV}/${BP}-source.tar.bz2 \
>> file://use_echo_only_in_init.patch \
>> file://0001-don-t-try-to-run-generated-binaries.patch \
>> file://cups_serverbin.patch \
>> + file://cups-no-gcrypt.patch \
>> "
>>
>> LEAD_SONAME = "libcupsdriver.so"
>> diff --git a/meta/recipes-extended/cups/cups/cups-no-gcrypt.patch
>> b/meta/recipes-extended/cups/cups/cups-no-gcrypt.patch
>> new file mode 100644
>> index 0000000..465d353
>> --- /dev/null
>> +++ b/meta/recipes-extended/cups/cups/cups-no-gcrypt.patch
>
> Please add an Upstream-Status: Backport and included the pointer to
> the bug entry.
>
> The patch file itself needs a Signed-off-by: tag.
>
> Thanks
> Sau!
>
>> @@ -0,0 +1,40 @@
>> +Description: Don't build-depend on libgcrypt, as nothing is used
>> from it
>> +Author: Didier Raboud <odyx at debian.org>
>> +Bug-Debian: https://bugs.debian.org/638416
>> +Bug: http://www.cups.org/str.php?L????
>> +Last-Update: 2014-04-07
>> +--- a/config-scripts/cups-ssl.m4
>> ++++ b/config-scripts/cups-ssl.m4
>> +@@ -66,7 +66,6 @@
>> + dnl Then look for GNU TLS...
>> + if test $have_ssl = 0 -a "x$enable_gnutls" != "xno" -a
>> "x$PKGCONFIG" != x; then
>> + AC_PATH_PROG(LIBGNUTLSCONFIG,libgnutls-config)
>> +- AC_PATH_PROG(LIBGCRYPTCONFIG,libgcrypt-config)
>> + if $PKGCONFIG --exists gnutls; then
>> + have_ssl=1
>> + SSLLIBS=`$PKGCONFIG --libs gnutls`
>> +@@ -84,14 +83,6 @@
>> + if test $have_ssl = 1; then
>> + CUPS_SERVERCERT="ssl/server.crt"
>> + CUPS_SERVERKEY="ssl/server.key"
>> +-
>> +- if $PKGCONFIG --exists gcrypt; then
>> +- SSLLIBS="$SSLLIBS `$PKGCONFIG --libs gcrypt`"
>> +- SSLFLAGS="$SSLFLAGS `$PKGCONFIG --cflags gcrypt`"
>> +- elif test "x$LIBGCRYPTCONFIG" != x; then
>> +- SSLLIBS="$SSLLIBS `$LIBGCRYPTCONFIG --libs`"
>> +- SSLFLAGS="$SSLFLAGS `$LIBGCRYPTCONFIG --cflags`"
>> +- fi
>> + fi
>> + fi
>> +
>> +--- a/cups/http-private.h
>> ++++ b/cups/http-private.h
>> +@@ -80,7 +80,6 @@
>> + # elif defined HAVE_GNUTLS
>> + # include <gnutls/gnutls.h>
>> + # include <gnutls/x509.h>
>> +-# include <gcrypt.h>
>> + # elif defined(HAVE_CDSASSL)
>> + # include <CoreFoundation/CoreFoundation.h>
>> + # include <Security/Security.h>
>>
More information about the Openembedded-core
mailing list