diff options
| author |   Sam James <sam@gentoo.org> | 2022-03-28 08:42:47 +0100 |
|---|---|---|
| committer | Sam James <sam@gentoo.org> | 2022-03-28 08:45:45 +0100 |
| commit | ffafdfd3eed513c49aa7047f4b09fd1dd0bb6768 (patch) | |
| tree | c869f46091df916e972a5f89a3ce85be865ef93d /sys-libs/zlib/files | |
| parent | net-firewall/ipt_netflow: remove redunant ${S} variable from pkg_setup (diff) | |
| download | gentoo-ffafdfd3eed513c49aa7047f4b09fd1dd0bb6768.tar.gz gentoo-ffafdfd3eed513c49aa7047f4b09fd1dd0bb6768.tar.bz2 gentoo-ffafdfd3eed513c49aa7047f4b09fd1dd0bb6768.zip | |
sys-libs/zlib: add link to upstream PR for 1.2.12 configure
Bug: https://bugs.gentoo.org/836308
Signed-off-by: Sam James <sam@gentoo.org>
Diffstat (limited to 'sys-libs/zlib/files')
| -rw-r--r-- | sys-libs/zlib/files/zlib-1.2.12-fix-CC-logic-in-configure.patch | 28 |
1 files changed, 22 insertions, 6 deletions
diff --git a/sys-libs/zlib/files/zlib-1.2.12-fix-CC-logic-in-configure.patch b/sys-libs/zlib/files/zlib-1.2.12-fix-CC-logic-in-configure.patch index da9455519b6f..8b6c5e26a766 100644 --- a/sys-libs/zlib/files/zlib-1.2.12-fix-CC-logic-in-configure.patch +++ b/sys-libs/zlib/files/zlib-1.2.12-fix-CC-logic-in-configure.patch @@ -1,20 +1,35 @@ -From 95b9053f10710f9f881f4daec2f2e78da87560cf Mon Sep 17 00:00:00 2001 +https://github.com/madler/zlib/pull/607 + +From c9b7587ff947638412a69df5ea820c563b8b1c1b Mon Sep 17 00:00:00 2001 From: Sam James <sam@gentoo.org> -Date: Mon, 28 Mar 2022 08:21:18 +0100 +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}. +the default setting of cc=${CC}. It was throwing away any pre-set CC value as +a result. This led to e.g. misdetection of inability to build shared libs -for say, multilib cases (w/ CC being one thing (e.g. x86_64-unknown-linux-gnu-gcc -m32 -and then 'cc' used for shared libs being w/o -m32). +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@gentoo.org> +--- + configure | 3 +++ + 1 file changed, 3 insertions(+) + +diff --git a/configure b/configure +index 52ff4a04e..3fa3e8618 100755 --- a/configure +++ b/configure -@@ -180,7 +180,10 @@ if test -z "$CC"; then +@@ -174,7 +174,10 @@ if test -z "$CC"; then else cc=${CROSS_PREFIX}cc fi @@ -25,3 +40,4 @@ Signed-off-by: Sam James <sam@gentoo.org> cflags=${CFLAGS-"-O3"} # to force the asm version use: CFLAGS="-O3 -DASMV" ./configure case "$cc" in + |