diff options
| author |   Sam James <sam@gentoo.org> | 2024-04-15 07:17:43 +0100 |
|---|---|---|
| committer | Sam James <sam@gentoo.org> | 2024-04-15 07:17:43 +0100 |
| commit | 9d08565368ff450c17a27bd0394a814d0aa928e7 (patch) | |
| tree | 07f7aa53cb24a0d349cad803b1baf9b819289fd3 | |
| parent | patches/openmp: new patch to allow -fopenmp=* (diff) | |
| download | elt-patches-9d08565368ff450c17a27bd0394a814d0aa928e7.tar.gz elt-patches-9d08565368ff450c17a27bd0394a814d0aa928e7.tar.bz2 elt-patches-9d08565368ff450c17a27bd0394a814d0aa928e7.zip | |
patches/verbose-pic: new patch to not suppress compiler output (!)20240415
Bug: https://bugs.gentoo.org/135865
Signed-off-by: Sam James <sam@gentoo.org>
| -rw-r--r-- | eltpatch.in | 2 | ||||
| -rw-r--r-- | patches/verbose-pic/2.4.7 | 39 |
2 files changed, 40 insertions, 1 deletions
diff --git a/eltpatch.in b/eltpatch.in index f41563a..8c6daee 100644 --- a/eltpatch.in +++ b/eltpatch.in @@ -129,7 +129,7 @@ elibtoolize() { local force="false" local elt_patches=" install-sh ltmain portage relink max_cmd_len sed test tmp cross as-needed target-nm ppc64le - g-debug specs fsanitize fuse-ld static-flags werror cxx-pthread color-record sed-quote openmp + g-debug specs fsanitize fuse-ld static-flags werror cxx-pthread color-record sed-quote openmp verbose-pic " for x in "$@" ; do diff --git a/patches/verbose-pic/2.4.7 b/patches/verbose-pic/2.4.7 new file mode 100644 index 0000000..3dfd34f --- /dev/null +++ b/patches/verbose-pic/2.4.7 @@ -0,0 +1,39 @@ +From ebe94942061d632ca2a6fa86cdebdf72c7863c07 Mon Sep 17 00:00:00 2001 +From: Sam James <sam@gentoo.org> +Date: Mon, 15 Apr 2024 07:15:15 +0100 +Subject: [PATCH] ltmain.in: don't suppress output for PIC compilations + +I just hit this in a fascinating place. + +When working on xz, I set `-Werror=suggest-attribute=returns_nonnull`, and +the build failed (as I expected it to), but with no visible error from +the compiler. + +There's a mysterious '>/dev/null 2>&1' on the second line where +liblzma_la-common.o is built without PIC. + +With -fPIC, IPA doesn't end up doing attribute discovery. Without it, +it does. This behaviour is insane and we should patch it out. + +Bug: https://bugs.gentoo.org/135865 +Signed-off-by: Sam James <sam@gentoo.org> +--- + build-aux/ltmain.in | 2 -- + 1 file changed, 2 deletions(-) + +diff --git a/build-aux/ltmain.in b/build-aux/ltmain.in +index 090a4fe8..e383e4cb 100644 +--- a/build-aux/ltmain.in ++++ b/build-aux/ltmain.in +@@ -1665,8 +1665,6 @@ compiler." + func_append command " -o $obj" + fi + +- # Suppress compiler output if we already did a PIC compilation. +- func_append command "$suppress_output" + func_show_eval_locale "$command" \ + '$opt_dry_run || $RM $removelist; exit $EXIT_FAILURE' + +-- +2.44.0 + |