Date: Mon, 8 Jul 2013 17:55:47 -0400 (EDT) From: Benjamin Kaduk <kaduk@MIT.EDU> To: Craig Rodrigues <rodrigc@freebsd.org> Cc: Marcel Moolenaar <marcel@freebsd.org>, freebsd-arch@freebsd.org Subject: Re: [RFC] Review of mount.conf man page Message-ID: <alpine.GSO.1.10.1307081739190.26275@multics.mit.edu> In-Reply-To: <CAG=rPVetRb=mRWGLZYg4%2BfoscnEwU%2BSDvFuigvk8_f64G=QzRg@mail.gmail.com> References: <CAG=rPVetRb=mRWGLZYg4%2BfoscnEwU%2BSDvFuigvk8_f64G=QzRg@mail.gmail.com>
next in thread | previous in thread | raw e-mail | index | archive | help
On Mon, 8 Jul 2013, Craig Rodrigues wrote: > Hi, > > In this GRN, Marcel overhauled the root mount logic: > > ------------------------------------------------------------------------ > r214006 | marcel | 2010-10-17 22:01:53 -0700 (Sun, 17 Oct 2010) | 20 lines > > Re-implement the root mount logic using a recursive approach, whereby each > root file system (starting with devfs and a synthesized configuration) can > contain directives for mounting another file system as root. > ------------------------------------------------------------------------ > > My diff is here: > > http://people.freebsd.org/~rodrigc/mount-conf/mount.conf.diff.txt > > I also have the man page rendered in ASCII, for easier reading: > > http://people.freebsd.org/~rodrigc/mount-conf/mount.conf.8.txt A few notes: our style for man pages (mdoc) is to start a new line for new sentences. I am not sure that the .Cm macro is appropriate to describe .onfail and friends; I guess that devd.conf.5 is using .Ic which might be better. The default font for .Em is supposedly italic, so it may not be best for the mountroot prompt, either. I'm not sure why there are square brackets around the 'N' argument to .timeout; does the default behavior of 3 seconds apply to a .timeout without 'N', or the absence of a .timeout? Can you double-check whether .Pp is needed after .Ed? Thanks for writing this, it's always good to have these things documented. -Ben
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?alpine.GSO.1.10.1307081739190.26275>