[PATCH] Fix untidy samba-tool exception handling

Rowland Penny rpenny at samba.org
Wed Jan 16 09:41:28 UTC 2019


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 

Rowland



More information about the samba-technical mailing list