[gentoo-commits] repo/gentoo:master commit in: app-shells/ksh/

public inbox for gentoo-commits@lists.gentoo.org
 help / color / mirror / Atom feed

From: "Sam James" <sam@gentoo.org>
To: gentoo-commits@lists.gentoo.org
Subject: [gentoo-commits] repo/gentoo:master commit in: app-shells/ksh/
Date: Tue,  9 Jan 2024 02:40:20 +0000 (UTC)	[thread overview]
Message-ID: <1704767978.10cd51b1cac70e3f180a04b2e9ea3c6eb46150fd.sam@gentoo> (raw)

commit:     10cd51b1cac70e3f180a04b2e9ea3c6eb46150fd
Author:     Sam James <sam <AT> gentoo <DOT> org>
AuthorDate: Tue Jan  9 02:39:38 2024 +0000
Commit:     Sam James <sam <AT> gentoo <DOT> org>
CommitDate: Tue Jan  9 02:39:38 2024 +0000
URL:        https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=10cd51b1

app-shells/ksh: clean up flag handling

* Filter LTO instead of just -Wno-erroring things indicating it's unsafe
* Try to respect CC/AR/NM more

Signed-off-by: Sam James <sam <AT> gentoo.org>

 app-shells/ksh/ksh-1.0.8.ebuild |  9 ++++-----
 app-shells/ksh/ksh-9999.ebuild  | 11 +++++------
 2 files changed, 9 insertions(+), 11 deletions(-)

diff --git a/app-shells/ksh/ksh-1.0.8.ebuild b/app-shells/ksh/ksh-1.0.8.ebuild
index 4cda30c54705..c97a769209a6 100644
--- a/app-shells/ksh/ksh-1.0.8.ebuild
+++ b/app-shells/ksh/ksh-1.0.8.ebuild
@@ -40,18 +40,17 @@ src_compile() {
 		"-Wno-unused-but-set-variable"
 		"-Wno-cpp"
 		"-Wno-maybe-uninitialized"
-		"-Wno-lto-type-mismatch"
-		"-Wno-error=int-conversion"
-		"-Wno-int-conversion"
 		"-P"
 	)
 	append-cflags $(test-flags-CC ${extraflags[@]})
+	append-cflags -fno-strict-aliasing
 	filter-flags '-fdiagnostics-color=always' # https://github.com/ksh93/ksh/issues/379
-	export CCFLAGS="${CFLAGS} -fno-strict-aliasing"
+	filter-lto
 
+	export CCFLAGS="${CFLAGS}"
 	tc-export AR CC LD NM
 
