[Freeipa-devel] [PATCH] 0040-0042 Fedora packages fixes merge

Rob Crittenden rcritten at redhat.com
Thu Feb 9 18:24:35 UTC 2012


Alexander Bokovoy wrote:
> On Mon, 06 Feb 2012, Rob Crittenden wrote:
>> Seems to work ok, Fix this and you have ACK x3:
>>
>> pylint error:
>>
>> init/systemd/freeipa-systemd-upgrade:38: [E0602] Undefined variable
>> 'IPAConfigError'
>>
>> My fix:
>>
>> +++ b/init/systemd/freeipa-systemd-upgrade
>> @@ -35,7 +35,7 @@ def convert_java_link(foo, topdir, filepaths):
>>   # 0. Init config
>>   try:
>>       config.init_config()
>> -except IPAConfigError, e:
>> +except config.IPAConfigError, e:
>>       # No configured IPA install, no need to upgrade anything
>>       exit(0)
>>
> Did the same. Attached new version. I'm not attaching other patches as
> they are intact.
>

ACK, pushed all three to master, ipa-2-2 and ipa-2-1

rob




More information about the Freeipa-devel mailing list