Add "cron -n".
ClosedPublic
Actions

Authored by trasz on Aug 19 2016, 12:42 PM.

Details

Reviewers

jilles
pfg

Group Reviewers

manpages

Commits

rS304570: Add the "-n" flag to cron(8), to prevent it from daemonizing.

Summary

Add the "-n" flag to cron(8), to prevent it from daemonizing.
This makes it possible to use it with external supervisors, like
the one in https://reviews.freebsd.org/D7474. The "-n" flag name
is compatible with Linux and NetBSD.

Diff Detail

Repository

rS FreeBSD src repository - subversion

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 4863
Build 4926: arc lint + arc unit

Event Timeline

trasz updated this revision to Diff 19489.Aug 19 2016, 12:42 PM

trasz retitled this revision from to Add "cron -n"..

trasz updated this object.

trasz edited the test plan for this revision. (Show Details)

Herald added a reviewer: manpages. · View Herald TranscriptAug 19 2016, 12:42 PM

Herald added a subscriber: imp. · View Herald Transcript

trasz added a reviewer: pfg.Aug 19 2016, 12:47 PM

Looks good to me. Make sure to update .Dd in cron.8 when committing.

This revision is now accepted and ready to land.Aug 19 2016, 1:45 PM

Looks very neat.

wblock added a subscriber: wblock.Aug 20 2016, 4:26 AM

wblock added inline comments.

usr.sbin/cron/cron/cron.8
137	Avoid contractions. The semicolon is questionable, could be replaced with a comma or just split the sentence there. Do not daemonize, run in foreground instead.