From f2032b672ac5f0c570f3a6d263ce6b764e7dca93 Mon Sep 17 00:00:00 2001 From: Alan Modra Date: Tue, 19 Nov 2019 21:19:24 +1030 Subject: PR24499, ignore --add-gnu-debuglink for archives objcopy --add-gnu-debuglink=foo.a.dbg foo.a just doesn't make any sense. Who puts executables in archives? PR 24499 * objcopy.c (copy_file): Ignore --add-gnu-debuglink for archives. --- binutils/ChangeLog | 5 +++++ binutils/objcopy.c | 8 ++++++++ 2 files changed, 13 insertions(+) diff --git a/binutils/ChangeLog b/binutils/ChangeLog index 19f1c40..eed7d33 100644 --- a/binutils/ChangeLog +++ b/binutils/ChangeLog @@ -1,5 +1,10 @@ 2019-11-19 Alan Modra + PR 24499 + * objcopy.c (copy_file): Ignore --add-gnu-debuglink for archives. + +2019-11-19 Alan Modra + PR 24968 * objcopy.c (copy_object): For ELF output and non-ELF input without arch, take arch from output file if not given by -B. Don't diff --git a/binutils/objcopy.c b/binutils/objcopy.c index 99cc7be..f682fbe 100644 --- a/binutils/objcopy.c +++ b/binutils/objcopy.c @@ -3752,6 +3752,14 @@ copy_file (const char *input_filename, const char *output_filename, status = 1; return; } + + if (gnu_debuglink_filename != NULL) + { + non_fatal (_("--add-gnu-debuglink ignored for archive %s"), + bfd_get_filename (ibfd)); + gnu_debuglink_filename = NULL; + } + /* This is a no-op on non-Coff targets. */ set_long_section_mode (obfd, ibfd, long_section_names); -- cgit v1.1