http://bugzilla.spamassassin.org/show_bug.cgi?id=3717





------- Additional Comments From [EMAIL PROTECTED]  2004-08-23 14:17 -------
?!! bizarre.

so does the 'plugin_file.t' test pass with this applied?

I took a look with strace to see what "do" looks for.  it appears that when it's
given a relative path, it uses @INC instead of trying the path directly, as it
should:
8309  stat64("lib/log/localrules.tmp/../../data/testplugin.pmc", 0xbffff590) = 
-1 E
NOENT (No such file or directory)
8309  open("lib/log/localrules.tmp/../../data/testplugin.pm", 
O_RDONLY|O_LARGEFILE)
 = -1 ENOENT (No such file or directory)
8309  stat64("../blib/lib/log/localrules.tmp/../../data/testplugin.pmc", 
0xbffff590
) = -1 ENOENT (No such file or directory)
8309  open("../blib/lib/log/localrules.tmp/../../data/testplugin.pm", 
O_RDONLY|O_LA
RGEFILE) = -1 ENOENT (No such file or directory)
8309  
stat64("/usr/local/share/perl/5.8.4/log/localrules.tmp/../../data/testplugin.
pmc", 0xbffff590) = -1 ENOENT (No such file or directory)
.... etc.
8309  write(2, "failed to load plugin log/localr"..., 93) = 93
8309  write(2, "failed to create instance of plu"..., 177) = 177


So this patch works, as long as it's an absolute path instead of a relative
path?   Looking at the perl 5.8.4 source, pp_ctl.c line 3180 approx, it does
indeed look like it!

It also appears that method uses file_is_absolute() which calls
PERL_FILE_IS_ABSOLUTE() which has lots of renditions for various OSes -- so I
think it's portable.  (yay for open source that I could check this.)

+1 (assuming you say the test passes. ;)




------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.

Reply via email to