Page MenuHomeFreeBSD
Differential D23905

smbfs(5): Add a basic man page
ClosedPublic
Actions

Authored by gbe on Mar 1 2020, 11:12 AM.
Tags
None
Referenced Files
Unknown Object (File)
Sat, Jul 18, 4:49 AM
Unknown Object (File)
Tue, Jul 14, 1:45 AM
Unknown Object (File)
Sat, Jul 11, 8:41 PM
Unknown Object (File)
Sun, Jul 5, 12:09 AM
Unknown Object (File)
Sat, Jul 4, 8:21 AM
Unknown Object (File)
Sat, Jul 4, 6:20 AM
Unknown Object (File)
Fri, Jul 3, 4:30 AM
Unknown Object (File)
Thu, Jul 2, 9:31 PM
View All Files
Subscribers
bcr
brueffer
imp

Details

Reviewers
brueffer
Group Reviewers
manpages
Commits
rS359731: Add a basic manpage for smbfs(5).
Summary

Add a basic man page for smbfs.

Test Plan

mandoc output review

Diff Detail

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

Event Timeline

gbe created this revision.Mar 1 2020, 11:12 AM
Harbormaster completed remote builds in B29702: Diff 69023.Mar 1 2020, 11:12 AM
gbe added a reviewer: manpages.Mar 1 2020, 11:12 AM
gbe added a comment.Apr 8 2020, 9:49 AM

Could somebody review this differential?

bcr added a subscriber: bcr.Apr 8 2020, 10:26 AM

Looks good. Did you run "mandoc -Tlint" and igor over it?

gbe updated this revision to Diff 70332.Apr 8 2020, 10:34 AM

smbfs(5): Eliminate Whitespaces and Correct Xr Order

Harbormaster completed remote builds in B30357: Diff 70332.Apr 8 2020, 10:34 AM
gbe added a comment.Apr 8 2020, 10:37 AM
In D23905#535330, @bcr wrote:

Looks good. Did you run "mandoc -Tlint" and igor over it?

I just did. The igor tool throws an error message, that the manpage date should be set to today, but I didn't change it, since the manpage was written on the first of march.

brueffer added a subscriber: brueffer.Apr 8 2020, 11:17 AM

Looks good apart from the missing comma.

share/man/man5/smbfs.5
50 ↗(On Diff #70332)

Missing comma. ".Xr smbutil 1 ,"

gbe updated this revision to Diff 70337.Apr 8 2020, 11:40 AM
  • Add mission common sign at the .Xr elements
Harbormaster completed remote builds in B30361: Diff 70337.Apr 8 2020, 11:40 AM
gbe added a comment.Apr 8 2020, 11:41 AM
In D23905#535341, @brueffer wrote:

Looks good apart from the missing comma.

I have corrected this.

brueffer accepted this revision.Apr 8 2020, 7:55 PM
This revision is now accepted and ready to land.Apr 8 2020, 7:55 PM
Closed by commit rS359731: Add a basic manpage for smbfs(5). (authored by brueffer). · Explain WhyApr 8 2020, 8:00 PM
This revision was automatically updated to reflect the committed changes.
brueffer added a commit: rS359731: Add a basic manpage for smbfs(5)..
Herald added a subscriber: imp. · View Herald TranscriptApr 8 2020, 8:00 PM

Revision Contents
Changeset List

PathSize
head/
share/
man/
man5/
1 line
79 lines

Diff 70351

View Options

head/share/man/man5/Makefile

Loading...
View Options

head/share/man/man5/smbfs.5

Loading...