Mailing-List: contact cygwin-apps-help AT sourceware DOT cygnus DOT com; run by ezmlm
Sender: cygwin-apps-owner AT sourceware DOT cygnus DOT com
List-Subscribe: <mailto:cygwin-apps-subscribe AT sources DOT redhat DOT com>
List-Archive: <http://sources.redhat.com/ml/cygwin-apps/>
List-Post: <mailto:cygwin-apps AT sources DOT redhat DOT com>
List-Help: <mailto:cygwin-apps-help AT sources DOT redhat DOT com>, <http://sources.redhat.com/lists.html#faqs>
Delivered-To: mailing list cygwin-apps AT sources DOT redhat DOT com
Message-ID: <3BAF8ADB.3000208@ece.gatech.edu>
Date: Mon, 24 Sep 2001 15:34:51 -0400
From: Charles Wilson <cwilson AT ece DOT gatech DOT edu>
User-Agent: Mozilla/5.0 (Windows; U; Windows NT 5.0; en-US; rv:0.9.2) Gecko/20010726 Netscape6/6.1
X-Accept-Language: en-us
MIME-Version: 1.0
To: Charles Wilson <cwilson AT ece DOT gatech DOT edu>
CC: cygwin-apps AT cygwin DOT com
Subject: Re: binutils patch [Was: Re: ncurses announcement - trial run]
References: <EA18B9FA0FE4194AA2B4CDB91F73C0EF08F1A3 AT itdomain002 DOT itdomain DOT net DOT au> <3BA958FB DOT 7060401 AT ece DOT gatech DOT edu> <20010919230018 DOT A26967 AT redhat DOT com> <3BAAE5C6 DOT 6030007 AT ece DOT gatech DOT edu>
Content-Type: text/plain; charset=us-ascii; format=flowed
Content-Transfer-Encoding: 7bit

Charles Wilson wrote:

> Christopher Faylor wrote:
> 
> 
>> I don't mind making another update if you think it is important.  I
>> thought it would be nice for the current version to get some exercise
>> first but if you think we need a new release, I'll make one.
>>
>> Just send me the patch that turns on auto import by default, if that
>> is what you want.

> I've attached a patch that makes ld default to using 
> --enable-auto-import.  However, I've also just submitted two patches to 
> binutils.


Okay, after recent binutils CVS commits (see below), the previously 
posted "make auto-import default" patch still applies cleanly.

> 
> The first attempts to fix the "pe_data_import_dll" variable defined in 
> pe.em, used in pe-dll.c problem.  (The previous situation was ugly which 
> was bad enough, but worse, it caused build failures on 
> non-DLL-supporting pei386 platforms, like arm-epoc-pe.  I'm waiting for 
> someone on one of those other affected platforms to verify that it works 
> for them)


Well, this is now a non-issue.  pe_data_import_dll is still a shared 
variable between pe-dll.c and emultempl/pe.em -- but Nick came up with a 
solution (now committed in CVS) that allows all pe platforms to build 
correctly.  It may be ugly, but it works -- and a "real" fix is FAR more 
intrusive than this "bug" requires.  See:
   http://sources.redhat.com/ml/binutils/2001-09/msg00429.html

> 
> The second fixes the documentation to clarify the recent long long 
> confusion.


this has been accepted and applied to CVS.

So, Chris -- when you get time, and I know that's in short supply this 
week, you might want to generate a new binutils package.  At that point, 
I'll rewrite my documentation within the new ncurses package, generate a 
new set of ncurses tarballs, generate a new dllhelpers package, re-do my 
ncurses announcement text, etc...

BTW, I am presently working on yet another update to the binutils.README 
if you want to include it within the new binutils package.  It'll be 
available at
 
http://www.neuro.gatech.edu/users/cwilson/cygutils/V1.1/dll-stuff/binutils.README-new

*when* I've gotten it ready.

--Chuck