On Fri, Feb 14, 2014 at 02:56:41PM -0800, Benoit Chesneau wrote:
> >          self.timeout = self.cfg.timeout
> >          self.proc_name = self.cfg.proc_name
> >  
> > -        if self.cfg.debug:
> > -            self.log.debug("Current configuration:")
> > -            for config, value in sorted(self.cfg.settings.items(),
> > -                    key=lambda setting: setting[1]):
> > -                self.log.debug("  %s: %s", config, value.value)
> > +        self.log.debug("Current configuration:")
> > +        for config, value in sorted(self.cfg.settings.items(),
> > +                key=lambda setting: setting[1]):
> > +            self.log.debug("  %s: %s", config, value.value)
> 
> forget about that, can be added in later commit.

Do you want the pre-log-string-join added as a new commit on this PR?
It seems like a separate issue to me, in which case I think the
current PR is good to go.