Doron Fediuck has posted comments on this change. Change subject: engine-config: Adding man page ......................................................................
Patch Set 2: I would prefer that you didn't submit this Douglas, I see 2 issues here need to be handled; 1. Please remove the bz url in the commit message, which has limited visibility. 2. The man page should include usage of the utility, but not the keys' description, which is given when listing the keys. This also creates data duplication as each modification of the keys should be handled here as well, and this is not desirable. -- To view, visit http://gerrit.ovirt.org/11176 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I546d90550ddf53707bc667c18e8c0d44e909593f Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Douglas Schilling Landgraf <[email protected]> Gerrit-Reviewer: Barak Azulay <[email protected]> Gerrit-Reviewer: Doron Fediuck <[email protected]> Gerrit-Reviewer: Douglas Schilling Landgraf <[email protected]> Gerrit-Reviewer: Mark Wu <[email protected]> Gerrit-Reviewer: Ofer Schreiber <[email protected]> Gerrit-Reviewer: Ravi Nori <[email protected]> Gerrit-Reviewer: Yaniv Bronhaim <[email protected]> _______________________________________________ Engine-patches mailing list [email protected] http://lists.ovirt.org/mailman/listinfo/engine-patches
