[packages/zlib] fix cc misdetection
atler
atler at pld-linux.org
Mon Mar 28 11:01:31 CEST 2022
commit 6f901dc3c490e541a8941459b06d641dee495f47
Author: Jan Palus <atler at pld-linux.org>
Date: Mon Mar 28 11:00:36 2022 +0200
fix cc misdetection
from https://github.com/madler/zlib/pull/607
cc.patch | 43 +++++++++++++++++++++++++++++++++++++++++++
zlib.spec | 2 ++
2 files changed, 45 insertions(+)
---
diff --git a/zlib.spec b/zlib.spec
index c7356af..352e781 100644
--- a/zlib.spec
+++ b/zlib.spec
@@ -24,6 +24,7 @@ Group: Libraries
Source0: http://www.zlib.net/current/%{name}-%{version}.tar.gz
# Source0-md5: 5fc414a9726be31427b440b434d05f78
Patch0: %{name}-asm.patch
+Patch1: cc.patch
URL: http://www.zlib.net/
BuildRequires: autoconf >= 2.50
BuildRequires: automake
@@ -302,6 +303,7 @@ cp contrib/asm686/match.S .
cp contrib/amd64/amd64-match.S match.S
%endif
%endif
+%patch1 -p1
%build
CC="%{__cc}" \
diff --git a/cc.patch b/cc.patch
new file mode 100644
index 0000000..f34c404
--- /dev/null
+++ b/cc.patch
@@ -0,0 +1,43 @@
+From 80d086357a55b94a13e43756cf3e131f25eef0e4 Mon Sep 17 00:00:00 2001
+From: Sam James <sam at gentoo.org>
+Date: Mon, 28 Mar 2022 08:40:45 +0100
+Subject: [PATCH] Fix CC logic in configure
+
+In https://github.com/madler/zlib/commit/e9a52aa129efe3834383e415580716a7c4027f8d,
+the logic was changed to try check harder for GCC, but it dropped
+the default setting of cc=${CC}. It was throwing away any pre-set CC value as
+a result.
+
+The rest of the script then cascades down a bad path because it's convinced
+it's not GCC or a GCC-like compiler.
+
+This led to e.g. misdetection of inability to build shared libs
+for say, multilib cases (w/ CC being one thing from the environment being used
+for one test (e.g. x86_64-unknown-linux-gnu-gcc -m32 and then 'cc' used for
+shared libs (but missing "-m32"!)). Obviously just one example of how
+the old logic could break.
+
+This restores the old default of 'CC' if nothing overrides it later
+in configure.
+
+Bug: https://bugs.gentoo.org/836308
+Signed-off-by: Sam James <sam at gentoo.org>
+---
+ configure | 3 +++
+ 1 file changed, 3 insertions(+)
+
+diff --git a/configure b/configure
+index 52ff4a04e..3fa3e8618 100755
+--- a/configure
++++ b/configure
+@@ -174,7 +174,10 @@ if test -z "$CC"; then
+ else
+ cc=${CROSS_PREFIX}cc
+ fi
++else
++ cc=${CC}
+ fi
++
+ cflags=${CFLAGS-"-O3"}
+ # to force the asm version use: CFLAGS="-O3 -DASMV" ./configure
+ case "$cc" in
================================================================
---- gitweb:
http://git.pld-linux.org/gitweb.cgi/packages/zlib.git/commitdiff/6f901dc3c490e541a8941459b06d641dee495f47
More information about the pld-cvs-commit
mailing list