This adds -media autofs map. It's supposed to be used with removable
media. Note that there is a known deficiency in that it never removes
obsolete nodes; it will be addressed later.
This and the next two notes reword the entries in this list to avoid the passive and repeated "This map" and nonspecific "It".
Query the remote NFS server and map exported volumes, which are traditionally mounted on
.It Li -media
Phabricator might not be showing the actual diff, so bear with me. I see
This map is traditionally mounted on
Does "traditionally" mean "by default", or "what most people do"? I suspect it is the first, so this can be split out of the too-long following sentence:
By default, this map is mounted on
nfs-server-ip and share-name need markup to show they are not literal, although I have not yet found out what to use.
It's not commented in the version in this review though?
Ugh. I'd make the comment a bit stronger with an XXX as you did below - e.g.
XXX this should use kern.geom.conftxt, but it lacks access counts.
Looking for hexagon in the dot output makes me feel really icky.
That said, you could simplify slightly with
sysctl ... | sed -n 's/^.*hexagon,label=/../p'
i.e. -n and /p modifier
can you expand on the comment
"Traditionally" here means "what our ancestors used to do". It's not mounted anywhere by default - it's commented out - but by removing a single "#" you can mount it where it used to be on Solaris and most other UNIX systems. And OS X.
The doc changes look good to me. I still would prefer that nfs-server-ip and share-name are shown as non-literal values. The only way I know to do that would be rewriting that section to split up the path as shown below. Is it worth that? I'm not sure.
Access to files on a remote NFS server is provided through the
Inside that directory are subdirectories for each NFS server IP address (WB: or host name?)
and the exported directories.
For example, files in the
directory on the NFS server at 192.168.1.202 will appear in the
directory without any additional configuration.
Courtesy of Ingo Schwarze:
.Pf /net/ Ar nfs-server-ip Ns / Ns Ar share-name Ns /
That shows the full path, but makes it more obvious that nfs-server-ip and share-name are not literal values.
Quoting https://wiki.freebsd.org/CodeReview :
Do this by scrolling down to the bottom of the review page, select the Close option from the Action dropdown menu just above the Comment field, and click Submit. The Close option will not be visible unless the review is in the Approved state.
You can close a revision (in any state) from the command line with the following:
arc close-revision DXXX