Closed stoecker closed 10 years ago
Is something wrong with this? I'd like to have that included in official repo, before I finally update the openSUSE package.
Hi, sorry, I didn't see this request and for some reason only got notified about your first.
Anyway, so what does this do... allow one to pass a blank --homedir parameter?
Thanks, -Max
Probably. But mainly it allows to pass no homedir option at all, the default for older installations.
But homedir isn't required, is it?
$sa_home_dir = '/var/spool/spamassassin/spampd'; .... if ( $options{'homedir'} ) { $sa_home_dir = $options{'homedir'}; }
Well, something strange here. You're right. The fix should not work, but it does. So I overlooked something. Will check it.
Had time again to look into this: Fix is wrong, reason for the problem is the default path does not exist.
Thanks for the update! Is there anything that could be done to make the problem/solution more obvious to future users? Like a message saying the path doesn't exist... :)
-Max
Problem now is that pyzor silently fails, whereas other non-executable filters work. If the dir does not exist and cannot be opened and the tool fails completely, then it would be obvious and easy to fix. So probably an abort with error message would be a good idea.
Another issue I found. Pyzor und thus spampd wont start, when new option --homedir isn't passed. Make the option optional again.
Seems to work now as expected on my system.