Page MenuHomeFreeBSD

Document cap_dns service.
ClosedPublic

Authored by oshogbo on Jan 4 2018, 8:08 AM.
Tags
None
Referenced Files
F103767618: D13762.diff
Fri, Nov 29, 4:19 AM
Unknown Object (File)
Mon, Nov 25, 3:45 PM
Unknown Object (File)
Oct 6 2024, 6:02 AM
Unknown Object (File)
Oct 5 2024, 10:03 AM
Unknown Object (File)
Sep 9 2024, 2:11 AM
Unknown Object (File)
Sep 8 2024, 10:54 PM
Unknown Object (File)
Sep 8 2024, 4:35 PM
Unknown Object (File)
Sep 8 2024, 11:22 AM
Subscribers

Details

Diff Detail

Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

A few suggestions and corrections. Thanks for writing the man page!

lib/libcasper/services/cap_dns/cap_dns.3
38

s/capabilitie/capability/

70

s/value/values/

93

s/on/one/

97

I would remove the "the" here and rewrite it to have it read:

"The following example first opens capability to casper and then uses this capability to create a new capability to the system.dns casper service and uses the latter capability to resolve an IP address.

108

I would remove "the" here, unless you add "service" after Casper.

Thank you @bcr for reviewing it!
I bet it was less painful to write it then to review it :)

Haha, yeah. But only because you have a certain Goat looking over your shoulder when writing it. :-)

Macro goatforit:

This revision is now accepted and ready to land.Jan 4 2018, 8:43 AM
brueffer added a subscriber: brueffer.
brueffer added inline comments.
lib/libcasper/services/cap_dns/cap_dns.3
37

geting -> getting

49

The cap_getnameinfo() signature is missing.

51

The functions

65

Not sure I understand this sentence correctly; is assume the system.dns connection is required for all four cap_* variants?

Maybe something like this?

and
.Xr getnameinfo 2 ,
and expect that a connection to the
.Nm system.dns
service is provided.

69

fallowing -> the following

73

Probably .Va instead of .Dv since it's a variable?

77

needs a space between NAME and .

78

.Dv ADDR

83

are allowed.

90

function is allowed

92

family -> families

96

a capability

148

All four entries above need a space between "3" and ","

This revision now requires changes to proceed.Jan 4 2018, 10:59 AM

After review from @bruffer .
After you review I also noticed I forgot to document to other functions, really sorry for my sloppiness here.
I also update a example.

oshogbo added inline comments.
lib/libcasper/services/cap_dns/cap_dns.3
65

I'm trying to say that the different between API of the get* and cap_get* are only in the one additional parameter with is connection to the system.dns.

73

I'm really not sure about that.
.Va is an funtion argument
And the "type" here is an value of argument.
nvlist_add_type(nvl, "type", "ADDR");

Good progress! Hopefully these are the last comments.

lib/libcasper/services/cap_dns/cap_dns.3
65

Ah! "expect -> except" below then :-)

Also "need" -> "needs"

73

Sounds like .Va would be OK then. .Dv is more for constants and other explicit values.

85

"for more deatls" -> "section for more details"

94

to limit

109

fallowing -> following

140

a capability

143

Lots of capabilities in one sentence :-) I'm not sure how to make this clearer though...

This revision now requires changes to proceed.Jan 5 2018, 11:38 AM
oshogbo marked 11 inline comments as done.

Thanks @brueffer!

lib/libcasper/services/cap_dns/cap_dns.3
143

Maybe like that?

brueffer requested changes to this revision.Jan 8 2018, 8:09 AM

Two more small changes, then it's ready to land.

lib/libcasper/services/cap_dns/cap_dns.3
72

services -> service

199

.Nm cap_dns

This revision now requires changes to proceed.Jan 8 2018, 8:09 AM

Thanks for your patience :-)

This revision is now accepted and ready to land.Jan 8 2018, 9:07 AM

Thank you @brueffer and @bcr for dealing with my poor English :)

This revision was automatically updated to reflect the committed changes.