On 3/8/02 2:24 AM, "Matt Sergeant" <[EMAIL PROTECTED]> wrote:
> On Thu, 7 Mar 2002, Bart Schaefer wrote:
>
>> On Thu, 7 Mar 2002, Timothy Demarest wrote:
>>
>>> Additionally, we have a goofy perl install with the prefix of [...] Is
>>> there a way that SpamAssassin could use the perl prefix when searching
>>> in addition to the hardcoded defaults?
>>
>> lib/Mail/SpamAssassin.pm should start out as lib/Mail/SpamAssassin.pm.PL
>> and be processed to generate the .pm file with $Config{prefix} from the
>> Config package pasted in. The same thing should really be done with the
>> assorted .raw files instead of running fixpath.pl on them.
>>
>> The Makefile.PL should contain
>>
>> WriteMakefile(...,
>> PL_FILES => {'spamassassin.PL' => 'spamassassin',
>> 'lib/Mail/SpamAssassin.pm.PL' =>
>> 'lib/Mail/SpamAssassin.pm',
>> ...},
>> ...);
>>
>> And then all of this would Just Work Right when installing with CPAN.
>
> Huh??? What is the problem there? What on earth do you want to
> pre-generate the .pm files from a .PL file for?
>
> fixpath.pl should die though, and we should be using EXE_FILES. Not sure
> why we have this horrible fixpath.pl - I'm going to kill that ASAP.
I agree that fixpath.pl needs to die -- it seems like kind of a kludge. I
haven't changed it primarily for two reasons:
1. Don't know what Justin's intention was with it in the first place (I
think it's mostly to convert '/' to '\' for windows-ness or something)
2. I don't know enough (or indeed much at all) about CPAN, Config{}, etc.
Apart from reservation number (1) above, I'd be very happy to have SA Do The
Right Thing as far as perl goes for its config files; but I think it should
at least keep trying to read config info from /usr/share/spamassassin/,
/usr/local/share/spamassassin/ and /etc/mail/spamassassin/ as it currently
does (pending fixing that one instance of /etc/spamassassin/ to
/etc/mail/spamasssasin/) which I'm about to do to CVS right......now!
C
_______________________________________________
Spamassassin-talk mailing list
[EMAIL PROTECTED]
https://lists.sourceforge.net/lists/listinfo/spamassassin-talk
