[Buildroot] [PATCH] binutils: Get rid of obsolete patches

Romain Naour romain.naour at gmail.com
Sat Oct 27 07:46:33 UTC 2018


Hi Alexey,

Thanks for doing this :)

Le 21/10/2018 à 20:18, Alexey Brodkin a écrit :
> Some patches for Binutils seems to not be really needed.
> Moreover for some of them I was not able to find any justification
> for their introduction.

Those patches are annoying while bumping the binutils version, I never figured
out why there are in Buildroot.

> 
> 1. 0002-ld-makefile.patch
> 
>    This one I may track down to at least 2.15.92.0.2 here [1].
>    Commit message just says: "Integrate some debian patches,
>    extracted from binutils_2.15-4.diff"
> 
>    But in OpenEmbedded they got rid of it long ago on switch to 2.26, see [2].
> 
> 2. 0003-check-ldrunpath-length.patch
> 
>    The same story here.
> 
> 3. 0004-add-sysroot-fix-from-bug-3049.patch
> 
>    Bug mentioned in this patch was marked as "fixed" in 2012, see [3].

Based on the commit date, this patch is part of Binutils 2.23 release.

Reviewed-by: Romain Naour <romain.naour at gmail.com>

Best regards,
Romain


> 
> [1] https://git.buildroot.org/buildroot/commit/?id=ce2722a8f2abee5b60862bc896333b77a7502c82
> [2] https://github.com/openembedded/openembedded-core/commit/3c7fe424f850af70989ad682dd9c5be70cd02ca5
> [3]https://sourceware.org/bugzilla/show_bug.cgi?id=10340
> 
> Signed-off-by: Alexey Brodkin <abrodkin at synopsys.com>
> Cc: Thomas Petazzoni <thomas.petazzoni at bootlin.com>
> Cc: Yann E. MORIN <yann.morin.1998 at free.fr>
> Cc: Peter Korsgaard <peter at korsgaard.com>
> Cc: Romain Naour <romain.naour at smile.fr>
> ---
>  package/binutils/2.31.1/0002-ld-makefile.patch     | 41 -----------------
>  .../2.31.1/0003-check-ldrunpath-length.patch       | 36 ---------------
>  .../0004-add-sysroot-fix-from-bug-3049.patch       | 51 ----------------------
>  3 files changed, 128 deletions(-)
>  delete mode 100644 package/binutils/2.31.1/0002-ld-makefile.patch
>  delete mode 100644 package/binutils/2.31.1/0003-check-ldrunpath-length.patch
>  delete mode 100644 package/binutils/2.31.1/0004-add-sysroot-fix-from-bug-3049.patch
> 
> diff --git a/package/binutils/2.31.1/0002-ld-makefile.patch b/package/binutils/2.31.1/0002-ld-makefile.patch
> deleted file mode 100644
> index 6893d65aa0..0000000000
> --- a/package/binutils/2.31.1/0002-ld-makefile.patch
> +++ /dev/null
> @@ -1,41 +0,0 @@
> -From ae435bc27e1eb59e4ad571a37a144bf99dc68f55 Mon Sep 17 00:00:00 2001
> -From: Romain Naour <romain.naour at gmail.com>
> -Date: Fri, 25 Dec 2015 11:40:53 +0100
> -Subject: [PATCH] ld-makefile
> -
> -[Romain: rebase on top of 2.26]
> -Signed-off-by: Romain Naour <romain.naour at gmail.com>
> ----
> - ld/Makefile.am | 2 +-
> - ld/Makefile.in | 2 +-
> - 2 files changed, 2 insertions(+), 2 deletions(-)
> -
> -diff --git a/ld/Makefile.am b/ld/Makefile.am
> -index d86ad0940c9..c95b0ef0252 100644
> ---- a/ld/Makefile.am
> -+++ b/ld/Makefile.am
> -@@ -57,7 +57,7 @@ endif
> - # We put the scripts in the directory $(scriptdir)/ldscripts.
> - # We can't put the scripts in $(datadir) because the SEARCH_DIR
> - # directives need to be different for native and cross linkers.
> --scriptdir = $(tooldir)/lib
> -+scriptdir = $(libdir)
> - 
> - EMUL = @EMUL@
> - EMULATION_OFILES = @EMULATION_OFILES@
> -diff --git a/ld/Makefile.in b/ld/Makefile.in
> -index 4792b2b013c..789df2d01b9 100644
> ---- a/ld/Makefile.in
> -+++ b/ld/Makefile.in
> -@@ -563,7 +563,7 @@ AM_CFLAGS = $(WARN_CFLAGS) $(ELF_CLFAGS)
> - # We put the scripts in the directory $(scriptdir)/ldscripts.
> - # We can't put the scripts in $(datadir) because the SEARCH_DIR
> - # directives need to be different for native and cross linkers.
> --scriptdir = $(tooldir)/lib
> -+scriptdir = $(libdir)
> - BASEDIR = $(srcdir)/..
> - BFDDIR = $(BASEDIR)/bfd
> - INCDIR = $(BASEDIR)/include
> --- 
> -2.14.4
> -
> diff --git a/package/binutils/2.31.1/0003-check-ldrunpath-length.patch b/package/binutils/2.31.1/0003-check-ldrunpath-length.patch
> deleted file mode 100644
> index 6cdf085422..0000000000
> --- a/package/binutils/2.31.1/0003-check-ldrunpath-length.patch
> +++ /dev/null
> @@ -1,36 +0,0 @@
> -From a216bfcd91363a8e8c14db320a2870fff2985d78 Mon Sep 17 00:00:00 2001
> -From: Romain Naour <romain.naour at gmail.com>
> -Date: Fri, 25 Dec 2015 11:41:47 +0100
> -Subject: [PATCH] check-ldrunpath-length
> -
> -[Romain: rebase on top of 2.26]
> -Signed-off-by: Romain Naour <romain.naour at gmail.com>
> ----
> - ld/emultempl/elf32.em | 4 ++++
> - 1 file changed, 4 insertions(+)
> -
> -diff --git a/ld/emultempl/elf32.em b/ld/emultempl/elf32.em
> -index 1ad9d6b6fa8..637a5d5d585 100644
> ---- a/ld/emultempl/elf32.em
> -+++ b/ld/emultempl/elf32.em
> -@@ -1471,6 +1471,8 @@ fragment <<EOF
> - 	      && command_line.rpath == NULL)
> - 	    {
> - 	      path = (const char *) getenv ("LD_RUN_PATH");
> -+	      if ((path) && (strlen (path) == 0))
> -+	        path = NULL;
> - 	      if (path
> - 		  && gld${EMULATION_NAME}_search_needed (path, &n, force))
> - 		break;
> -@@ -1751,6 +1753,8 @@ gld${EMULATION_NAME}_before_allocation (void)
> -   rpath = command_line.rpath;
> -   if (rpath == NULL)
> -     rpath = (const char *) getenv ("LD_RUN_PATH");
> -+  if ((rpath) && (strlen (rpath) == 0))
> -+    rpath = NULL;
> - 
> -   for (abfd = link_info.input_bfds; abfd; abfd = abfd->link.next)
> -     if (bfd_get_flavour (abfd) == bfd_target_elf_flavour)
> --- 
> -2.14.4
> -
> diff --git a/package/binutils/2.31.1/0004-add-sysroot-fix-from-bug-3049.patch b/package/binutils/2.31.1/0004-add-sysroot-fix-from-bug-3049.patch
> deleted file mode 100644
> index 1ef1385062..0000000000
> --- a/package/binutils/2.31.1/0004-add-sysroot-fix-from-bug-3049.patch
> +++ /dev/null
> @@ -1,51 +0,0 @@
> -From 73ecf6f6a1f327c9fa3af1fc924d152321aac801 Mon Sep 17 00:00:00 2001
> -From: Romain Naour <romain.naour at gmail.com>
> -Date: Fri, 25 Dec 2015 11:42:48 +0100
> -Subject: [PATCH] add sysroot fix from bug #3049
> -
> -Always try to prepend the sysroot prefix to absolute filenames first.
> -
> -http://bugs.gentoo.org/275666
> -http://sourceware.org/bugzilla/show_bug.cgi?id=10340
> -
> -Signed-off-by: Sven Rebhan <odinshorse at googlemail.com>
> -[Romain: rebase on top of 2.26]
> -Signed-off-by: Romain Naour <romain.naour at gmail.com>
> ----
> - ld/ldfile.c | 10 ++++++++--
> - 1 file changed, 8 insertions(+), 2 deletions(-)
> -
> -diff --git a/ld/ldfile.c b/ld/ldfile.c
> -index a72ff135264..b3d166cbd60 100644
> ---- a/ld/ldfile.c
> -+++ b/ld/ldfile.c
> -@@ -338,18 +338,24 @@ ldfile_open_file_search (const char *arch,
> -      directory first.  */
> -   if (!entry->flags.maybe_archive)
> -     {
> --      if (entry->flags.sysrooted && IS_ABSOLUTE_PATH (entry->filename))
> -+     /* For absolute pathnames, try to always open the file in the
> -+	 sysroot first. If this fails, try to open the file at the
> -+	 given location.  */
> -+     entry->flags.sysrooted = is_sysrooted_pathname (entry->filename);
> -+     if (!entry->flags.sysrooted && IS_ABSOLUTE_PATH (entry->filename)
> -+     && ld_sysroot)
> - 	{
> - 	  char *name = concat (ld_sysroot, entry->filename,
> - 			       (const char *) NULL);
> - 	  if (ldfile_try_open_bfd (name, entry))
> - 	    {
> - 	      entry->filename = name;
> -+	      entry->flags.sysrooted = TRUE;
> - 	      return TRUE;
> - 	    }
> - 	  free (name);
> - 	}
> --      else if (ldfile_try_open_bfd (entry->filename, entry))
> -+      if (ldfile_try_open_bfd (entry->filename, entry))
> - 	return TRUE;
> - 
> -       if (IS_ABSOLUTE_PATH (entry->filename))
> --- 
> -2.14.4
> -
> 




More information about the buildroot mailing list