[oe] [PATCH][meta-networking][v3] squid: upgrade to 3.5.5
Joe MacDonald
Joe_MacDonald at mentor.com
Tue Sep 1 13:31:53 UTC 2015
[Re: [oe] [PATCH][meta-networking][v3] squid: upgrade to 3.5.5] On 15.09.01 (Tue 15:40) Rongqing Li wrote:
> ping
I marked this patch rejected in patchworks after asking about the PAM
dependency, I encourage everyone to check the state of any patch there
as that's my primary workflow now since I know that's also the
management system Martin uses and I want to make the coordination as
easy as possible.
I initially missed your follow up because it was after the unrelated
comment about a queue of patches that'd recently been merged.
Your response was that there was no configuration option to disable PAM
after you looked at the code. I had a quick look through and the issue
is more complicated than that. There absolutely is a way to turn off
building the PAM module (see --enable-auth-basic) but that will only
address one of a large number of potential QA issues, now that I look at
it. The default will also depend on LDAP, NIS, SASL, RADIUS and so on.
I think the correct solution for this would be to have each of the
helpers optionally enabled based on what your image contains
(PKGCONFIG/DISTRO_FEATURES maybe) but for now it's probably okay to
remove the dependency and re-submit something like below without change
#5. Please also have another look at the error from the world builds to
see if we need to either blacklist Squid or if there's a 'insane'
workaround that'll squelch the error until we get it cleaned up
properly.
Then on the upgrade to 3.5.7 (the current stable) we'll want to do
something cleaner.
-J.
>
> -Roy
>
> On 2015年07月07日 09:55, rongqing.li at windriver.com wrote:
> >From: Roy Li <rongqing.li at windriver.com>
> >
> >1. Remove the squid-change-ksh-referen*.patch which is not needed, since
> > 3.5.5 did not use ksh by default.
> >2. Update the checksum of COPYING,since the date in it has been changed.
> >3. Define BUILDCXXFLAGS, otherwise the target gcc options -std=c++11 will
> > add into it, and lead to building failure since host gcc maybe not
> > support "-std=c++11"
> >4. Assume to support GNU atomic operations by default, the running check
> > on cross-compile setup does not work
> >5. Add the dependency on libpam
> >
> >Signed-off-by: Roy Li <rongqing.li at windriver.com>
> >---
> > ...ge-ksh-reference-in-krb-ldap-helper-to-sh.patch | 39 ----------------------
> > .../squid/{squid_3.4.7.bb => squid_3.5.5.bb} | 11 +++---
> > 2 files changed, 6 insertions(+), 44 deletions(-)
> > delete mode 100644 meta-networking/recipes-daemons/squid/files/squid-change-ksh-reference-in-krb-ldap-helper-to-sh.patch
> > rename meta-networking/recipes-daemons/squid/{squid_3.4.7.bb => squid_3.5.5.bb} (86%)
> >
> >diff --git a/meta-networking/recipes-daemons/squid/files/squid-change-ksh-reference-in-krb-ldap-helper-to-sh.patch b/meta-networking/recipes-daemons/squid/files/squid-change-ksh-reference-in-krb-ldap-helper-to-sh.patch
> >deleted file mode 100644
> >index 8e03860..0000000
> >--- a/meta-networking/recipes-daemons/squid/files/squid-change-ksh-reference-in-krb-ldap-helper-to-sh.patch
> >+++ /dev/null
> >@@ -1,39 +0,0 @@
> >-squid: change ksh reference in krb ldap helper to sh
> >-
> >-Very simple cert_tool script with no ksh specifics.
> >-Change it to use sh so a package dependency doesn't
> >-get created to ksh.
> >-
> >-Upstream-Status: Pending
> >-
> >-Signed-off-by: Jim Somerville <Jim.Somerville at windriver.com>
> >-Signed-off-by: Chong Lu <Chong.Lu at windriver.com>
> >----
> >- helpers/external_acl/kerberos_ldap_group/cert_tool | 6 +++---
> >- 1 file changed, 3 insertions(+), 3 deletions(-)
> >-
> >-diff --git a/helpers/external_acl/kerberos_ldap_group/cert_tool b/helpers/external_acl/kerberos_ldap_group/cert_tool
> >-index 9f14959..8c4ea11 100644
> >---- a/helpers/external_acl/kerberos_ldap_group/cert_tool
> >-+++ b/helpers/external_acl/kerberos_ldap_group/cert_tool
> >-@@ -1,4 +1,4 @@
> >--#!/bin/ksh
> >-+#!/bin/sh
> >- #
> >- # -----------------------------------------------------------------------------
> >- #
> >-@@ -64,9 +64,9 @@ QUIT
> >- # Create database for Sun ldap and pem file for Openldap
> >- #
> >- rm ${server}_[0-9]*.pem 2>/dev/null
> >--let i=0
> >-+i=0
> >- ls ${server}_[0-9]*.cert | while read file; do
> >-- let i=i+1
> >-+ i=`expr $i + 1`
> >- cat $file >> ${server}_$i.pem
> >- CA=`openssl x509 -noout -text -in ${server}_$i.pem | grep -i "CA:.*true"`
> >- if [ -n "$CA" ]; then
> >---
> >-1.9.1
> >-
> >diff --git a/meta-networking/recipes-daemons/squid/squid_3.4.7.bb b/meta-networking/recipes-daemons/squid/squid_3.5.5.bb
> >similarity index 86%
> >rename from meta-networking/recipes-daemons/squid/squid_3.4.7.bb
> >rename to meta-networking/recipes-daemons/squid/squid_3.5.5.bb
> >index 7b00dfc..198319d 100644
> >--- a/meta-networking/recipes-daemons/squid/squid_3.4.7.bb
> >+++ b/meta-networking/recipes-daemons/squid/squid_3.5.5.bb
> >@@ -16,19 +16,18 @@ SRC_URI = "http://www.squid-cache.org/Versions/v${MAJ_VER}/${MIN_VER}/${BPN}-${P
> > file://Set-up-for-cross-compilation.patch \
> > file://Skip-AC_RUN_IFELSE-tests.patch \
> > file://Fix-flawed-dynamic-ldb-link-test-in-configure.patch \
> >- file://squid-change-ksh-reference-in-krb-ldap-helper-to-sh.patch \
> > file://squid-use-serial-tests-config-needed-by-ptest.patch \
> > file://run-ptest \
> > file://volatiles.03_squid \
> > "
> >
> > LIC_FILES_CHKSUM = "file://COPYING;md5=c492e2d6d32ec5c1aad0e0609a141ce9 \
> >- file://COPYRIGHT;md5=2900f50789c498be8e9f1eb23b55cbe9 \
> >+ file://errors/COPYRIGHT;md5=0fed8f1462f6fdbc62bb431bcb618f46 \
> > "
> >-SRC_URI[md5sum] = "9951034b10f7ee0f45a95cfae61c57c2"
> >-SRC_URI[sha256sum] = "7b423f3d3495a317503ca559ea535f80445fd7c4e3c3e268cb7a8c97c61af2b6"
> >+SRC_URI[md5sum] = "6aac5c2e9cbbeabcbf2e9e49a178a931"
> >+SRC_URI[sha256sum] = "741c24a307c50f0d845d53cabb66b36d91ce9a73c8a165eae5def5e4d11e6a0d"
> >
> >-DEPENDS = "libtool krb5 openldap db cyrus-sasl"
> >+DEPENDS = "libtool krb5 openldap db cyrus-sasl libpam"
> >
> > inherit autotools useradd ptest
> >
> >@@ -38,6 +37,8 @@ USERADD_PARAM_${PN} = "--system --no-create-home --home-dir /var/run/squid --she
> > PACKAGECONFIG ??= ""
> > PACKAGECONFIG[libnetfilter-conntrack] = "--with-netfilter-conntrack=${includedir}, --without-netfilter-conntrack, libnetfilter-conntrack"
> > EXTRA_OECONF += "--with-default-user=squid"
> >+export BUILDCXXFLAGS="${BUILD_CXXFLAGS}"
> >+CACHED_CONFIGUREVARS += "squid_cv_gnu_atomics=yes"
> >
> > TESTDIR = "test-suite"
> > do_compile_ptest() {
> >
>
--
-Joe MacDonald.
:wq
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 473 bytes
Desc: Digital signature
URL: <http://lists.openembedded.org/pipermail/openembedded-devel/attachments/20150901/6d89049e/attachment-0002.sig>
More information about the Openembedded-devel
mailing list