Page MenuHomeFreeBSD

address pr 202971: portsnap(8): manpage should reference pkg-version(1), not portversion(1)
ClosedPublic

Authored by jgh on Feb 25 2016, 12:44 AM.
Tags
None
Referenced Files
Unknown Object (File)
Dec 31 2023, 9:24 AM
Unknown Object (File)
Dec 31 2023, 9:23 AM
Unknown Object (File)
Dec 20 2023, 1:24 AM
Unknown Object (File)
Nov 15 2023, 10:44 AM
Unknown Object (File)
Nov 6 2023, 2:42 PM
Unknown Object (File)
Nov 3 2023, 1:00 PM
Unknown Object (File)
Nov 2 2023, 6:50 AM
Unknown Object (File)
Oct 22 2023, 6:48 AM
Subscribers

Details

Summary

202971

update porstsnap man page

MFC after 1 week

Test Plan

mandoc -Tlint (clean)
man render (clean)

Diff Detail

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

Event Timeline

jgh retitled this revision from to address pr 202971.
jgh updated this object.
jgh edited the test plan for this revision. (Show Details)
jgh added reviewers: bcr, remko, wblock.
jgh set the repository for this revision to rD FreeBSD doc repository - subversion.
jgh updated this object.
wblock retitled this revision from address pr 202971 to address pr 202971: portsnap(8): manpage should reference pkg-version(1), not portversion(1).Feb 25 2016, 3:04 AM
wblock edited edge metadata.

Please put the title of the bug in the title of the review. All the numbers look the same otherwise.

.Xr pkg-version should be added to the See Also section, too. And since it is from ports and might not be present, .Xr pkg 7 should also be added there. (I think pkg(7) is the base system one of the same name as the one installed from ports.)

jgh edited edge metadata.

address feedback

bdrewery added a reviewer: bdrewery.
bdrewery added a subscriber: bdrewery.
bdrewery added inline comments.
usr.sbin/portsnap/portsnap/portsnap.8
275โ€“276 โ†—(On Diff #13798)

missing commas

This revision now requires changes to proceed.Feb 26 2016, 9:17 PM
jgh edited edge metadata.

address feedback, good catch.

wblock added a reviewer: wblock.
wblock added a subscriber: wblock.

Looks okay from here, but there is no context for full checking. Please check with igor and mandoc before commit. Thanks!

This revision was automatically updated to reflect the committed changes.