martbab's pull request #59: "Fix BadSyntax exception in ldapupdate.py" was 
opened

PR body:
"""
This complements commit 00d43095da211f542189c95c88fc2e2c32e75565 and fixes two
failing testcases in `ipatests/test_install/test_updates.py`

https://fedorahosted.org/freeipa/ticket/6294
"""

See the full pull-request at https://github.com/freeipa/freeipa/pull/59
... or pull the PR as Git branch:
git remote add ghfreeipa https://github.com/freeipa/freeipa
git fetch ghfreeipa pull/59/head:pr59
git checkout pr59
From 3fabb5b64d4df9ef1a7a00312e5244b92af67eb0 Mon Sep 17 00:00:00 2001
From: Martin Babinsky <mbabi...@redhat.com>
Date: Tue, 6 Sep 2016 11:46:58 +0200
Subject: [PATCH] ldapupdate: Use proper inheritance in BadSyntax exception

https://fedorahosted.org/freeipa/ticket/6294
---
 ipaserver/install/ldapupdate.py | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/ipaserver/install/ldapupdate.py b/ipaserver/install/ldapupdate.py
index 32fa4e2..1b39745 100644
--- a/ipaserver/install/ldapupdate.py
+++ b/ipaserver/install/ldapupdate.py
@@ -83,8 +83,8 @@ def connect(ldapi=False, realm=None, fqdn=None, dm_password=None, pw_name=None):
 class BadSyntax(installutils.ScriptError):
     def __init__(self, value):
         self.value = value
-        self.msg = "LDAPUpdate: syntax error: \n  %s" % value
-        self.rval = 1
+        super(BadSyntax, self).__init__(
+            msg="LDAPUpdate: syntax error: \n  %s" % value, rval=1)
 
     def __str__(self):
         return repr(self.value)
-- 
Manage your subscription for the Freeipa-devel mailing list:
https://www.redhat.com/mailman/listinfo/freeipa-devel
Contribute to FreeIPA: http://www.freeipa.org/page/Contribute/Code

Reply via email to