[prev in list] [next in list] [prev in thread] [next in thread] 

List:       spamassassin-devel
Subject:    Re: [Bug 4124] New: New spamassassin script doesn't work due to tainting
From:       "Malte S. Stretz" <msquadrat.nospamplease () gmx ! net>
Date:       2005-02-17 8:47:25
Message-ID: 200502170947.26726 () malte ! stretz ! eu ! org
[Download RAW message or body]

On Thursday 17 February 2005 09:30 CET Daniel Quinlan wrote:
> Daniel Quinlan wrote:
> >> I'm -1 on switching M::B until it's standard in our minimum required
> >> version of Perl.
>
> "Malte S. Stretz" <msquadrat.nospamplease@gmx.net> writes:
> > Fine, thanks for discussing.
>
> Malte, I really appreciate your work on the Makefile system.  My
> concerns about the complicated Makefile code in no way reflect on your
> excellent work.  The Makefiles are complicated because of necessity.

That's why I hope (I'm still trying M::B, maybe it sucks too for what we 
need) that all that complicated underdocumentated hacks which rely on 
EU::MM internals and of which I'm afraid that they will blow up one day 
something is fixed in EU::MM can go away.  The Makefile.PL hacks were 
indeed hard work but I'm the last one who wouldn't like to see it die :)

> However, I am still going to disagree sometimes about how things are
> done and when that happens, it would be nice to do so without hostility.

Sorry, I'm currently a bit in exams stress and just after I sent the mail I 
noticed that I could have worded it more carefully.

> To put it another way, I can't recall a -1 ever ending a discussion.

Hm, k, so it seems like we've got a different interpretation of -1.  I 
remember the -1 as "veto, don't do that".  But... ah, hey, I'll have my 
last exam on Friday and go for skiing vacation next week, maybe I'm less 
grumpy when I'm back :)

> > You have noticed that we already require a version of EU::MM which is
> > not part of our minimum required version of Perl?  Sorry but ditching
> > M::B just because it isn't shipped with Perl 5.6.1 is complete
> > nonsense IMO.
>
> We only seem to require 5.45 which is the version that ships with 5.6.1
> by default and it works for me.

Perl 5.6.1 contains 5.45?  Cool, I thought it was added for 5.6.2.

But that version does not support the DESTDIR stuff which is why the hacks 
for that feature are in the Makefile.PL -- which broke at least once when 
something deep inside EU::MM was changed.

Well, let's see after I'm back from vacation, I guess I can think clearer 
then.

Cheers,
Malte

-- 
[SGT] Simon G. Tatham: "How to Report Bugs Effectively"
      <http://www.chiark.greenend.org.uk/~sgtatham/bugs.html>
[ESR] Eric S. Raymond: "How To Ask Questions The Smart Way"
      <http://www.catb.org/~esr/faqs/smart-questions.html>
[prev in list] [next in list] [prev in thread] [next in thread]