Mailing-List: contact cygwin-developers-help AT sourceware DOT cygnus DOT com; run by ezmlm
List-Subscribe: <mailto:cygwin-developers-subscribe AT sourceware DOT cygnus DOT com>
List-Archive: <http://sourceware.cygnus.com/ml/cygwin-developers/>
List-Post: <mailto:cygwin-developers AT sourceware DOT cygnus DOT com>
List-Help: <mailto:cygwin-developers-help AT sourceware DOT cygnus DOT com>, <http://sourceware.cygnus.com/ml/#faqs>
Sender: cygwin-developers-owner AT sourceware DOT cygnus DOT com
Delivered-To: mailing list cygwin-developers AT sourceware DOT cygnus DOT com
Message-ID: <38779BDD.12518FDE@vinschen.de>
Date: Sat, 08 Jan 2000 21:19:41 +0100
From: Corinna Vinschen <corinna AT vinschen DOT de>
X-Mailer: Mozilla 4.7 [en] (WinNT; I)
X-Accept-Language: de,en
MIME-Version: 1.0
To: Chris Faylor <cgf AT cygnus DOT com>
CC: cygdev <cygwin-developers AT sourceware DOT cygnus DOT com>
Subject: Re: ntsec-patch16 (and some other)
References: <38778A0D DOT 5633ADAF AT vinschen DOT de> <20000108141938 DOT A7248 AT cygnus DOT com>
Content-Type: text/plain; charset=us-ascii
Content-Transfer-Encoding: 7bit

Chris Faylor wrote:
> 
> On Sat, Jan 08, 2000 at 08:03:41PM +0100, Corinna Vinschen wrote:
> >[...]
> >- configure: I had to add the directory `mingw/profile'
> >  else it was ignored.
> 
> This can't be right.  The profile directory should be handled by mingw's
> configure.  The "top level" configure is only supposed to handle
> directories immediately below it.

That's curious, because on the other hand a `make' in the winsup dir
want to `make all' in mingw/profile which then failes because there's
no Makefile...

> >- cygwin/smallprintf.c: I had to add `#define __INSIDE_CYGWIN__'
> >  else the linker said sth. about `undef'd ref to _ctype_'.
> >
> >- cygwin/regexp/regexp.c: Ditto.
> 
> These are supposed to be handled already.  It obviously isn't required
> all the time or we wouldn't be able to make snapshots.

I was astonished, too, because the sources are exactly
the same. The _only_ difference I saw:

In the previous snapshots the compiler got the flag
`-D__INSIDE_CYGWIN__' as well. This is not the case
in the latest snapshot.

> >- mingw/profile/configure: This configure script couldn't find
> >  install-sh, because the search paths for $ac_dir didn't reach
> >  back enough in the dir hirarchy. Here I have added the path
> >  `$srcdir/../../..'.
> 
> I can see that this could be required, but you shouldn't modify
> configure scripts.  They're autogenerated from configure.in.

Sorry. I've never done an autoconf, so I forgot about this fact.

> I'll weed out the configure, regexp, and smallprint patches and
> apply what's left.

That's ok. The patches should show you my problems with the build.
Since I know that you have a better insight in the new structure,
I asked you to take this special look. 

Corinna