Bug 209255 - [PATCH] mail/postsrsd: init script doesn't kill all processes
Summary: [PATCH] mail/postsrsd: init script doesn't kill all processes
Status: Closed FIXED
Alias: None
Product: Ports & Packages
Classification: Unclassified
Component: Individual Port(s) (show other bugs)
Version: Latest
Hardware: Any Any
: --- Affects Some People
Assignee: Dmitry Marakasov
URL:
Keywords: patch
Depends on:
Blocks:
 
Reported: 2016-05-04 07:51 UTC by geoffroy desvernay
Modified: 2016-05-11 13:18 UTC (History)
1 user (show)

See Also:
ports: maintainer-feedback+

Attachments
patch postsrsd init script (882 bytes, patch)
2016-05-04 07:51 UTC, geoffroy desvernay 		no flags Details | Diff
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description geoffroy desvernay 2016-05-04 07:51:04 UTC 
Created attachment 169949 [details]
patch postsrsd init script

postsrsd seems to launch many processes here, and "service postsrsd stop" doesn't kill them all…

Here is a simple patch to the port to let rc.subr find PIDS (works for me(tm))
Comment 1 Krzysztof 2016-05-04 08:46:37 UTC 
Comment on attachment 169949 [details]
patch postsrsd init script

OK, I've not noticed such problems. As I can see you've added some rc.conf variables (thank you).

So I think this should be added to port.
Comment 2 commit-hook freebsd_committer freebsd_triage 2016-05-11 13:18:05 UTC 
A commit references this bug:

Author: amdmi3
Date: Wed May 11 13:17:48 UTC 2016
New revision: 414992
URL: https://svnweb.freebsd.org/changeset/ports/414992

Log:
  - Make rc.d script able to stop all postsrsd processes
  - Document some rc variables
  - While here, clarify license and fix uneeded variable appends

  PR:		209255
  Submitted by:	dgeo@centrale-marseille.fr
  Approved by:	ports@bsdserwis.com (maintainer, for PR), portmgr blanket (for other fixes)

Changes:
  head/mail/postsrsd/Makefile
  head/mail/postsrsd/files/postsrsd.in