Date: Wed, 25 Sep 2019 07:57:04 -0500 From: Kyle Evans <kevans@freebsd.org> To: Baptiste Daroussin <bapt@freebsd.org> Cc: src-committers <src-committers@freebsd.org>, svn-src-all <svn-src-all@freebsd.org>, svn-src-head <svn-src-head@freebsd.org> Subject: Re: svn commit: r352668 - in head/usr.sbin/cron: cron crontab lib Message-ID: <CACNAnaGodPFy8%2B7uLgfSCnxcXX_VRQFRHLcHofrnECQcDHNkCQ@mail.gmail.com> In-Reply-To: <20190925065753.vmw4ls426xighbag@ivaldir.net> References: <201909250237.x8P2bf0X033449@repo.freebsd.org> <20190925065753.vmw4ls426xighbag@ivaldir.net>
next in thread | previous in thread | raw e-mail | index | archive | help
On Wed, Sep 25, 2019 at 1:58 AM Baptiste Daroussin <bapt@freebsd.org> wrote: > > On Wed, Sep 25, 2019 at 02:37:41AM +0000, Kyle Evans wrote: > > Author: kevans > > Date: Wed Sep 25 02:37:40 2019 > > New Revision: 352668 > > URL: https://svnweb.freebsd.org/changeset/base/352668 > > > > Log: > > cron: add log suppression and mail suppression for successful runs > > > > This commit adds two new extensions to crontab, ported from OpenBSD: > > - -n: suppress mail on succesful run > > - -q: suppress logging of command execution > > > > The -q option appears decades old, but -n is relatively new. The > > original proposal by Job Snijder can be found here [1], and gives very > > convincing reasons for inclusion in base. > > > > This patch is a nearly identical port of OpenBSD cron for -q and -n > > features. It is written to follow existing conventions and style of the > > existing codebase. > > > > Example usage: > > > > # should only send email, but won't show up in log > > * * * * * -q date > > > > # should not send email > > * * * * * -n date > > > > # should not send email or log > > * * * * * -n -q date > > > > # should send email because of ping failure > > * * * * * -n -q ping -c 1 5.5.5.5 > > > > [1]: https://marc.info/?l=openbsd-tech&m=152874866117948&w=2 > > > > PR: 237538 > > Submitted by: Naveen Nathan <freebsd_t.lastninja.net> > > Reviewed by: bcr (manpages) > > MFC after: 1 week > > Differential Revision: https://reviews.freebsd.org/D20046 > > > I do think this deserves an entry in the release notes > Ahh, I'm inclined to agree. I'll write up a RELNOTES entry here shortly. Thanks, Kyle Evans
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?CACNAnaGodPFy8%2B7uLgfSCnxcXX_VRQFRHLcHofrnECQcDHNkCQ>