My patch to the binutils strip-10.d test was wrong. The osabi field should always...

ELFOSABI_GNU for binaries containing unique symbols.  So I am reverting that patch and
instead applying the patch below to fix up the targets that were triggering the test failure.

bfd/ChangeLog
2014-01-29  Nick Clifton  <nickc@redhat.com>

	* elf32-metag.c (elf_metag_post_process_headers): Call
	_bfd_elf_post_process_headers.
	* elf32-sh64.c (sh64_elf_copy_private_data): Call
	_bfd_elf_copy_private_data.
	* elf64-sh64.c (sh_elf64_copy_private_data_internal): Likewise.

binutils/testsuite/ChangeLog
2014-01-29  Nick Clifton  <nickc@redhat.com>

	* binutils-all/strip-10.d: Revert previous delta.

diff --git a/bfd/ChangeLog b/bfd/ChangeLog
index bf0540119be7487521e0884340b1d10a098a8c5f..ec39127ed9586f1f37094378fad49369c5b871a8 100644 (file)
--- a/bfd/ChangeLog
+++ b/bfd/ChangeLog
@@ -1,3 +1,12 @@
+2014-01-29  Nick Clifton  <nickc@redhat.com>
+
+       PR binutils/16318
+       * elf32-metag.c (elf_metag_post_process_headers): Call
+       _bfd_elf_post_process_headers.
+       * elf32-sh64.c (sh64_elf_copy_private_data): Call
+       _bfd_elf_copy_private_data.
+       * elf64-sh64.c (sh_elf64_copy_private_data_internal): Likewise.
+
 2014-01-29  Nick Clifton  <nickc@redhat.com>
 
        * bfd-in.h (bfd_set_section_vma): Delete.

diff --git a/bfd/elf32-metag.c b/bfd/elf32-metag.c
index 9c11dd3d2f1a532b5639ef0572bd8e7ab0e0810e..4b85f22ba033aa47c740abab49c789ab5a13787c 100644 (file)
--- a/bfd/elf32-metag.c
+++ b/bfd/elf32-metag.c
@@ -3249,10 +3249,11 @@ elf_metag_finish_dynamic_symbol (bfd *output_bfd,
 /* Set the Meta ELF ABI version.  */
 
 static void
-elf_metag_post_process_headers (bfd * abfd, struct bfd_link_info * link_info ATTRIBUTE_UNUSED)
+elf_metag_post_process_headers (bfd * abfd, struct bfd_link_info * link_info)
 {
   Elf_Internal_Ehdr * i_ehdrp; /* ELF file header, internal form.  */
 
+  _bfd_elf_post_process_headers (abfd, link_info);
   i_ehdrp = elf_elfheader (abfd);
   i_ehdrp->e_ident[EI_ABIVERSION] = METAG_ELF_ABI_VERSION;
 }

diff --git a/bfd/elf32-sh64.c b/bfd/elf32-sh64.c
index 2848f19b488840b9992f5455ff291087ecb6686f..d1daa0deaeefbc5bd648c6ae2c7f4b5000364c8d 100644 (file)
--- a/bfd/elf32-sh64.c
+++ b/bfd/elf32-sh64.c
@@ -199,7 +199,8 @@ sh64_elf_copy_private_data (bfd * ibfd, bfd * obfd)
                  == elf_elfheader (ibfd)->e_flags));
 
   elf_elfheader (obfd)->e_flags = elf_elfheader (ibfd)->e_flags;
-  return TRUE;
+
+  return _bfd_elf_copy_private_bfd_data (ibfd, obfd);
 }
 
 static bfd_boolean

diff --git a/bfd/elf64-sh64.c b/bfd/elf64-sh64.c
index e70887bdfc67346f73d015ee9eee1f0e25989a9d..10a9396e95b3d94142362761fa61417b25a6a17f 100644 (file)
--- a/bfd/elf64-sh64.c
+++ b/bfd/elf64-sh64.c
@@ -2286,8 +2286,8 @@ sh_elf64_copy_private_data_internal (bfd *ibfd, bfd *obfd)
     }
 
   /* Copy object attributes.  */
-  _bfd_elf_copy_obj_attributes (ibfd, obfd);
-
+  _bfd_elf_copy_private_bfd_data (ibfd, obfd);
+  
   return sh_elf64_set_private_flags (obfd, elf_elfheader (ibfd)->e_flags);
 }
 

diff --git a/binutils/testsuite/ChangeLog b/binutils/testsuite/ChangeLog
index 4294f81366783d49a9ff695dd6a431a276f8e005..59db81ced228cc6ebe95e8661778286a450e873d 100644 (file)
--- a/binutils/testsuite/ChangeLog
+++ b/binutils/testsuite/ChangeLog
@@ -1,3 +1,8 @@
+2014-01-29  Nick Clifton  <nickc@redhat.com>
+
+       PR binutils/16318
+       * binutils-all/strip-10.d: Revert previous delta.
+
 2014-01-28  Nick Clifton  <nickc@redhat.com>
 
        PR binutils/16318

diff --git a/binutils/testsuite/binutils-all/strip-10.d b/binutils/testsuite/binutils-all/strip-10.d
index 949bc679c38eb6c31ec18c89eda98c56ec167960..f4cf76d2032abc36c969eead41f09afe1d9592dd 100644 (file)
--- a/binutils/testsuite/binutils-all/strip-10.d
+++ b/binutils/testsuite/binutils-all/strip-10.d
@@ -5,6 +5,6 @@
 #name: strip on STB_GNU_UNIQUE
 
 #...
-  OS/ABI:[ \t]+UNIX - (GNU|System V)
+  OS/ABI:[ \t]+UNIX - GNU
 #...
  +[0-9]+: +[0-9a-f]+ +[0-9]+ +OBJECT +(UNIQUE|<OS specific>: 10) +DEFAULT +[1-9] foo