From: "H.J. Lu" <hjl.tools@gmail.com>
To: Kai Tietz <ktietz70@googlemail.com>
Cc: Nick Clifton <nickc@redhat.com>,
Binutils <binutils@sourceware.org>,
Dave Korn <dave.korn.cygwin@gmail.com>
Subject: Re: [patch ld]: Let binutils understand DOS-paths (3 of 4)
Date: Tue, 01 Mar 2011 22:59:00 -0000 [thread overview]
Message-ID: <AANLkTinhf8hmdTGX-iWBFj=4ULP5+kZ938mNakWXbkj1@mail.gmail.com> (raw)
In-Reply-To: <AANLkTikvOYhRLdBHdpww9vDAsa0BH-MaTbGJNO0qKmgc@mail.gmail.com>
On Tue, Mar 1, 2011 at 2:42 PM, H.J. Lu <hjl.tools@gmail.com> wrote:
> On Mon, Feb 28, 2011 at 10:39 AM, Kai Tietz <ktietz70@googlemail.com> wrote:
>> 2011/2/28 Nick Clifton <nickc@redhat.com>:
>>> Hi Kai,
>>>
>>>> ChangeLog ld/
>>>> 2011-02-27 Kai Tietz
>>>>
>>>> * emultempl/beos.em (sort_by_file_name): Use filename_(n)cmp.
>>>> * emultempl/elf32.em (gld${EMULATION_NAME}_vercheck):
>>>> Likewise.
>>>> (gld${EMULATION_NAME}_stat_needed): Likewise.
>>>> (gld${EMULATION_NAME}_check_needed): Likewise.
>>>> * emultempl/pe.em (gld_${EMULATION_NAME}_after_open): Likewise.
>>>> (gld_${EMULATION_NAME}_unrecognized_file): Likewise.
>>>> * emultempl/pep.em (gld_${EMULATION_NAME}_after_open): Likewise.
>>>> (gld_${EMULATION_NAME}_unrecognized_file): Likewise.
>>>> * ldfile.c (ldfile_open_file): Likewise.
>>>> * ldlang.c (wild_sort): Likewise.
>>>> (lookup_name): Likewise.
>>>> (check_excluded_libs): Likewise.
>>>> * ldmisc.c (vfinfo): Likewise.
>>>> * pe-dll.c (libnamencmp): Likewise.
>>>> (auto_export): Likewise.
>>>> (pe_dll_generate_implib): Likewise.
>>>> * testplug.c (onclaim_file): Likewise.
>>>
>>> Approved - please apply.
>>>
>>> Cheers
>>> Nick
>>>
>
> This caused:
>
> http://www.sourceware.org/bugzilla/show_bug.cgi?id=12529
>
Linker failed to load libldtestplug.so since filename_cmp is
undefined. I checked in this patch to revert it.
Please make sure that "make check" works on Linux/x86
before making changes like this.
Thanks.
--
H.J.
---
diff --git a/ld/ChangeLog b/ld/ChangeLog
index 8ed86f3..4f5f2c8 100644
--- a/ld/ChangeLog
+++ b/ld/ChangeLog
@@ -1,3 +1,9 @@
+2011-03-01 H.J. Lu <hongjiu.lu@intel.com>
+
+ PR ld/12529
+ * testplug.c: Don't include "filenames.h".
+ (onclaim_file): Don't use filename_cmp.
+
2011-02-28 Kai Tietz <kai.tietz@onevision.com>
* emultempl/beos.em (sort_by_file_name): Use filename_(n)cmp.
diff --git a/ld/testplug.c b/ld/testplug.c
index 9e1561c..f04df10 100644
--- a/ld/testplug.c
+++ b/ld/testplug.c
@@ -23,7 +23,6 @@
#include "plugin-api.h"
/* For ARRAY_SIZE macro only - we don't link the library itself. */
#include "libiberty.h"
-#include "filenames.h"
extern enum ld_plugin_status onload (struct ld_plugin_tv *tv);
static enum ld_plugin_status onclaim_file (const struct
ld_plugin_input_file *file,
@@ -520,7 +519,7 @@ onclaim_file (const struct ld_plugin_input_file
*file, int *claimed)
claim_file_t *claimfile = claimfiles_list;
while (claimfile)
{
- if (!filename_cmp (file->name, claimfile->file.name))
+ if (!strcmp (file->name, claimfile->file.name))
break;
claimfile = claimfile->next;
}
next prev parent reply other threads:[~2011-03-01 22:59 UTC|newest]
Thread overview: 6+ messages / expand[flat|nested] mbox.gz Atom feed top
2011-02-28 8:00 Kai Tietz
2011-02-28 13:41 ` Nick Clifton
2011-02-28 18:39 ` Kai Tietz
2011-03-01 22:42 ` H.J. Lu
2011-03-01 22:59 ` H.J. Lu [this message]
2011-03-01 23:16 ` DJ Delorie
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='AANLkTinhf8hmdTGX-iWBFj=4ULP5+kZ938mNakWXbkj1@mail.gmail.com' \
--to=hjl.tools@gmail.com \
--cc=binutils@sourceware.org \
--cc=dave.korn.cygwin@gmail.com \
--cc=ktietz70@googlemail.com \
--cc=nickc@redhat.com \
/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;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).