[PATCH] Added chunkname prefix to some elements of the debug log.

Richard Maw richard.maw at codethink.co.uk
Thu Oct 10 11:04:17 BST 2013


On Thu, Oct 10, 2013 at 10:59:23AM +0100, Richard Ipsum wrote:
> On 04/10/13 15:25, Daniel Firth wrote:
> >      def runcmd_unchecked(self, *args, **kwargs):
> >          return cliapp.runcmd_unchecked(*args, **kwargs)
> >+    def status(self, msg):
> >+        pass
> >+
> >
> Why is this function just a pass?

Presumably because it's not testing the messages produced, and you can't
have a function with an empty body.

I'm more concerned with the change in visible indentation, which usually
means a tab has snuck in, which implies the test suite hasn't been run.



More information about the baserock-dev mailing list