From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 18279 invoked by alias); 4 Jul 2002 07:21:54 -0000 Mailing-List: contact binutils-help@sources.redhat.com; run by ezmlm Precedence: bulk List-Subscribe: List-Archive: List-Post: List-Help: , Sender: binutils-owner@sources.redhat.com Received: (qmail 18231 invoked from network); 4 Jul 2002 07:21:52 -0000 Received: from unknown (HELO ns.logos-m.ru) (195.7.187.66) by sources.redhat.com with SMTP; 4 Jul 2002 07:21:52 -0000 Received: from gibe.logos-m.ru (gibe.logos-m.ru [195.7.187.101]) by ns.logos-m.ru (8.12.1/8.12.1) with ESMTP id g647d5Qe018545; Thu, 4 Jul 2002 11:39:05 +0400 (MSD) Date: Thu, 04 Jul 2002 03:43:00 -0000 From: egor duda Reply-To: egor duda Organization: deo X-Priority: 3 (Normal) Message-ID: <553442059.20020704111927@logos-m.ru> To: Charles Wilson CC: binutils@sources.redhat.com, cygwin-apps@sources.redhat.com Subject: Re: --enable-auto-import extension In-Reply-To: <3D234258.9050008@ece.gatech.edu> References: <1212929671.20020628141818@logos-m.ru> <3D1D08A1.9070505@ece.gatech.edu> <180259441557.20020701104656@logos-m.ru> <3D20C981.8020407@ece.gatech.edu> <903891375.20020702193614@logos-m.ru> <3D234258.9050008@ece.gatech.edu> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Transfer-Encoding: 7bit X-SW-Source: 2002-07/txt/msg00090.txt.bz2 Hi! Wednesday, 03 July, 2002 Charles Wilson cwilson@ece.gatech.edu wrote: CW> The new version looks good to me; I built and ran your test without CW> problems. I do have a suggestion for later, when CW> --enable-runtime-pseudo-reloc is made the default: in pe-dll.c (around CW> line 2209) change CW> if (pe_dll_extra_pe_debug) CW> printf ("creating runtime pseudo-reloc entry for %s (addend=%d)\n", CW> fixup_name, addend); CW> to CW> if (link_info.pei386_runtime_pseudo_reloc == -1) CW> info_msg (_("creating runtime pseudo-reloc entry for %s (addend=%d)\n"), CW> fixup_name, addend); CW> So that if pseudo_reloc is implicitly enabled, print messages for each CW> psuedo-reloc entry (but not all that other pe_dll_extra_pe_debug stuff). CW> It's not a warning, but its info the user probably needs to know if he CW> didn't explicitly say "--enable-runtime-pseudo-reloc". Agreed. CW> The business with fork() and -DNO_RUNTIME_PSEUDO_RELOC_SUPPORT stuff in CW> your example is a bit confusing -- do you have some pending CW> patches to cygwin1.dll? Yes, i've sent them yesterday to cygwin-patches@. Of course, they should be added after new binutils package (in which ld exports ___RUNTIME_PSEUDO_RELOC_LIST__ and ___RUNTIME_PSEUDO_RELOC_LIST_END__) is released. Or we can only export them in pe.sc for now, and go ahead with cygwin (and mingw, if anybody's interested) change. It'd just have no effect until full ld patch is applied. Egor. mailto:deo@logos-m.ru ICQ 5165414 FidoNet 2:5020/496.19