From: "Sam James" <sam@gentoo.org>
To: gentoo-commits@lists.gentoo.org
Subject: [gentoo-commits] repo/gentoo:master commit in: sys-apps/kbd/files/, sys-apps/kbd/
Date: Fri, 6 Oct 2023 15:04:09 +0000 (UTC) [thread overview]
Message-ID: <1696604637.68508019cec4a5622ffab2825ea9e8f6cdd42a84.sam@gentoo> (raw)
commit: 68508019cec4a5622ffab2825ea9e8f6cdd42a84
Author: Sam James <sam <AT> gentoo <DOT> org>
AuthorDate: Fri Oct 6 15:03:57 2023 +0000
Commit: Sam James <sam <AT> gentoo <DOT> org>
CommitDate: Fri Oct 6 15:03:57 2023 +0000
URL: https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=68508019
sys-apps/kbd: BDEPEND on autoconf-archive
Needed for ax_check_compile_flag too.
Closes: https://bugs.gentoo.org/915257
Signed-off-by: Sam James <sam <AT> gentoo.org>
.../kbd-2.6.3-no-redefine-fortify-source.patch | 126 ---------------------
sys-apps/kbd/kbd-2.6.3-r1.ebuild | 2 +
2 files changed, 2 insertions(+), 126 deletions(-)
diff --git a/sys-apps/kbd/files/kbd-2.6.3-no-redefine-fortify-source.patch b/sys-apps/kbd/files/kbd-2.6.3-no-redefine-fortify-source.patch
index b18fb9f751cf..2edd547d1ed2 100644
--- a/sys-apps/kbd/files/kbd-2.6.3-no-redefine-fortify-source.patch
+++ b/sys-apps/kbd/files/kbd-2.6.3-no-redefine-fortify-source.patch
@@ -43,129 +43,3 @@ index 66023ff8..7fdbe7d6 100644
fi
case "$GCC,$ac_cv_prog_cc_g" in
-diff --git a/m4/ax_add_fortify_source.m4 b/m4/ax_add_fortify_source.m4
-new file mode 100644
-index 00000000..860c1598
---- /dev/null
-+++ b/m4/ax_add_fortify_source.m4
-@@ -0,0 +1,119 @@
-+# ===========================================================================
-+# https://www.gnu.org/software/autoconf-archive/ax_add_fortify_source.html
-+# ===========================================================================
-+#
-+# SYNOPSIS
-+#
-+# AX_ADD_FORTIFY_SOURCE
-+#
-+# DESCRIPTION
-+#
-+# Check whether -D_FORTIFY_SOURCE=2 can be added to CPPFLAGS without macro
-+# redefinition warnings, other cpp warnings or linker. Some distributions
-+# (such as Ubuntu or Gentoo Linux) enable _FORTIFY_SOURCE globally in
-+# their compilers, leading to unnecessary warnings in the form of
-+#
-+# <command-line>:0:0: error: "_FORTIFY_SOURCE" redefined [-Werror]
-+# <built-in>: note: this is the location of the previous definition
-+#
-+# which is a problem if -Werror is enabled. This macro checks whether
-+# _FORTIFY_SOURCE is already defined, and if not, adds -D_FORTIFY_SOURCE=2
-+# to CPPFLAGS.
-+#
-+# Newer mingw-w64 msys2 package comes with a bug in
-+# headers-git-7.0.0.5546.d200317d-1. It broke -D_FORTIFY_SOURCE support,
-+# and would need -lssp or -fstack-protector. See
-+# https://github.com/msys2/MINGW-packages/issues/5803. Try to actually
-+# link it.
-+#
-+# LICENSE
-+#
-+# Copyright (c) 2017 David Seifert <soap@gentoo.org>
-+# Copyright (c) 2019, 2023 Reini Urban <rurban@cpan.org>
-+#
-+# Copying and distribution of this file, with or without modification, are
-+# permitted in any medium without royalty provided the copyright notice
-+# and this notice are preserved. This file is offered as-is, without any
-+# warranty.
-+
-+#serial 10
-+
-+AC_DEFUN([AX_ADD_FORTIFY_SOURCE],[
-+ ac_save_cflags=$CFLAGS
-+ ac_cwerror_flag=yes
-+ AX_CHECK_COMPILE_FLAG([-Werror],[CFLAGS="$CFLAGS -Werror"])
-+ ax_add_fortify_3_failed=
-+ AC_MSG_CHECKING([whether to add -D_FORTIFY_SOURCE=3 to CPPFLAGS])
-+ AC_LINK_IFELSE([
-+ AC_LANG_PROGRAM([],
-+ [[
-+ #ifndef _FORTIFY_SOURCE
-+ return 0;
-+ #else
-+ _FORTIFY_SOURCE_already_defined;
-+ #endif
-+ ]]
-+ )],
-+ AC_LINK_IFELSE([
-+ AC_LANG_SOURCE([[
-+ #define _FORTIFY_SOURCE 3
-+ #include <string.h>
-+ int main(void) {
-+ char *s = " ";
-+ strcpy(s, "x");
-+ return strlen(s)-1;
-+ }
-+ ]]
-+ )],
-+ [
-+ AC_MSG_RESULT([yes])
-+ CFLAGS=$ac_save_cflags
-+ CPPFLAGS="$CPPFLAGS -D_FORTIFY_SOURCE=3"
-+ ], [
-+ AC_MSG_RESULT([no])
-+ ax_add_fortify_3_failed=1
-+ ],
-+ ),
-+ [
-+ AC_MSG_RESULT([no])
-+ ax_add_fortify_3_failed=1
-+ ])
-+ if test -n "$ax_add_fortify_3_failed"
-+ then
-+ AC_MSG_CHECKING([whether to add -D_FORTIFY_SOURCE=2 to CPPFLAGS])
-+ AC_LINK_IFELSE([
-+ AC_LANG_PROGRAM([],
-+ [[
-+ #ifndef _FORTIFY_SOURCE
-+ return 0;
-+ #else
-+ _FORTIFY_SOURCE_already_defined;
-+ #endif
-+ ]]
-+ )],
-+ AC_LINK_IFELSE([
-+ AC_LANG_SOURCE([[
-+ #define _FORTIFY_SOURCE 2
-+ #include <string.h>
-+ int main(void) {
-+ char *s = " ";
-+ strcpy(s, "x");
-+ return strlen(s)-1;
-+ }
-+ ]]
-+ )],
-+ [
-+ AC_MSG_RESULT([yes])
-+ CFLAGS=$ac_save_cflags
-+ CPPFLAGS="$CPPFLAGS -D_FORTIFY_SOURCE=2"
-+ ], [
-+ AC_MSG_RESULT([no])
-+ CFLAGS=$ac_save_cflags
-+ ],
-+ ),
-+ [
-+ AC_MSG_RESULT([no])
-+ CFLAGS=$ac_save_cflags
-+ ])
-+ fi
-+])
-
diff --git a/sys-apps/kbd/kbd-2.6.3-r1.ebuild b/sys-apps/kbd/kbd-2.6.3-r1.ebuild
index 8109218a18e0..0bacea6493e4 100644
--- a/sys-apps/kbd/kbd-2.6.3-r1.ebuild
+++ b/sys-apps/kbd/kbd-2.6.3-r1.ebuild
@@ -42,7 +42,9 @@ RDEPEND="
${DEPEND}
selinux? ( sec-policy/selinux-loadkeys )
"
+# autoconf-archive for F_S patch
BDEPEND="
+ sys-devel/autoconf-archive
virtual/pkgconfig
test? ( dev-libs/check )
"
next reply other threads:[~2023-10-06 15:04 UTC|newest]
Thread overview: 9+ messages / expand[flat|nested] mbox.gz Atom feed top
2023-10-06 15:04 Sam James [this message]
-- strict thread matches above, loose matches on Subject: below --
2023-12-28 4:04 [gentoo-commits] repo/gentoo:master commit in: sys-apps/kbd/files/, sys-apps/kbd/ Sam James
2020-12-16 17:46 Lars Wendler
2019-08-14 6:21 Lars Wendler
2019-07-31 16:18 Lars Wendler
2019-07-27 10:40 Lars Wendler
2018-04-20 11:13 Lars Wendler
2018-01-26 1:17 Mikle Kolyada
2017-01-09 18:33 Lars Wendler
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=1696604637.68508019cec4a5622ffab2825ea9e8f6cdd42a84.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