Message-Id: <200209201533.g8KFX0O09636@delorie.com>
Mailing-List: contact cygwin-help AT cygwin DOT com; run by ezmlm
List-Subscribe: <mailto:cygwin-subscribe AT cygwin DOT com>
List-Archive: <http://sources.redhat.com/ml/cygwin/>
List-Post: <mailto:cygwin AT cygwin DOT com>
List-Help: <mailto:cygwin-help AT cygwin DOT com>, <http://sources.redhat.com/ml/#faqs>
Sender: cygwin-owner AT cygwin DOT com
Mail-Followup-To: cygwin AT cygwin DOT com
Delivered-To: mailing list cygwin AT cygwin DOT com
X-Sent: 20 Sep 2002 15:32:33 GMT
X-Newsgroups: gmane.os.cygwin
Subject: Re: Building SpamAssassin for Cygwin
From: news AT garydjones DOT mailshell DOT com
References: <02Sep19.100358-0700_pdt.290499-31649+137 AT lmg DOT ahnet DOT net> <20020920124805 DOT GD1852 AT tishler DOT net>
User-Agent: Xnews/4.11.09
X-No-Archive: yes
To: cygwin AT cygwin DOT com
X-Gate: Hamster/1.3.23.185 NewsToMail-Gate
Date: Fri, 20 Sep 2002 17:12:50 +0200

On 20 Sep 2002, Jason Tishler <jason AT tishler DOT net> wrote:

> I "fixed" problem with the (first hunk of the) first patch. 
> Since I'm not a Perl person, I haven't figured out the right
> way to fix the problem

That's exactly my problem too. I would prefer to fix the 
problems correctly and ask the SA team to take that onboard and 
get something that works not just for me, but for anyone who 
comes along later as well.

 -- for example, patching Makefile.PL
> or whatever generates the offending line in Makefile.  Does
> anyone know the right way to do this? 

Applying your patches doesn't work on 2.41 :-( I may simply use
an older version if I can still get it. Thanks for your suggestion
though.

The second "chunk" of your first patch is no longer required
for 2.41, its already in there, so I guess they fixed that already.
I can't find anything like the code in your second patch either,
so they must have changed that completely.

> BTW, I needed the second patch to stop spamassassin from
> thinking that Perl was running under Windows instead of
> Cygwin. 

Huh? I didn't know SA knew anything about Windows. AFAIK it 
isn't supported. Wait, maybe it was in 2.31 and is not now,
which would explain why the code in your second patch is
no longer there.

--
Unsubscribe info:      http://cygwin.com/ml/#unsubscribe-simple
Bug reporting:         http://cygwin.com/bugs.html
Documentation:         http://cygwin.com/docs.html
FAQ:                   http://cygwin.com/faq/