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

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

Reply via email to