public inbox for gentoo-commits@lists.gentoo.org
 help / color / mirror / Atom feed
From: "Mats Lidell" <matsl@gentoo.org>
To: gentoo-commits@lists.gentoo.org
Subject: [gentoo-commits] repo/gentoo:master commit in: app-editors/xemacs/, app-editors/xemacs/files/
Date: Fri, 18 Oct 2024 20:38:17 +0000 (UTC)	[thread overview]
Message-ID: <1729283868.c390fa2cb398038e30190e64d6ed41e88db5d5ea.matsl@gentoo> (raw)

commit:     c390fa2cb398038e30190e64d6ed41e88db5d5ea
Author:     Mats Lidell <matsl <AT> gentoo <DOT> org>
AuthorDate: Fri Oct 18 20:29:31 2024 +0000
Commit:     Mats Lidell <matsl <AT> gentoo <DOT> org>
CommitDate: Fri Oct 18 20:37:48 2024 +0000
URL:        https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=c390fa2c

app-editors/xemacs: Let compiler driver handle LDFLAGS

Add upstream patch to no longer use XE_PROTECT_LINKER_FLAGS on
LDFLAGS.

Closes: https://bugs.gentoo.org/940327

Signed-off-by: Mats Lidell <matsl <AT> gentoo.org>

 .../xemacs/files/xemacs-21.5.35-linker-flags.patch | 24 ++++++++++++++++++++++
 app-editors/xemacs/xemacs-21.5.35-r4.ebuild        |  1 +
 2 files changed, 25 insertions(+)

diff --git a/app-editors/xemacs/files/xemacs-21.5.35-linker-flags.patch b/app-editors/xemacs/files/xemacs-21.5.35-linker-flags.patch
new file mode 100644
index 000000000000..4d95b56608f0
--- /dev/null
+++ b/app-editors/xemacs/files/xemacs-21.5.35-linker-flags.patch
@@ -0,0 +1,24 @@
+changeset:   7521:ce06831b299c
+parent:      7519:fb35ba719d24
+user:        Aidan Kehoe <kehoea@parhasard.net>
+date:        Thu Oct 17 09:01:54 2024 +0100
+summary:     configure.ac: No longer use XE_PROTECT_LINKER_FLAGS on LDFLAGS.
+
+diff -r fb35ba719d24 -r ce06831b299c configure.ac
+--- a/configure.ac	Mon Oct 14 19:18:08 2024 +0100
++++ b/configure.ac	Thu Oct 17 09:01:54 2024 +0100
+@@ -2155,8 +2155,12 @@
+ XE_PROTECT_LINKER_FLAGS(ld_switch_system)
+ XE_PROTECT_LINKER_FLAGS(ld_switch_machine)
+ XE_PROTECT_LINKER_FLAGS(ld_switch_site)
+-XE_PROTECT_LINKER_FLAGS(LDFLAGS)
+ XE_PROTECT_LINKER_FLAGS(ld_call_shared)
++dnl No longer do this for LDFLAGS, since that interacts poorly with link time
++dnl optimization. Unclear if XE_PROTECT_LINKER_FLAGS() is still needed in
++dnl general, it was introduced for GCC versions in the 90s and things have
++dnl changed, but unlikely to be an issue with the other flags. See
++dnl https://bugs.gentoo.org/940327 for a relevant bug report.
+ 
+ dnl Add s&m-determined objects to link line
+ test -n "$objects_machine" && XE_ADD_OBJS($objects_machine)
+

diff --git a/app-editors/xemacs/xemacs-21.5.35-r4.ebuild b/app-editors/xemacs/xemacs-21.5.35-r4.ebuild
index 7e74432b1405..766ac26ff39b 100644
--- a/app-editors/xemacs/xemacs-21.5.35-r4.ebuild
+++ b/app-editors/xemacs/xemacs-21.5.35-r4.ebuild
@@ -68,6 +68,7 @@ src_prepare() {
 	eapply "${FILESDIR}/${P}-which.patch"
 	eapply "${FILESDIR}/${P}-misalignment.patch"
 	eapply "${FILESDIR}/${P}-va_args.patch"
+	eapply "${FILESDIR}/${P}-linker-flags.patch"
 
 	eapply_user
 


             reply	other threads:[~2024-10-18 20:38 UTC|newest]

Thread overview: 14+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2024-10-18 20:38 Mats Lidell [this message]
  -- strict thread matches above, loose matches on Subject: below --
2025-02-19 18:58 [gentoo-commits] repo/gentoo:master commit in: app-editors/xemacs/, app-editors/xemacs/files/ Mats Lidell
2025-01-22 16:01 Mats Lidell
2024-11-01 22:48 Mats Lidell
2024-09-28 20:28 Mats Lidell
2024-09-26 16:04 Mats Lidell
2024-03-09 15:27 Mats Lidell
2022-06-16 21:16 Mats Lidell
2020-08-30 18:11 Mats Lidell
2020-01-12 19:15 Mats Lidell
2020-01-11 16:50 Mats Lidell
2019-07-09  8:25 Mats Lidell
2017-03-31 20:02 Mats Lidell
2016-04-03  0:04 Mats Lidell

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=1729283868.c390fa2cb398038e30190e64d6ed41e88db5d5ea.matsl@gentoo \
    --to=matsl@gentoo.org \
    --cc=gentoo-commits@lists.gentoo.org \
    --cc=gentoo-dev@lists.gentoo.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox