Tue, May 16
Mon, May 15
Strictly speaking, you are correct, but man pages often have references elsewhere, and having it here wouldn't be terrible.
So, shall we proceed with this? Still pending review from manpages...
Thu, May 11
Tue, Apr 25
Guys, I don't entirely understand how TARGET/MACHINE and/or TARGET/MACHINE_ARCH come into play here: this patch is about uname(1) only; these variables are not referenced anywhere in its sources, and are partially (?) documented in build(7).
If "platform" (-m) and "processor architecture" (-p) are the same as the TARGET and TARGET_ARCH, then it would be great to include that too.
Apr 16 2017
Mar 10 2017
Fix bcr@'s request.
Mar 6 2017
Don't forget to bump the document date.
Mar 5 2017
Jan 17 2017
Dec 5 2016
Continue the review in D8691
Dec 2 2016
Added D8691 with an updated patch
Nov 21 2016
Nov 20 2016
I'd prefer to not see more \fB and \fR escapes going in; we should use the richer mdoc macros.
Oct 8 2016
Oct 7 2016
Oct 5 2016
Oct 2 2016
Approved. Make sure to bump .Dd before committing.
Approved. Make sure to reference all PRs you might have already opened for the individual man page.
Good catch. Make sure to bump the .Dd to today before committing to head. Is there a PR for it? If so, then reference it in the PR: line.
Don't forget the "Obtained from:" line if you took that patch from another project. Also, reference this review with the full URL. Commit to head, set a "MFC after:" timeout for a number of days.
You need to bump the document date .Dd to the date when you do the commit, i.e. October 2, 2016 (today). If you have submitted this issue as a bug earlier, you need to reference the PR number as well.
Oct 1 2016
Addressed the issues highlighted in the previous revision.
Reworded the statement in mount_fusefs.8 so perhaps it's better understood? as well switched from a noun (effect) to a verb (affect). This was in the manual already but igor suggested break up the sentence.
I've found a few typos.
Missed a utility in previous rev.