Re: [Freeipa-devel] [PATCHES 21-22] ipautil log messages and API version to env

2014-06-09 Thread Petr Viktorin
On 06/06/2014 05:02 PM, Gabe Alford wrote: Patch 21: Update per recommendation Patch 22: Added version option as well as updated the manpage. Thanks, Gabe Great! Thanks, ACK, pushed to master: 2a8c509567754877ed0188784d7c38250484be48 In the future, you can put typo fixes in a separate p

Re: [Freeipa-devel] [PATCHES 21-22] ipautil log messages and API version to env

2014-06-06 Thread Gabe Alford
Patch 21: Update per recommendation Patch 22: Added version option as well as updated the manpage. Thanks, Gabe On Fri, Jun 6, 2014 at 5:47 AM, Petr Viktorin wrote: > On 06/06/2014 08:36 AM, Gabe Alford wrote: > >> Hello, >> >> Patch 21: >> This is a patch for https://fedorahosted.org/freeip

Re: [Freeipa-devel] [PATCHES 21-22] ipautil log messages and API version to env

2014-06-06 Thread Petr Viktorin
On 06/06/2014 08:36 AM, Gabe Alford wrote: Hello, Patch 21: This is a patch for https://fedorahosted.org/freeipa/ticket/3724 Nope, doesn't work for me: nolog_replace expects a string. File "/usr/lib/python2.7/site-packages/ipaserver/install/installutils.py", line 639, in run_script re

[Freeipa-devel] [PATCHES 21-22] ipautil log messages and API version to env

2014-06-05 Thread Gabe Alford
Hello, Patch 21: This is a patch for https://fedorahosted.org/freeipa/ticket/3724 Patch 22: This is part of https://fedorahosted.org/freeipa/ticket/4316 and adds the API to 'ipa env' One question that I had for adding 'ipa --version' is if using the default optparse version argument would be suf