[Freeipa-devel] [PATCH] display name as separate attributes in ipa-finduser
Rob Crittenden
rcritten at redhat.com
Thu Aug 21 15:21:02 UTC 2008
Martin Nagy wrote:
> On Thu, 21 Aug 2008 10:37:24 -0400, Rob Crittenden
> <rcritten at redhat.com> wrote:
>> + try:
>> + l = attr.index('givenname')
>> + try:
>> + attr.remove('sn')
>> + attr.insert(l+1, 'sn')
>> + except ValueError:
>> + pass
>> + except ValueError:
>> + pass
>
> Please remove the nested try, the code will be equivalent.
>
Updated patch attached.
rob
-------------- next part --------------
An embedded and charset-unspecified text was scrubbed...
Name: freeipa-78-name.patch2
URL: <http://listman.redhat.com/archives/freeipa-devel/attachments/20080821/cc13834b/attachment.ksh>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: smime.p7s
Type: application/x-pkcs7-signature
Size: 3245 bytes
Desc: S/MIME Cryptographic Signature
URL: <http://listman.redhat.com/archives/freeipa-devel/attachments/20080821/cc13834b/attachment.bin>
More information about the Freeipa-devel
mailing list