Hi Ethan --
A couple of comments for the man page. The rest looks good.
ginnie
ai_manifest.4.txt
----------------------
ln 46: remove one of the "was"
ln 60: a the zone gets installed - remove either "a" or "the"
ln 134: Should this be a separate bullet item? It seems like it goes
with the previous one.
ln 200: what is seen when by running - remove either "when or "by"
ln 209 - 211 and on.... not sure what that is, but I assume it won't
appear in the final.
ln 335: source of of the iSCSI - remove one "of'
ln 549: this sentence seems incomplete
ln 628: be the a swap zvol - remove either 'the" or "a"
ln 636: the same as 628
ln 910: this is really a nit, but "it's" should actually be "its" (no
apostrophe)
ln 929: should be "is inherited"
ln 948: "as" should be "has"
ln 976: same as 948
ln 1045: The order of that the publishers - remove "of" or "that"
ln 1320: default/defaults
On 08/ 2/11 04:45 PM, Ethan Quach wrote:
Can I get a review for the following bugfix:
BugID:
--------
http://monaco.us.oracle.com/detail.jsf?cr=7025229
(7025229 Need an AI manifest man page)
Webrev:
----------
https://cr.opensolaris.org/action/browse/caiman/equach/7025229/webrev/
thanks,
-ethan
_______________________________________________
caiman-discuss mailing list
[email protected]
http://mail.opensolaris.org/mailman/listinfo/caiman-discuss
_______________________________________________
caiman-discuss mailing list
[email protected]
http://mail.opensolaris.org/mailman/listinfo/caiman-discuss
