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

[Martin Kosek]

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
-------------- next part --------------
A non-text attachment was scrubbed...
Name: freeipa-mkosek-102-3-fix-automountkey-add-summary.patch
Type: text/x-patch
Size: 2830 bytes
Desc: not available
URL: <http://listman.redhat.com/archives/freeipa-devel/attachments/20110727/e4cd8c74/attachment.bin>