[bzr-gtk: MERGE] selftest with notification

Aaron Bentley aaron.bentley at utoronto.ca
Mon Jul 23 18:51:13 BST 2007


-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

Jelmer Vernooij wrote:
> On Mon, 2007-07-23 at 13:19 -0400, Aaron Bentley wrote:
>> Here's a patch to bzr-gtk to add notification to the selftest command.
>> This means you can run bzr gselftest in a terminal, and when it
>> terminates, it will notify you that it's finished.
>>
>> Thoughts?
> A commit message appears to be missing, or is that the fault of the
> merge directive format?

The metadata is in the bundle, base64 encoded to protect it from mail
mangling (e.g. by Evolution).  The only data outside the bundle is a
description of how to perform the merge.

> Why did you change the base of cmd_gstatus() from GTKCommand to
> Command ?

User error.  I've already corrected that.

> Other than that, patch looks good - would be nice to have in :-)

Done.

> Eventually I think gselftest should show a GUI window to run the tests
> (perhaps using tribunal, once it's finished?). If you'd want a separate
> command that does just the GUI notification but prints output to the
> console, I think it should be done as an additional argument to "bzr
> selftest" (bzr selftest --gui-notify ?).

I think I'd be happy with a full gui for test running.  Especially if
that would separate the traceback out from the logging.

Aaron
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.6 (GNU/Linux)
Comment: Using GnuPG with Mozilla - http://enigmail.mozdev.org

iD8DBQFGpOqR0F+nu1YWqI0RArN3AJ9adf77KSJas9w/SW+xQ/v+O+Qg0gCePAgi
Yjb6ds6rf/QjD3uqgsG4Kmo=
=dHBo
-----END PGP SIGNATURE-----



More information about the bazaar mailing list