-	sh bin/package make SHELL="${BROOT}"/bin/sh || die
+	sh bin/package make AR="${AR}" CC="${CC}" NM="${NM}" SHELL="${BROOT}"/bin/sh || die
 }
 
 src_test() {

diff --git a/app-shells/ksh/ksh-9999.ebuild b/app-shells/ksh/ksh-9999.ebuild
index f59df92a2e4a..c97a769209a6 100644
--- a/app-shells/ksh/ksh-9999.ebuild
+++ b/app-shells/ksh/ksh-9999.ebuild
@@ -1,4 +1,4 @@
-# Copyright 2021-2022 Gentoo Authors
+# Copyright 2021-2024 Gentoo Authors
 # Distributed under the terms of the GNU General Public License v2
 
 EAPI=8
@@ -40,18 +40,17 @@ src_compile() {
 		"-Wno-unused-but-set-variable"
 		"-Wno-cpp"
 		"-Wno-maybe-uninitialized"
-		"-Wno-lto-type-mismatch"
-		"-Wno-error=int-conversion"
-		"-Wno-int-conversion"
 		"-P"
 	)
 	append-cflags $(test-flags-CC ${extraflags[@]})
+	append-cflags -fno-strict-aliasing
 	filter-flags '-fdiagnostics-color=always' # https://github.com/ksh93/ksh/issues/379
-	export CCFLAGS="${CFLAGS} -fno-strict-aliasing"
+	filter-lto
 
+	export CCFLAGS="${CFLAGS}"
 	tc-export AR CC LD NM
 
-	sh bin/package make SHELL="${BROOT}"/bin/sh || die
+	sh bin/package make AR="${AR}" CC="${CC}" NM="${NM}" SHELL="${BROOT}"/bin/sh || die
 }
 
 src_test() {

next             reply	other threads:[~2024-01-09  2:40 UTC|newest]

Thread overview: 68+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2024-01-09  2:40 Sam James [this message]
  -- strict thread matches above, loose matches on Subject: below --
2025-07-13  9:13 [gentoo-commits] repo/gentoo:master commit in: app-shells/ksh/ Sam James
2025-07-13  8:58 Sam James
2025-04-26  2:04 Sam James
2025-04-26  2:04 Sam James
2025-04-26  1:24 Sam James
2025-04-26  1:17 Sam James
2024-08-11 11:29 Sam James
2024-07-03  5:38 Joonas Niilola
2024-07-03  2:33 Sam James
2024-07-03  2:10 Sam James
2024-07-03  2:10 Sam James
2024-07-03  2:10 Sam James
2024-07-03  0:54 Sam James
2024-07-03  0:54 Sam James
2024-01-09  1:02 Sam James
2023-09-18  9:50 Sam James
2022-12-27 19:53 Sam James
2022-12-27 19:45 Sam James
2022-12-27 13:40 Sam James
2022-12-27 11:31 Sam James
2022-12-02 18:44 WANG Xuerui
2022-11-23  0:27 Georgy Yakovlev
2022-11-23  0:27 Georgy Yakovlev
2022-09-19  6:27 Georgy Yakovlev
2022-09-16 20:57 Georgy Yakovlev
2022-09-16 20:57 Georgy Yakovlev
2022-09-16 20:57 Georgy Yakovlev
2022-02-16  7:01 Matt Turner
2021-12-27 20:08 Arthur Zamarin
2021-12-21 11:03 James Le Cuirot
2021-12-18  3:08 Georgy Yakovlev
2021-12-18  2:10 Sam James
2021-12-17 15:28 Stephan Hartmann
2021-12-15 16:53 Mike Gilbert
2021-12-15 15:48 Arthur Zamarin
2021-12-15  8:23 Arthur Zamarin
2021-12-15  7:57 Arthur Zamarin
2021-12-15  7:13 Georgy Yakovlev
2021-12-15  7:07 Georgy Yakovlev
2021-12-15  6:55 Georgy Yakovlev
2021-12-15  6:45 Georgy Yakovlev
2021-12-15  6:45 Georgy Yakovlev
2021-12-15  6:45 Georgy Yakovlev
2021-12-15  6:45 Georgy Yakovlev
2021-12-15  5:32 Georgy Yakovlev
2021-12-15  5:32 Georgy Yakovlev
2021-12-15  5:32 Georgy Yakovlev
2021-09-05 19:01 Mike Gilbert
2020-12-01 16:13 Mike Gilbert
2019-12-11 20:39 Mike Gilbert
2019-09-17  2:19 Mike Gilbert
2019-09-01 16:24 Mike Gilbert
2019-09-01 14:32 Mike Gilbert
2019-05-28  4:01 Mike Gilbert
2019-05-04 16:03 Mike Gilbert
2019-05-04 16:03 Mike Gilbert
2018-04-01 20:10 Mike Gilbert
2018-02-15 17:20 Mike Gilbert
2018-02-14 20:07 Mike Gilbert
2018-01-29 23:10 Mike Gilbert
2017-01-02  8:00 Justin Lecher
2016-12-30 11:12 Agostino Sarubbo
2016-12-30  9:39 Agostino Sarubbo
2016-09-05  0:33 Mike Gilbert
2016-05-31 15:05 Tobias Klausmann
2016-05-13  8:41 Agostino Sarubbo
2016-04-26 17:35 Markus Meier

find likely ancestor, descendant, or conflicting patches for this message:
( dfblob:4cda30c5470 dfblob:c97a769209a dfblob:f59df92a2e4
dfblob:c97a769209a )
 OR (
bs:"[gentoo-commits] repo/gentoo:master commit in: app-shells/ksh/" )
	(help)

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=1704767978.10cd51b1cac70e3f180a04b2e9ea3c6eb46150fd.sam@gentoo \
    --to=sam@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