git: 74bd20769706 - main - Reference newfs_msdos(8) from the newfs(8) manual

[Warner Losh]

On Fri, Jan 8, 2021 at 5:51 AM Mateusz Piotrowski <0mp at freebsd.org> wrote:

> Hello Konstantin,
>
> On 1/7/21 7:16 PM, Konstantin Belousov wrote:
> > On Thu, Jan 07, 2021 at 09:24:14AM +0000, Mateusz Piotrowski wrote:
> >> The branch main has been updated by 0mp (doc, ports committer):
> >>
> >> URL:
> https://cgit.FreeBSD.org/src/commit/?id=74bd20769706041108a573601cf0b61c755bdc56
> >>
> >> commit 74bd20769706041108a573601cf0b61c755bdc56
> >> Author:     Mateusz Piotrowski <0mp at FreeBSD.org>
> >> AuthorDate: 2021-01-07 09:22:20 +0000
> >> Commit:     Mateusz Piotrowski <0mp at FreeBSD.org>
> >> CommitDate: 2021-01-07 09:24:07 +0000
> >>
> >>      Reference newfs_msdos(8) from the newfs(8) manual
> > Why ?  newfs is not the interface to create arbitrary kind of filesystem,
> > it is just UFS writer.
>
> I think that it is beneficial nevertheless. Now users can discover other
> newfs_* commands FreeBSD
> offers. Also, newfs_msdos(8) already references newfs(8) (there are only
> two items in the SEE ALSO
> section there, the other one being gpart(8)).
>
>  From my perspective it is not too bad to have this reference in the newfs
> manual. It seems like
> it's also something some FreeBSD users expect to see in the manual page as
> we received a bug report
> for that.
>
> I can revert this patch and also remove newfs(8) from the newfs_msdos
> manual while at it if desired.
> Perhaps newfs commands are easy enough to find so this patch only hurts
> the readability of the manual...
>

I think it's a good change because many other filesystem manipulation
commands will do arbitrary filesystems. Since it breaks this symmetry,
having the xref is fine.

Warner