[MERGE] Include formatted traceback in SmartMessageHandlerError details.

Andrew Bennetts andrew.bennetts at canonical.com
Tue Jan 6 06:37:37 GMT 2009


John Arbash Meinel wrote:
> Andrew Bennetts wrote:
> > This is just a simple patch to extend SmartMessageHandlerError a little.
> > I've found it very helpful when debugging my work on adding a
> > VersionedFiles.insert_record_stream RPC.
> > 
> > This would also help with diagnosing bugs like
> > <https://bugs.launchpad.net/bzr/+bug/304544>.
> > 
> > -Andrew.
> > 
> > 
> 
> This only effects debugging right? Otherwise I would think that seeing a
> formatted traceback as part of an already formatted traceback would be a
> bit weird.

Right.  The protocol decoder always catches this error, and calls
log_exception_quietly.  So it's in the log file (handy for test failures and
handy for bug reports), but nowhere else.

> Also, shouldn't there be a test in bzrlib/tests/test_errors.py that
> needs updating?

Good point.  Added.

-Andrew.




More information about the bazaar mailing list