[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: patch to common.make for cygwin paths
From: |
Stephen Brandon |
Subject: |
Re: patch to common.make for cygwin paths |
Date: |
Wed, 07 Nov 2001 10:06:56 +0000 |
Hi,
Hmmm, I'm still getting the warnings after updating from
CVS. What's going on is that $(GNUSTEP_TARGET_OS) has not
been set by the time it's being checked in common.make. I
tested setting GNUSTEP_TARGET_OS from the command line
before doing the "make target=i386-mingw32" and that did the
right thing.
But - shouldn't the "target" I specify in "make
target=i386-mingw32" become the internal GNUSTEP_TARGET_OS
variable? Or perhaps that only happens in a subsequent part
of the makefile system?
Cheers,
Stephen
On Tue, 6 Nov 2001 02:50:10 +0000 (GMT)
Nicola Pero <n.pero@mi.flashnet.it> wrote:
>
> > I get the bogus warning when compiling on Windows and
> it really is
> > annoying/confusing. So it would be nice to fix it. I
> guess at the risk
> > of expanding the code infinitely, we could do:
> >
> > ifeq ($(findstring mingw, $(GNUSTEP_TARGET_OS)), mingw)
> > ...windows specific check
> > else
> > ...unix specific check
> > endif
> >
> > (also including cygwin)
>
> Ok - I committed something of this kind - it didn't break
> unix :-) - let
> me know if it fixes the spurious warning on windows or
> not.
>
- patch to common.make for cygwin paths, Stephen Brandon, 2001/11/05
- Re: patch to common.make for cygwin paths, Nicola Pero, 2001/11/06
- Re: patch to common.make for cygwin paths, Stephen Brandon, 2001/11/06
- Re: patch to common.make for cygwin paths, Adam Fedor, 2001/11/06
- Re: patch to common.make for cygwin paths, Stephen Brandon, 2001/11/06
- Re: patch to common.make for cygwin paths, Nicola Pero, 2001/11/06
- Re: patch to common.make for cygwin paths,
Stephen Brandon <=
- Re: patch to common.make for cygwin paths, Nicola Pero, 2001/11/07
- Re: patch to common.make for cygwin paths, Nicola Pero, 2001/11/08