Page MenuHomeFreeBSD

Check for running daemons before a custom start_cmd is executed
ClosedPublic

Authored by erdgeist_erdgeist.org on Nov 14 2015, 10:21 PM.
Tags
None
Referenced Files
Unknown Object (File)
Wed, Nov 6, 7:35 PM
Unknown Object (File)
Oct 22 2024, 3:52 AM
Unknown Object (File)
Oct 22 2024, 3:52 AM
Unknown Object (File)
Oct 22 2024, 3:52 AM
Unknown Object (File)
Oct 22 2024, 3:36 AM
Unknown Object (File)
Sep 17 2024, 11:26 AM
Unknown Object (File)
Sep 16 2024, 7:44 PM
Unknown Object (File)
Sep 16 2024, 11:47 AM
Subscribers

Details

Summary

Currently rc scripts implementing their own start_cmd do not enjoy the benefits of rc.subr's own check for rc_pid.

This leads to around a third of ports with such a start_cmd not to check for the process at all and two thirds of ports to re-implement this check (sometimes wrongly).

This patch moves the check for rc_pid to before ${rc_arg}_cmd is executed.

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

erdgeist_erdgeist.org retitled this revision from to Check or running daemons before a custom start_cmd is executed.
erdgeist_erdgeist.org updated this object.
erdgeist_erdgeist.org edited the test plan for this revision. (Show Details)
erdgeist_erdgeist.org added reviewers: feld, jilles.
erdgeist_erdgeist.org set the repository for this revision to rS FreeBSD src repository - subversion.
erdgeist_erdgeist.org retitled this revision from Check or running daemons before a custom start_cmd is executed to Check for running daemons before a custom start_cmd is executed.Nov 14 2015, 10:44 PM
jilles edited edge metadata.
This revision is now accepted and ready to land.Nov 21 2015, 2:37 PM
feld edited edge metadata.

This solves an annoyance for me. Thanks for doing this!

This revision was automatically updated to reflect the committed changes.