Also remove the "JUJU:" prefix and the automatic
command prefix and name which was just repeating
what's already in the message itself.
So now, instead of:
ERROR JUJU:jujutest:blah jujutest blah command failed: BAM!
ERROR JUJU:juju:bootstrap juju bootstrap command failed: dummy.Bootstrap
is broken
INFO JUJU:test hello
We'll have just:
ERROR command failed: BAM!
INFO hello
ERROR command failed: dummy.Bootstrap is broken
The timestamp with format like 2013/04/12 09:59:16
is still there, before the message.
*** Submitted:
deuglify logging for main packages.
Removed "cmd/juju*: " badge from all commands.
Also remove the "JUJU:" prefix and the automatic
command prefix and name which was just repeating
what's already in the message itself.
So now, instead of:
ERROR JUJU:jujutest:blah jujutest blah command failed: BAM!
ERROR JUJU:juju:bootstrap juju bootstrap command failed: dummy.Bootstrap
is broken
INFO JUJU:test hello
We'll have just:
ERROR command failed: BAM!
INFO hello
ERROR command failed: dummy.Bootstrap is broken
The timestamp with format like 2013/04/12 09:59:16
is still there, before the message.
R=rog, dfc /codereview. appspot. com/8674043
CC=
https:/
https:/ /codereview. appspot. com/8674043/ diff/6001/ cmd/supercomman d_test. go d_test. go (right):
File cmd/supercomman
https:/ /codereview. appspot. com/8674043/ diff/6001/ cmd/supercomman d_test. go#newcode112 d_test. go:112: c.Assert( bufferString( ctx.Stderr) ,
cmd/supercomman
Matches, `^.* ERROR jujutest blah command failed: BAM!
On 2013/04/12 08:27:52, rog wrote:
> i wonder if "command" is necessary here.
> ERROR jujutest blah failed: BAM!
> would perhaps be better i think.
As agreed online, I'll leave just "ERROR command failed: BAM!"
https:/ /codereview. appspot. com/8674043/