On 10/19/2012 09:45 AM, Petr Viktorin wrote:
> On 10/18/2012 07:20 PM, John Dennis wrote:
>> On 10/18/2012 05:06 AM, Petr Viktorin wrote:
>>> This looks much better. I found one more issue, though.
>>>
>>>> +        if detail is not None:
>>>> +            err_msg += ' (%s)' % detail
>>>
>>> Here I get TypeError: unsupported operand type(s) for +=: 'Gettext' and
>>> 'unicode'.
>>> Until our Gettext class supports addition (part of #3188), please use
>>> `err_msg = u'%s (%s)' % (err_msg, detail)` instead.
>>
>> Good catch, fixed. New patch attached.
>>
> 
> It works fine now, thanks. ACK
> 

Pushed to master, ipa-3-0.

Martin

_______________________________________________
Freeipa-devel mailing list
Freeipa-devel@redhat.com
https://www.redhat.com/mailman/listinfo/freeipa-devel

Reply via email to