On Thu, Oct 11, 2012 at 06:05:22PM +0100, Sam Thursfield wrote:
Lars' new patch uses logging.warning(), while mine uses
self.app.output.write("WARNING: ")
Are we being inconsistent here? As far as I can see,
logging.warning() writes nothing to stdout or stderr, possibly
outside of the 'morph build' command. I've used the alternative so
that the user will actually see the warnings, because they won't be
checking morph.log when using the branch and merge commands.
logging.* goes to the log file IIRC.
It'd be nice to have app.status(msg="blahfoo", warning=True) for producing
user-facing warnings.
Not sure when we'll have time to do that though.
D.
--
Daniel Silverstone
http://www.codethink.co.uk/
Senior Developer GPG 4096/R Key Id: 3CCE BABE 206C 3B69