Differential D24196

Document the exit status and the message of nologin(8)
ClosedPublic
Actions

Authored by 0mp on Mar 26 2020, 6:14 PM.

Details

Reviewers

bcr

Group Reviewers

manpages

Commits

rS363554: MFC 359967:
rS359967: Document the exit status and the stdout message of nologin(8)

Summary

Document the exit status and the message of nologin(8)

MFC after: 2 weeks

Diff Detail

Repository

rS FreeBSD src repository - subversion

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 30192
Build 27983: arc lint + arc unit

Event Timeline

0mp created this revision.Mar 26 2020, 6:14 PM

Herald added a reviewer: manpages. · View Herald TranscriptMar 26 2020, 6:14 PM

Herald added a subscriber: imp. · View Herald Transcript

Harbormaster completed remote builds in B30118: Diff 69898.Mar 26 2020, 6:14 PM

0mp edited the summary of this revision. (Show Details)Mar 26 2020, 6:15 PM

0mp added a reviewer: bcr.Mar 26 2020, 7:39 PM

According to mdoc(7), the .Li macro is deprecated - should Ql be used instead?

Other than that, it looks good to me, but I don't think I can approve in good conscience.

Add EXAMPLES
Rewrite description

Harbormaster completed remote builds in B30192: Diff 70018.Mar 30 2020, 7:55 PM

In D24196#532232, @debdrup wrote:

According to mdoc(7), the .Li macro is deprecated - should Ql be used instead?

Oh, that's new! Thank you

This revision was not accepted when it landed; it landed in state Needs Review.Apr 15 2020, 1:14 PM

Closed by commit rS359967: Document the exit status and the stdout message of nologin(8) (authored by 0mp). · Explain Why

This revision was automatically updated to reflect the committed changes.

0mp added a commit: rS359967: Document the exit status and the stdout message of nologin(8).

0mp added a commit: rS363554: MFC 359967:.Jul 26 2020, 10:04 AM

yuripv added a subscriber: yuripv.Jul 26 2020, 10:12 AM

yuripv added inline comments.

head/usr.sbin/nologin/nologin.8
47 ↗	(On Diff #70600)	I'm late to the party as always, just noticed the MFC commit, but anyway. This sentence sounds somewhat broken to me, you don't write to a function, rather log something using syslog(3)?
49 ↗	(On Diff #70600)	Erroneous 'a' after 'then'.

Revision Contents
Changeset List

Path

Size

usr.sbin/

nologin/

nologin.8

25 lines

Diff 70018

View Options

usr.sbin/nologin/nologin.8

Document the exit status and the message of nologin(8)ClosedPublicActions