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
Date:	Thu, 17 May 2001 14:10:00 -0400
From:	Christopher Faylor <cgf AT redhat DOT com>
To:	cygwin-apps AT cygwin DOT com
Cc:	danny_r_smith_2001 AT yahoo DOT co DOT nz
Subject:	Re: gcc-2.95.3-4 to mingw
Message-ID:	<20010517141000.A2669@redhat.com>
Reply-To:	cygwin-apps AT cygwin DOT com
Mail-Followup-To:	cygwin-apps AT cygwin DOT com, danny_r_smith_2001 AT yahoo DOT co DOT nz
References:	<20010517043410 DOT 19503 DOT qmail AT web6403 DOT mail DOT yahoo DOT com> <20010517030135 DOT A17843 AT redhat DOT com> <3B03C0D3 DOT 7995654D AT yahoo DOT com>
Mime-Version:	1.0
User-Agent:	Mutt/1.3.11i
In-Reply-To:	<3B03C0D3.7995654D@yahoo.com>; from earnie_boyd@yahoo.com on Thu, May 17, 2001 at 08:15:15AM -0400

Ok.  If I am going to be applying these patches, then I have a few
observations.

1) They should not have MS-DOS line endings.

2) They should not reference MS-DOS paths.

3) I'm not going to add the word "mingw" to the compiler output for the cygwin
   gcc compiler.  You are, of course, welcome to change that for any mingw
   compiler.  I assume that you are doing that already.

4) I'd appreciate a different ChangeLog entry for each directory that is
   modified.  I can't use the one that you sent since it is non-standard.

I'll try to use the patch that was submitted here by stripping the CR's
and changing all of the paths to '/' but I would appreciate it if future
submissions adhered to the above criteria.

>One other little bit, a recurring problem with sys_nerr and
>sys_errlist[] and dllimport in libiberty/strerror.c.  This should
>probably be sent to libiberty; I haven't included in ChangeLog.

Also, it seems to me that this patch is being handled in the wrong way.
There is already a configure test for this functionality.  Why is it
being handled incorrectly in the mingw case?

cgf

- Raw text -