[Freeipa-devel] [PATCH] 956 user lockout status
Martin Kosek
mkosek at redhat.com
Fri Mar 2 08:29:17 UTC 2012
On Thu, 2012-03-01 at 16:26 -0500, Rob Crittenden wrote:
> Martin Kosek wrote:
> > On Wed, 2012-02-29 at 11:20 +0100, Petr Viktorin wrote:
> >> On 02/27/2012 06:31 PM, Martin Kosek wrote:
> >>>
> >>> 4) Minor change:
> >>> - except Exception:
> >>> + except:
> >>>
> >>
> >> Don't do that. It would for example disable Ctrl+C by trapping
> >> KeyboardInterrupt.
> >>
> >> PEP8 has a paragraph on this, search for 'except Exception:'
> >>
> >>
> >
> > Good to know, thanks. Rob, in that case please ignore issue #4.
> >
> > Martin
>
> Updated patch attached.
>
> rob
This does not look like the right patch.
Martin
More information about the Freeipa-devel
mailing list