Page MenuHomeFreeBSD

rewrite of intro(4) manpage
ClosedPublic

Authored by andrew_tao173.riddles.org.uk on Apr 2 2019, 6:01 PM.
Tags
Referenced Files
Unknown Object (File)
Jan 10 2024, 12:39 AM
Unknown Object (File)
Dec 20 2023, 4:12 AM
Unknown Object (File)
Nov 8 2023, 9:41 AM
Unknown Object (File)
Nov 8 2023, 12:15 AM
Unknown Object (File)
Nov 6 2023, 11:57 PM
Unknown Object (File)
Nov 6 2023, 1:19 PM
Unknown Object (File)
Oct 16 2023, 3:48 PM
Unknown Object (File)
Oct 7 2023, 8:35 AM

Details

Summary

intro(4) was very obsolete; here is a proposed rewrite.

The general changes are:

  • Remove issues that no longer apply thanks to devfs
  • add language pointing out devfs's role and referencing its config
  • add a "historical notes" section and move discussion of block vs character devs to it, including pointing out the removal of block devs
  • modernize some examples

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

rpokala added inline comments.
share/man/man4/intro.4
64 ↗(On Diff #55740)

I think we're supposed to start a new line after ;.

106 ↗(On Diff #55740)

To reduce the redundancy of devfs.conf, perhaps something like this:

But since device nodes are not stored persistently between reboots,
those file permissions are set from rules specified in
.Xr devfs.conf 5 ,
at boot time or dynamically, or else set manually using the
.Xr devfs 8
command.
share/man/man4/intro.4
64 ↗(On Diff #55740)

why? I find many counter-examples.

106 ↗(On Diff #55740)

redundancy of devfs.conf vs. devfs.rules?

share/man/man4/intro.4
64 ↗(On Diff #55740)

I'm pretty sure I've been told that by various manpage reviewers over the past few years.

I suggest you run the page through igor and mandoc -Tlint, and defer to their wisdom rather than mine. :-)

106 ↗(On Diff #55740)

I clearly can't read today... 🤦‍♂️

So, never mind.

andrew_tao173.riddles.org.uk added inline comments.
share/man/man4/intro.4
64 ↗(On Diff #55740)

igor and mandoc complain about the missing linebreak for new sentence on line 187 below (which I had already noticed), but neither of them say anything about this semicolon. So I'll leave this as is.

187 ↗(On Diff #55740)

Missing linebreak for new sentence here, I will fix in due course.

0mp added a subscriber: 0mp.

Looks good. Thanks for the contribution!

Minor nit: while here, could you replace Qq with Dq as per style.mdoc(5)?

This revision is now accepted and ready to land.Apr 3 2019, 8:45 AM

Qq? there are none of those, only Ql for inline literals (though on second reading some of those are wrong and should have been Dq - will fix).

Use Dq Li in place of most Ql uses, and use Dq alone in place of some uses of Ql that were not in fact literals.

Add missing sentence linebreak.

This revision now requires review to proceed.Apr 3 2019, 9:26 AM
This revision is now accepted and ready to land.Apr 3 2019, 9:37 AM

Who's gonna commit this?

In D19799#424790, @bcr wrote:

Who's gonna commit this?

I can commit it if no one else is going to. Just took PR236970. :)

In D19799#424885, @ygy wrote:
In D19799#424790, @bcr wrote:

Who's gonna commit this?

I can commit it if no one else is going to. Just took PR236970. :)

Go for it!

This revision was automatically updated to reflect the committed changes.