On 4/20/2013 8:19 PM, William Roberts wrote:
As a quick glance, the only thing that stands out is the all caps OBJECT, as well as the fact that its not a very descriptive name. However, this could be just following convention from elsewhere. On Sat, Apr 20, 2013 at 5:03 PM, Dave Quigley <dpquigl@xxxxxxxxxxxxxxx <mailto:dpquigl@xxxxxxxxxxxxxxx>> wrote: semenage-argparse: Now with full help test (except -E) and support for the login subcommand Hello, I added more help text to the semanage-argparse rewrite and also implemented support for login. I would like anyone and everyone to look at the code and give me feedback. I am in no way shape or form a python guru so I'm glad to have any and all constructive feedback. Please let me know what you think. https://github.com/dpquigl/__semanage-argparse/ <https://github.com/dpquigl/semanage-argparse/> Dave -- This message was distributed to subscribers of the selinux mailing list. If you no longer wish to subscribe, send mail to majordomo@xxxxxxxxxxxxx <mailto:majordomo@xxxxxxxxxxxxx> with the words "unsubscribe selinux" without quotes as the message. -- Respectfully, William C Roberts
To I copied and pasted those lines exactly as they were from the original semanage. I can definitely try to make them something more descriptive.
Dave -- This message was distributed to subscribers of the selinux mailing list. If you no longer wish to subscribe, send mail to majordomo@xxxxxxxxxxxxx with the words "unsubscribe selinux" without quotes as the message.
