[Freeipa-devel] [PATCH] 102 Fix automountkey-add summary

[Adam Young]

On 07/27/2011 11:13 AM, Rob Crittenden wrote:
> Martin Kosek wrote:
>> On Wed, 2011-07-27 at 10:35 -0400, Rob Crittenden wrote:
>>> Martin Kosek wrote:
>>>> On Wed, 2011-07-27 at 11:28 +0200, Martin Kosek wrote:
>>>>> The summary value was set to invalid primary key - automount map
>>>>> instead of automount key.
>>>>>
>>>>> https://fedorahosted.org/freeipa/ticket/1524
>>>>>
>>>>
>>>> The other commands' summary may be wrong in some cases as well. 
>>>> Updated
>>>> patch fixes summary for all automountkey commands.
>>>>
>>>> Martin
>>>
>>> nack
>>>
>>> ipalib/plugins/automount.py:926: [E0602, automountkey_show.execute]
>>> Undefined variable 'key'
>>>
>>> I think you meant to set result['value'] = options['automountkey']
>>>
>>> rob
>>
>> Ah, that's embarrassing. That happens when one makes "just one small
>> change before commit". Attaching fixed patch.
>>
>> Martin
>
> ack
>
> _______________________________________________
> Freeipa-devel mailing list
> Freeipa-devel at redhat.com
> https://www.redhat.com/mailman/listinfo/freeipa-devel
Before you push:  was it tested against the UI?