Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!

Bug 455604

Summary: mail-filter/spamassassin-3.3.2-r1 initd script fails because checkpath is missing
Product: Gentoo Linux Reporter: Frieder Bürzele <evermind>
Component: Current packagesAssignee: No maintainer - Look at if you want to take care of it <maintainer-needed>
Severity: normal CC: bug
Priority: Normal Keywords: PATCH
Version: unspecified   
Hardware: All   
OS: Linux   
Package list:
Runtime testing required: ---
Bug Depends on:    
Bug Blocks: 583908    
Attachments: add checkpath and userdetection

Description Frieder Bürzele 2013-02-05 14:14:03 UTC
checkpath is missing in init script, so if you configure spamd to use another directory for the pidfile. eg. if you want to run it with another user it fails.

Reproducible: Always
Comment 1 Frieder Bürzele 2013-02-05 14:17:55 UTC
Created attachment 338012 [details, diff]
add checkpath and userdetection

fix the problem:
* add checkpath
* detect and set permission/directory
* chown if user specified
Comment 2 Jeroen Roovers gentoo-dev 2013-02-05 21:32:46 UTC
Comment on attachment 338012 [details, diff]
add checkpath and userdetection

It shouldn't use /var/run at all, but /run instead.
Comment 3 Frieder Bürzele 2013-02-05 21:53:32 UTC
(In reply to comment #2)
> Comment on attachment 338012 [details, diff] [details, diff]
> add checkpath and userdetection
> It shouldn't use /var/run at all, but /run instead.

agree it should not use /var/run but the patch still fixes the path creation and user chown.

So in the file /etc/conf.d/spamd the PIDFILE should also be moved to /run/
Comment 4 Quincy 2013-02-26 09:38:18 UTC
I also stumbled across this when upgrading several production systems to use tmpfs for /run. I don't quite understand why the working patch got removed, nothing said about /var/run in that. Definitely there should be an updated init script in portage, because it can stop spamassassin from working at all which is really a bad thing in webhosting environments.
Comment 5 Marcin Mirosław 2016-05-31 12:23:35 UTC
See also bug #523960, "-u" doesn't change running user of master process. I don't see reason to move pidfile elsewhere. Is a good reason to have option to set PIDFILE in conf.d/spamd? I think it can be removed.
Comment 6 Michael Orlitzky gentoo-dev 2016-06-04 15:37:46 UTC
I just fixed this in yet-another-revision:

I reworked the spamd init/config -- now the PID file is hard-coded, and all that is needed is one simple call to `checkpath --directory`.