https://issues.apache.org/SpamAssassin/show_bug.cgi?id=6313

--- Comment #7 from Mark Martinec <[email protected]> 2010-03-02 14:36:30 
UTC ---
> My claim should be: running spamd with -Q or -q AND -x should not result in 
> creation of a ~/.spamassasin directory (since the -x indicates "Disable user
> config files" (per man-page) )  and both -q and -Q state "(needs -x)"

Good, that is also what my patch does: -x turns off $opt{'user-config'},
which in turn prevents creation of a $spam_conf_dir:

  if ($opt{'user-config'} && ! -d $spam_conf_dir) {
    if (mkdir $spam_conf_dir, 0700) {

There is no disagreement between myself (and my patch) and Juergen.

The only open question is a matter of redundancy:

> Now, this opens a question, why the sub handle_user_setuid_with_sql()
> tries to create the user's .spamassassin directory at all, considering
> that it _only_ gets called when $opt{'user-config'} is false

If it is really just redundant, that's fine with me. My only worry is
that the current code might be intentional, but somehow turned out
to work differently that originally anticipated. So, can anyone answer
why the sub handle_user_setuid_with_sql() tries to create the user's
.spamassassin directory at all? Options -q and -Q need -x, according to docs.

-- 
Configure bugmail: 
https://issues.apache.org/SpamAssassin/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.

Reply via email to