From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail-wm0-f51.google.com (mail-wm0-f51.google.com [74.125.82.51]) by mail.openembedded.org (Postfix) with ESMTP id C2E5E6057A for ; Tue, 17 Nov 2015 21:44:53 +0000 (UTC) Received: by wmdw130 with SMTP id w130so173992235wmd.0 for ; Tue, 17 Nov 2015 13:44:54 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20120113; h=message-id:subject:from:to:cc:date:in-reply-to:references :content-type:mime-version:content-transfer-encoding; bh=XJpV8J1wGVDPTrcAAVKEjwizp7LjSa5XmIzVhy+juIc=; b=NJgGZVbh8jLYfGwqo63/WFr/H+cepdasav3vsFkBgw+QDeC9j7Y0e9r6eeELKBb/CA BrEW1PGWAWpptb2kE2laM3/YKjWdvolTjUTtdRBPXnU1FPbN1jKB/+pB688YFjfZrgOk H+uHqa8veWo220UQ7y/TJsY1VOROkGKHWPCtami8am9X7pPyU5g13OeWy+CuKzO8QT+y A4lAc4DpevNPXL7w2oR3f0/ggXItopCuIL/x08BKgQu9f3gte41m5BKZIdCSnsGX6hfv 49L8pA22sxUEk5QksJbv11RkCk5MtNTeIMo6HZ0F6wvdzrgFloQjJ05GKTbJrqUqvu3h rMAw== X-Received: by 10.28.22.21 with SMTP id 21mr4843242wmw.72.1447796694275; Tue, 17 Nov 2015 13:44:54 -0800 (PST) Received: from chl500346 (c-82-192-248-24.customer.ggaweb.ch. [82.192.248.24]) by smtp.gmail.com with ESMTPSA id ot9sm28564859wjc.12.2015.11.17.13.44.53 (version=TLS1_2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128/128); Tue, 17 Nov 2015 13:44:53 -0800 (PST) Message-ID: <1447796692.2296.21.camel@gmail.com> From: Adrian Freihofer To: openembedded-devel@lists.openembedded.org Date: Tue, 17 Nov 2015 22:44:52 +0100 In-Reply-To: <20151116213457.GG2240@jama> References: <1447271300-4153-1-git-send-email-adrian.freihofer@gmail.com> <1447271300-4153-8-git-send-email-adrian.freihofer@gmail.com> <20151116213457.GG2240@jama> X-Mailer: Evolution 3.16.5 (3.16.5-3.fc22) Mime-Version: 1.0 Subject: Re: [meta-oe][PATCH V2 07/17] networkmanager: support gnutls and nss X-BeenThere: openembedded-devel@lists.openembedded.org X-Mailman-Version: 2.1.12 Precedence: list List-Id: Using the OpenEmbedded metadata to build Distributions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Tue, 17 Nov 2015 21:44:59 -0000 Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: 7bit On Mon, 2015-11-16 at 22:34 +0100, Martin Jansa wrote: > On Wed, Nov 11, 2015 at 08:48:10PM +0100, Adrian Freihofer wrote: > > Signed-off-by: Adrian Freihofer > > --- > > .../0002-add-pkg-config-for-libgcrypt.patch | 41 ++++++++++++++++++++++ > > .../networkmanager/networkmanager_1.0.6.bb | 14 ++++++-- > > 2 files changed, 52 insertions(+), 3 deletions(-) > > create mode 100644 meta-oe/recipes-connectivity/networkmanager/networkmanager/0002-add-pkg-config-for > > -libgcrypt.patch > > > > diff --git a/meta-oe/recipes-connectivity/networkmanager/networkmanager/0002-add-pkg-config-for-libgcrypt.patch > > b/meta-oe/recipes-connectivity/networkmanager/networkmanager/0002-add-pkg-config-for-libgcrypt.patch > > new file mode 100644 > > index 0000000..bb95eaa > > --- /dev/null > > +++ b/meta-oe/recipes-connectivity/networkmanager/networkmanager/0002-add-pkg-config-for-libgcrypt.patch > > @@ -0,0 +1,41 @@ > > +From a9261fee4a41045c42e0d292237cee532e11982c Mon Sep 17 00:00:00 2001 > > +From: Adrian > > +Date: Thu, 27 Nov 2014 15:43:26 +0100 > > +Subject: [PATCH] add pkg config for libgcrypt > > + > > +--- > > + configure.ac | 18 +++++++----------- > > + 1 file changed, 7 insertions(+), 11 deletions(-) > > + > > +diff --git a/configure.ac b/configure.ac > > +index 94b0758..654a015 100644 > > +--- a/configure.ac > > ++++ b/configure.ac > > +@@ -480,17 +480,13 @@ if test x"$ac_crypto" = xnss; then > > + with_nss=yes > > + elif test x"$ac_crypto" = xgnutls; then > > + PKG_CHECK_MODULES(GNUTLS, [gnutls >= 1.2]) > > +- AC_PATH_PROG(LIBGCRYPT_CONFIG, libgcrypt-config, no) > > +- if test x"$LIBGCRYPT_CONFIG" = xno; then > > +- AC_MSG_ERROR([gnutls explicitly requested but gcrypt not found on system]) > > +- else > > +- AC_DEFINE(HAVE_GNUTLS, 1, [Define if you have libgnutls]) > > +- LIBGCRYPT_CFLAGS=`$LIBGCRYPT_CONFIG --cflags` > > +- LIBGCRYPT_LIBS=`$LIBGCRYPT_CONFIG --libs` > > +- AC_SUBST(LIBGCRYPT_CFLAGS) > > +- AC_SUBST(LIBGCRYPT_LIBS) > > +- with_gnutls=yes > > +- fi > > ++ AC_DEFINE(HAVE_GNUTLS, 1, [Define if you have libgnutls]) > > ++ PKG_CHECK_MODULES(LIBGCRYPT, [libgcrypt >= 1.1.42], [ > > ++ AC_DEFINE(HAVE_GCRYPT, 1, [Define if gcrypt library is available.]) > > ++ with_gnutls=yes > > ++ ], [ > > ++ AC_MSG_ERROR([gnutls explicitly requested but gcrypt not found on system]) > > ++ ]) > > + else > > + AC_MSG_ERROR([Please choose either 'nss' or 'gnutls' for certificate and crypto operations]) > > + fi > > +-- > > +1.9.1 > > + > > diff --git a/meta-oe/recipes-connectivity/networkmanager/networkmanager_1.0.6.bb b/meta-oe/recipes > > -connectivity/networkmanager/networkmanager_1.0.6.bb > > index d529b4a..3a59f83 100644 > > --- a/meta-oe/recipes-connectivity/networkmanager/networkmanager_1.0.6.bb > > +++ b/meta-oe/recipes-connectivity/networkmanager/networkmanager_1.0.6.bb > > @@ -6,13 +6,14 @@ LIC_FILES_CHKSUM = "file://COPYING;md5=cbbffd568227ada506640fe950a4823b \ > > file://libnm-util/COPYING;md5=1c4fa765d6eb3cd2fbd84344a1b816cd \ > > file://docs/api/html/license.html;md5=51d7fb67bde992e58533a8481cee070b" > > > > -DEPENDS = "libnl dbus dbus-glib libgudev wireless-tools nss util-linux libndp" > > +DEPENDS = "libnl dbus dbus-glib libgudev wireless-tools util-linux libndp" > > > > inherit gnomebase gettext systemd > > > > SRC_URI = " \ > > ${GNOME_MIRROR}/NetworkManager/${@gnome_verdir("${PV}")}/NetworkManager-${PV}.tar.xz \ > > file://0001-don-t-try-to-run-sbin-dhclient-to-get-the-version-nu.patch \ > > + file://0002-add-pkg-config-for-libgcrypt.patch \ > > " > > > > SRC_URI[md5sum] = "00f5f9ec69725a9f9b99366853c6f73e" > > @@ -26,7 +27,6 @@ EXTRA_OECONF = " \ > > --disable-ifnet \ > > --disable-ifcfg-suse \ > > --with-netconfig \ > > - --with-crypto=nss \ > > --disable-more-warnings \ > > --with-dhclient=${base_sbindir}/dhclient \ > > --with-iptables=${sbindir}/iptables \ > > @@ -34,7 +34,13 @@ EXTRA_OECONF = " \ > > --with-dnsmasq=${bindir}/dnsmasq \ > > " > > > > -PACKAGECONFIG ??= "${@bb.utils.contains('DISTRO_FEATURES','systemd','systemd','consolekit',d)}" > > +# If you need a different configuration for networkmanager, add lines similar to > > +# the following example to your local.conf or distro config > > +# PACKAGECONFIG_remove_pn-networkmanager = "nss" > > +# PACKAGECONFIG_append_pn-networkmanager = " gnutls modemmanager ppp" > > +PACKAGECONFIG ??= "nss \ > > + ${@bb.utils.contains('DISTRO_FEATURES','systemd','systemd','consolekit',d)} \ > > +" > > PACKAGECONFIG[systemd] = " \ > > --with-systemdsystemunitdir=${systemd_unitdir}/system --with-session-tracking=systemd --enable-polkit, \ > > --without-systemdsystemunitdir, \ > > @@ -46,6 +52,8 @@ PACKAGECONFIG[bluez5] = "--enable-bluez5-dun,--disable-bluez5-dun,bluez5" > > PACKAGECONFIG[concheck] = "--with-libsoup=yes,--with-libsoup=no,libsoup-2.4" > > PACKAGECONFIG[modemmanager] = "--with-modem-manager-1=yes,--with-modem-manager-1=no,modemmanager" > > PACKAGECONFIG[ppp] = "--enable-ppp,--disable-ppp,ppp" > > +PACKAGECONFIG[nss] = "--with-crypto=nss,,nss" > > won't it detect nss when available in sysroot? Why is nss available in sysroot if not because of networkmanager depends on it? > > > +PACKAGECONFIG[gnutls] = "--with-crypto=gnutls,,gnutls libgcrypt,gnutls libgcrypt" > > Why do you need to add gnutls and libgcrypt manually to RDEPENDS? Your are right. Should be: +PACKAGECONFIG[gnutls] = "--with-crypto=gnutls,,gnutls libgcrypt" May you apply another fixup for me? (Sorry, I can't see a way how I could do this task.) > > > > > PACKAGES =+ "libnmutil libnmglib libnmglib-vpn ${PN}-tests ${PN}-bash-completion" > > > > -- > > 2.4.3 > > > > -- > > _______________________________________________ > > Openembedded-devel mailing list > > Openembedded-devel@lists.openembedded.org > > http://lists.openembedded.org/mailman/listinfo/openembedded-devel > > -- > _______________________________________________ > Openembedded-devel mailing list > Openembedded-devel@lists.openembedded.org > http://lists.openembedded.org/mailman/listinfo/openembedded-devel