Bug 177788 - Directory for pid-file not included in package
Summary: Directory for pid-file not included in package
Status: CLOSED RAWHIDE
Alias: None
Product: Fedora
Classification: Fedora
Component: spamassassin
Version: rawhide
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Warren Togami
QA Contact:
URL:
Whiteboard:
Keywords:
Depends On:
Blocks: 171491
TreeView+ depends on / blocked
 
Reported: 2006-01-13 23:27 UTC by Roy-Magne Mo
Modified: 2007-11-30 22:11 UTC (History)
7 users (show)

(edit)
Clone Of:
(edit)
Last Closed: 2006-01-24 06:51:38 UTC


Attachments (Terms of Use)
Patch to allow building under FC-3 (1.03 KB, patch)
2006-01-19 02:33 UTC, Jason Tibbitts
no flags Details | Diff

Description Roy-Magne Mo 2006-01-13 23:27:00 UTC
Description of problem:
Directory for pid-file for spamassassin is not included in spamassassin package. 
/var/run/spamassassin/ should be included in spec-file

Version-Release number of selected component (if applicable):
spamassassin-3.1.0-3.fc5.1 (installed on FC4 server)

How reproducible:


Steps to Reproduce:
1.
2.
3.
  
Actual results:


Expected results:


Additional info:

Comment 1 Roy-Magne Mo 2006-01-13 23:30:30 UTC
Also when stopping and removing the pid file, only the symbolic link are being
erased not the actual spec-file referenced in $SPAMD_PID in initscript.

Comment 2 Warren Togami 2006-01-14 07:35:02 UTC
Good catch on the first problem, this will go to rawhide soon.

On the second problem, I suppose it wouldn't hurt to remove it, but are there
any instances where this would cause any problems as-is?

Comment 3 Roy-Magne Mo 2006-01-14 15:27:53 UTC
(In reply to comment #2)
 > On the second problem, I suppose it wouldn't hurt to remove it, but are there
> any instances where this would cause any problems as-is?

No, seems like spamd is removing the pid file cleanly on shutdown.


Comment 4 Jason Tibbitts 2006-01-19 02:32:55 UTC
Take this with a grain of salt because I'm building on FC-3, but the current
spamassassin package (3.1.0-4) needs to create
$RPM_BUILD_ROOT%{_localstatedir}/run/spamassassin in order to build for me.

I used the attached patch, which also removes a redundant creation of
$RPM_BUILD_ROOT%{_sysconfdir}/sysconfig

Comment 5 Jason Tibbitts 2006-01-19 02:33:51 UTC
Created attachment 123408 [details]
Patch to allow building under FC-3

Comment 6 Warren Togami 2006-01-24 06:51:38 UTC
Thanks, done in FC5.



Note You need to log in before you can comment on or make changes to this bug.