[Freeipa-devel] [PATCH 0099] ipa-nis-manage: add status option

Abhijeet Kasurde akasurde at redhat.com
Fri May 20 10:45:20 UTC 2016



On 05/20/2016 04:02 PM, Petr Spacek wrote:
> On 12.5.2016 16:17, Petr Spacek wrote:
>> On 28.4.2016 17:15, Petr Spacek wrote:
>>> On 28.4.2016 14:52, Abhijeet Kasurde wrote:
>>>> Hi Petr,
>>>>
>>>> On 04/25/2016 08:28 PM, Petr Spacek wrote:
>>>>> Hello,
>>>>>
>>>>> ipa-nis-manage: add status option
>>>>>
>>>>> https://bugzilla.redhat.com/show_bug.cgi?id=1329275
>>>>>
>>>>>
>>>>>
>>>> Can you reword the error message here as well ?
>>>>
>>>>       if len(args) != 1:
>>>>           sys.exit("You must specify one action, either enable or disable")
>>>>
>>>> Thanks,
>>>> Abhijeet Kasurde
>>> Good catch!
>> Please review this, thanks.
> Ping, please review it.
>
LGTM. But someone else should approve it.

-- 
Thanks,
Abhijeet Kasurde

IRC: akasurde
http://akasurde.github.io




More information about the Freeipa-devel mailing list