bin/145912: trivial enhancement patch for man crontab

[Garrett Cooper]

The following reply was made to PR bin/145912; it has been noted by GNATS.

From: Garrett Cooper <yanefbsd at gmail.com>
To: "Julian H. Stacey" <jhs at berklix.com>
Cc: FreeBSD-gnats-submit at freebsd.org
Subject: Re: bin/145912: trivial enhancement patch for man crontab
Date: Wed, 21 Apr 2010 05:51:40 -0700

 On Wed, Apr 21, 2010 at 3:12 AM, Julian H. Stacey <jhs at berklix.com> wrote:
 >
 
 ...
 
 > =A0.Bl -tag -width /var/cron/allow -compact
 > =A0.It Pa /var/cron/allow
 > =A0.It Pa /var/cron/deny
 > + .It Pa /var/cron/tabs/{login_names}
 
     Should {login_names} be removed because of the potential ambiguity
 that it introduces and be better defined through a description like
 with cron(8)? Also, there's no description of the proposed change in
 the bug report, so for someone that's trying to figure out what this
 change is doing from a customer perspective it would be nice if it
 said something like 'add a reference noting where the default
 installed crontabs are located', etc.
     Finally, this documentation kind of duplicates what's already in cron(8=
 ):
 
      The cron utility searches /var/cron/tabs for crontab files which are
      named after accounts in /etc/passwd; crontabs found are loaded into me=
 m-
      ory.  The cron utility also searches for /etc/crontab which is in a di=
 f-
      ferent format (see crontab(5)).
 
 [...]
 
 FILES
      /etc/crontab     System crontab file
      /etc/pam.d/cron  pam.conf(5) configuration file for cron
      /var/cron/tabs   Directory for personal crontab files
 
 Thanks,
 -Garrett