Re: [Freeipa-devel] [PATCHES 0442-0449] Pylint: sunday code cleanup

2016-03-22 Thread Martin Basti
On 21.03.2016 16:52, Petr Spacek wrote: On 21.3.2016 13:50, Lukas Slebodnik wrote: On (21/03/16 12:30), Martin Basti wrote: On 21.03.2016 10:33, Christian Heimes wrote: On 2016-03-21 10:29, Petr Spacek wrote: On 20.3.2016 21:56, Martin Basti wrote: Patches attached. I do not really like f

Re: [Freeipa-devel] [PATCHES 0442-0449] Pylint: sunday code cleanup

2016-03-21 Thread Petr Spacek
On 21.3.2016 13:50, Lukas Slebodnik wrote: > On (21/03/16 12:30), Martin Basti wrote: >> On 21.03.2016 10:33, Christian Heimes wrote: >>> On 2016-03-21 10:29, Petr Spacek wrote: On 20.3.2016 21:56, Martin Basti wrote: > Patches attached. I do not really like freeipa-mbasti-0442-p

Re: [Freeipa-devel] [PATCHES 0442-0449] Pylint: sunday code cleanup

2016-03-21 Thread Lukas Slebodnik
On (21/03/16 12:30), Martin Basti wrote: >On 21.03.2016 10:33, Christian Heimes wrote: >>On 2016-03-21 10:29, Petr Spacek wrote: >>>On 20.3.2016 21:56, Martin Basti wrote: Patches attached. >>>I do not really like >>>freeipa-mbasti-0442-pylint-remove-bare-except >>>because it replaces most of >

Re: [Freeipa-devel] [PATCHES 0442-0449] Pylint: sunday code cleanup

2016-03-21 Thread Martin Basti
On 21.03.2016 10:33, Christian Heimes wrote: On 2016-03-21 10:29, Petr Spacek wrote: On 20.3.2016 21:56, Martin Basti wrote: Patches attached. I do not really like freeipa-mbasti-0442-pylint-remove-bare-except because it replaces most of try: ... except: with try: ... except Exception:

Re: [Freeipa-devel] [PATCHES 0442-0449] Pylint: sunday code cleanup

2016-03-21 Thread Christian Heimes
On 2016-03-21 10:29, Petr Spacek wrote: > On 20.3.2016 21:56, Martin Basti wrote: >> Patches attached. > > I do not really like > freeipa-mbasti-0442-pylint-remove-bare-except > because it replaces most of > > try: ... except: > > with > > try: ... except Exception: > > > which AFAIK does not

Re: [Freeipa-devel] [PATCHES 0442-0449] Pylint: sunday code cleanup

2016-03-21 Thread Petr Spacek
On 20.3.2016 21:56, Martin Basti wrote: > Patches attached. I do not really like freeipa-mbasti-0442-pylint-remove-bare-except because it replaces most of try: ... except: with try: ... except Exception: which AFAIK does not add any value. It would be better to replace Exception with more spe

[Freeipa-devel] [PATCHES 0442-0449] Pylint: sunday code cleanup

2016-03-20 Thread Martin Basti
Patches attached. From e7a0809e0e202bd3729f2408fbc9855d9a7f10d4 Mon Sep 17 00:00:00 2001 From: Martin Basti Date: Fri, 11 Mar 2016 19:51:07 +0100 Subject: [PATCH 1/8] pylint: remove bare except Bare except should not be used. --- client/ipa-client-install| 4 ++-- install/tools