[Freeipa-devel] [PATCH] 956 user lockout status
Martin Kosek
mkosek at redhat.com
Fri Mar 2 14:24:44 UTC 2012
On Fri, 2012-03-02 at 08:46 -0500, Rob Crittenden wrote:
> Martin Kosek wrote:
> > 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
> >
>
> Right, it was just the new changes. All squashed together now.
>
> rob
The new changes are ok, dates and default values are fine.
But it seems like you squashed it with a wrong patch, its again raising
an error when any master is not reachable (you can just interdiff 956-2
and 956-3 to see these malicious changes).
Martin
More information about the Freeipa-devel
mailing list