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: 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 )
 "


             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