Christopher Faylor wrote: > On Mon, May 20, 2002 at 12:50:46AM -0400, Charles Wilson wrote: > >>1) Ralf's "removing unused _nm_ symbol exports" fix >>2) Danny's (or Ralf's?) export-list fix (where whole static archives can >>be marked for non-export, or DO export...helpful for convenience libs) >>3) Ralf's patch for "objdump/cygwin crashes on auto-imported libs" >> > > If Ralf and Danny will resubmit their patches against the current CVS, > I will at least incorporate them in a new version of binutils, even > if I don't check them in. Very good. Ralf? Danny? > I'd say go ahead and turn on auto-import in CVS and remove the warning, > Chuck. I've gotten approval from Nick Clifton to do this if we agree > it's a good idea. > > It's obviously a good idea. Attached. --chuck
Index: pe.em =================================================================== RCS file: /cvs/src/src/ld/emultempl/pe.em,v retrieving revision 1.59 diff -u -r1.59 pe.em --- pe.em 15 Feb 2002 02:11:05 -0000 1.59 +++ pe.em 20 May 2002 05:26:50 -0000 @@ -173,7 +173,7 @@ #ifdef DLL_SUPPORT config.dynamic_link = true; config.has_shared = 1; -/* link_info.pei386_auto_import = true; */ + link_info.pei386_auto_import = true; #if (PE_DEF_SUBSYSTEM == 9) || (PE_DEF_SUBSYSTEM == 2) #if defined TARGET_IS_mipspe || defined TARGET_IS_armpe @@ -906,7 +906,10 @@ sym = bfd_link_hash_lookup (link_info.hash, buf, 0, 0, 1); if (sym && sym->type == bfd_link_hash_defined) { - einfo (_("Warning: resolving %s by linking to %s (auto-import)\n"), + if (pe_dll_extra_pe_debug) + { + einfo (_("Warning: resolving %s by linking to %s (auto-import)\n"), + } undef->root.string, buf); { bfd *b = sym->u.def.section->owner;