[PATCH] Fix untidy samba-tool exception handling
Jeremy Allison
jra at samba.org
Wed Jan 16 17:46:08 UTC 2019
On Wed, Jan 16, 2019 at 09:41:28AM +0000, Rowland Penny via samba-technical wrote:
> On Wed, 16 Jan 2019 16:35:17 +1300
> Tim Beale via samba-technical <samba-technical at lists.samba.org> wrote:
>
> > During a join, our exception handling was throwing an exception. The
> > resulting samba-tool output wasn't particularly user-friendly. I
> > spotted this problem during a failed CI autobuild run.
> >
> > Attached is a fix. I also raised a bug for backporting it:
> > https://bugzilla.samba.org/show_bug.cgi?id=13747
> >
> > CI link: https://gitlab.com/catalyst-samba/samba/pipelines/43503776
> >
> > Review appreciated, thanks.
> >
>
> Whilst it looks good to me, you have lines greater than 80 columns and
> I sort of thought we were moving from '%s' to .format
Perfect is the enemy of the good in this case :-).
RB+ also.
Jeremy.
More information about the samba-technical
mailing list