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





------- Additional Comments From [EMAIL PROTECTED]  2004-08-23 15:25 -------
doing some more debugging...  the "spamassassin -D --lint" one fixes the
relative path to an absolute path:

>before> ../rules/Test.pm
debug: plugin: fixed relative path:
/home/felicity/SA/spamassassin-head/rules/../rules/Test.pm
>after> /home/felicity/SA/spamassassin-head/rules/../rules/Test.pm

but plugin_file.t goes relative to relative:

>> ../../data/testplugin.pm
>> log/localrules.tmp/../../data/testplugin.pm

and the reason, so far as I can tell, is that -C/--siteconfigpath is set
differently between the two runs.  for the "spamassassin" run, it explicitly
gets "-C $HOME/..../rules" versus the test which gets "-C log/test_rules_copy"


So if we want to force an actual absolute value, yeah, there needs to be a
patch, but it seems like the current patches do it in the wrong place.  Conf.pm
has a load_plugin function which calls
Conf::Parser::fix_path_relative_to_current_file to set the relative path.  So
why not either 1) do the rel2abs thing in load_plugin(), or 2) change
fix_path_relative_to_current_file to do rel2abs?



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

Reply via email to