[pkgsrc/trunk]: pkgsrc/devel/binutils binutils: remove now unneeded patch dro...

[fcambus <fcambus%pkgsrc.org@localhost>]

details:   https://anonhg.NetBSD.org/pkgsrc/rev/3d19a142b20c
branches:  trunk
changeset: 443378:3d19a142b20c
user:      fcambus <fcambus%pkgsrc.org@localhost>
date:      Fri Dec 11 10:02:16 2020 +0000

description:
binutils: remove now unneeded patch dropping hidden symbols warning.

Issue has been resolved upstream in binutils 2.26, more details can be
found here: https://sourceware.org/bugzilla/show_bug.cgi?id=15574

diffstat:

 devel/binutils/Makefile                      |   4 ++--
 devel/binutils/distinfo                      |   3 +--
 devel/binutils/patches/patch-gold_resolve.cc |  22 ----------------------
 3 files changed, 3 insertions(+), 26 deletions(-)

diffs (56 lines):

diff -r 4cb8cccdc68d -r 3d19a142b20c devel/binutils/Makefile
--- a/devel/binutils/Makefile   Fri Dec 11 09:49:47 2020 +0000
+++ b/devel/binutils/Makefile   Fri Dec 11 10:02:16 2020 +0000
@@ -1,7 +1,7 @@
-# $NetBSD: Makefile,v 1.88 2020/12/10 22:26:49 fcambus Exp $
+# $NetBSD: Makefile,v 1.89 2020/12/11 10:02:16 fcambus Exp $
 
 DISTNAME=      binutils-2.35.1
-PKGREVISION=   1
+PKGREVISION=   2
 CATEGORIES=    devel
 MASTER_SITES=  ${MASTER_SITE_GNU:=binutils/}
 EXTRACT_SUFX=  .tar.bz2
diff -r 4cb8cccdc68d -r 3d19a142b20c devel/binutils/distinfo
--- a/devel/binutils/distinfo   Fri Dec 11 09:49:47 2020 +0000
+++ b/devel/binutils/distinfo   Fri Dec 11 10:02:16 2020 +0000
@@ -1,4 +1,4 @@
-$NetBSD: distinfo,v 1.34 2020/12/06 18:07:53 fcambus Exp $
+$NetBSD: distinfo,v 1.35 2020/12/11 10:02:16 fcambus Exp $
 
 SHA1 (binutils-2.35.1.tar.bz2) = df4eb943bf65df4bbbd0a001efcc98113423c5dd
 RMD160 (binutils-2.35.1.tar.bz2) = ed8ad923db716388b83b0234d41abc6beb01c9d6
@@ -7,7 +7,6 @@
 SHA1 (patch-bfd_cache.c) = e2d96bad350552eacdffa83532f9dc9e15ee9be9
 SHA1 (patch-gold_Makefile.in) = e01d973f9625a1653851f796c123efec37102fbd
 SHA1 (patch-gold_options.h) = cdb27cb5c967a873535ce5c63026c6ace3538fd6
-SHA1 (patch-gold_resolve.cc) = ab64091fa2bd727993dd0428f117b658cd4dd3e3
 SHA1 (patch-gold_system.h) = 9b4130b5315763daa66e0a91a8be6d1df0d10344
 SHA1 (patch-include_safe-ctype.h) = 66e636f8200ff5e9b4bfa0a5aee13cd072a23887
 SHA1 (patch-libiberty_Makefile.in) = bca8d36e3813412d0c075eab4c194c0bc9f504f5
diff -r 4cb8cccdc68d -r 3d19a142b20c devel/binutils/patches/patch-gold_resolve.cc
--- a/devel/binutils/patches/patch-gold_resolve.cc      Fri Dec 11 09:49:47 2020 +0000
+++ /dev/null   Thu Jan 01 00:00:00 1970 +0000
@@ -1,22 +0,0 @@
-$NetBSD: patch-gold_resolve.cc,v 1.3 2020/12/06 18:07:53 fcambus Exp $
-
- The gold linker is overly pedantic for dynamic references to symbols
- with hidden visibility.  It will spew error messages about weak symbols
- in libc.so that ld.bfd does not complain about. Until the bug is resolved
- disable the hidden symbol warnings.  These warning also cause > 200
- false failures in the gcc gnat.dg testsuite.
-
- https://sourceware.org/bugzilla/show_bug.cgi?id=15574
-
---- gold/resolve.cc.orig       2020-07-24 09:12:20.000000000 +0000
-+++ gold/resolve.cc
-@@ -295,8 +295,7 @@ Symbol_table::resolve(Sized_symbol<size>
-       to->set_in_reg();
-     }
-   else if (st_shndx == elfcpp::SHN_UNDEF
--           && (to->visibility() == elfcpp::STV_HIDDEN
--               || to->visibility() == elfcpp::STV_INTERNAL))
-+           && (to->visibility() == elfcpp::STV_INTERNAL))
-     {
-       // The symbol is hidden, so a reference from a shared object
-       // cannot bind to it.  We tried issuing a warning in this case,