[email protected] wrote:
On Mon, Aug 03, 2009 at 02:13:56PM -0700, Danek Duvall wrote:
On Mon, Aug 03, 2009 at 01:50:26PM -0700, [email protected] wrote:
Folks,
I have another small code review. This fixes a couple of transport
issues, a traceback in the CLI, and provides more information when we
fail assertions in the progress tracker. The total change here is less
than 100 lines.
http://cr.opensolaris.org/~johansen/webrev-10411/
progress.py:
- Is the base ProgressTracker class the right place to put messaging? I
would expect this to go in FancyUNIXPT and CommandLinePT, but not the
others. This raises the question of whether FUPT should inherit from
CLPT.
Given that the traceback from failing the assertion is going to go to
stderr, I assumed that it wouldn't be a problem to have the actual
numbers get printed there too. Am I missing something?
The GUI.
--
Shawn Walker
_______________________________________________
pkg-discuss mailing list
[email protected]
http://mail.opensolaris.org/mailman/listinfo/pkg-discuss
