Page MenuHomeFreeBSD
Differential D53589

exterror.9 man page
ClosedPublic
Actions

Authored by kib on Nov 4 2025, 9:55 PM.
Tags
None
Referenced Files
Unknown Object (File)
Thu, Dec 4, 3:08 PM
Unknown Object (File)
Thu, Dec 4, 2:18 PM
Unknown Object (File)
Thu, Dec 4, 4:18 AM
Unknown Object (File)
Mon, Dec 1, 4:25 AM
Unknown Object (File)
Sun, Nov 30, 10:22 PM
Unknown Object (File)
Sun, Nov 30, 3:08 PM
Unknown Object (File)
Sat, Nov 29, 11:36 AM
Unknown Object (File)
Sat, Nov 29, 12:34 AM
View All 44 Files
Subscribers
imp
pauamma_gundo.com
ziaee

Details

Reviewers
emaste
mckusick
pauamma_gundo.com
Group Reviewers
manpages
Commits
rG0eca7fa1c96f: exterror.9 man page
Summary

With help from: mckusick

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

kib created this revision.Nov 4 2025, 9:55 PM
Herald added subscribers: ziaee, imp. · View Herald TranscriptNov 4 2025, 9:55 PM
kib requested review of this revision.Nov 4 2025, 9:55 PM
kib added a comment.Nov 4 2025, 9:58 PM

After this goes in, I will write the documentation for buf(9) and bio(9) BIO_EXTERR. This cannot be done in the same commit, because buf/bio layout are changed by exterr, which seems to prevent the merge into the stable branches (I did not decided yet).

kib updated this revision to Diff 165851.Nov 4 2025, 11:30 PM

Add note about valid contexts.

kib updated this revision to Diff 165852.Nov 5 2025, 12:13 AM

Mention scope of intent.

emaste added a comment.Nov 5 2025, 12:15 AM

Two small grammatical notes, I will try to provide some more in-depth comments soon.

share/man/man9/exterror.9
7

Is this cut and paste from a template man page or did Mark start this one?

50
52
kib marked 3 inline comments as done.Nov 5 2025, 1:07 AM
kib added inline comments.
share/man/man9/exterror.9
7

Rather file copy; indeed.

kib updated this revision to Diff 165863.Nov 5 2025, 1:09 AM
kib marked an inline comment as done.

Fix attribution. Remove two articles.

mckusick added a comment.Fri, Nov 14, 7:22 AM

I have sent you my edit of the manual page in a separate email rather than trying to put all my comments into the above draft.

kib updated this revision to Diff 166435.EditedFri, Nov 14, 9:23 AM

Take edits and language corrections from mckusick.

kib edited the summary of this revision. (Show Details)Fri, Nov 14, 9:24 AM
ziaee added inline comments.Fri, Nov 14, 12:22 PM
share/man/man9/exterror.9
7

You can use the much shorter form shown in style.9 now. It's also on the licensing guide:

https://docs.freebsd.org/en/articles/license-guide/

154

I don't know if this makes sense, but these are kinda an error code interface that's also extant.

kib added inline comments.Fri, Nov 14, 1:15 PM
share/man/man9/exterror.9
7

Well, the 'written by' part should stay. I can strip the text of the license itself perhaps?

154

No sysexits are irrelevant there.

ziaee added inline comments.Fri, Nov 14, 1:18 PM
share/man/man9/exterror.9
7

I think so too

emaste accepted this revision.Fri, Nov 14, 1:26 PM

Looks good, one typo and two suggestions on articles.

share/man/man9/exterror.9
56
77
78
This revision is now accepted and ready to land.Fri, Nov 14, 1:26 PM
kib updated this revision to Diff 166437.Fri, Nov 14, 1:28 PM

Strip the license text

This revision now requires review to proceed.Fri, Nov 14, 1:28 PM
kib marked 6 inline comments as done.Fri, Nov 14, 1:29 PM
kib added inline comments.Fri, Nov 14, 1:35 PM
share/man/man9/exterror.9
78

In fact, is it correct? 'that would, ..., identifies' or identify?

kib updated this revision to Diff 166438.Fri, Nov 14, 1:37 PM

emaste suggestions.
Removed 'would' for 'identifies'

emaste accepted this revision.Fri, Nov 14, 1:39 PM
This revision is now accepted and ready to land.Fri, Nov 14, 1:39 PM
pauamma_gundo.com accepted this revision.Fri, Nov 14, 1:51 PM
pauamma_gundo.com added a subscriber: pauamma_gundo.com.

Minor nits, fixable on commit.

share/man/man9/exterror.9
104
113

I think.

kib updated this revision to Diff 166439.Fri, Nov 14, 1:58 PM
kib marked 2 inline comments as done.

pauamma corrections

This revision now requires review to proceed.Fri, Nov 14, 1:58 PM
This revision was not accepted when it landed; it landed in state Needs Review.Fri, Nov 14, 2:34 PM
Closed by commit rG0eca7fa1c96f: exterror.9 man page (authored by kib). · Explain Why
This revision was automatically updated to reflect the committed changes.
kib added a commit: rG0eca7fa1c96f: exterror.9 man page.

Revision Contents
Changeset List

PathSize
share/
man/
man9/
1 line
153 lines

Diff 165863

View Options

share/man/man9/Makefile

Loading...
View Options

share/man/man9/exterror.9

Loading...