John,
Might it be better to change the superclass of GUIProgressTracker to progress.NullProgressTracker so that if methods are added to ProgressTracker in the future, you will automatically get an empty implementation of them?

Tom

jmr wrote:
Simple change to add in new methods introduced to ProgressTracker as part of cset 1087 (#8214) which breaks UpdateManager as it implements GUIProgressTracker, derived from ProgressTracker.

http://cr.opensolaris.org/~jmr/pm_8581_01May_1005am/
8581 UM broken by changes in ProgressTracker

1087 changed PM but not UM. A nightly touch test of the GUI apps would be useful to catch this type of thing, I normally do this before a checkin which is how I picked this up. Michal is working on automated tests with Conny so sooner we get these tests landed the better.

JR

_______________________________________________
pkg-discuss mailing list
[email protected]
http://mail.opensolaris.org/mailman/listinfo/pkg-discuss

begin:vcard
fn:Tom Mueller
n:Mueller;Tom
org:Sun Microsystems, Inc.;Update Center Software
adr:;;21915 Hillandale Dr;Elkhorn;NE;68022;USA
email;internet:[email protected]
title:Senior Staff Engineer
tel;work:877-250-4011
tel;fax:877-250-4011
tel;home:402-916-9943
x-mozilla-html:TRUE
version:2.1
end:vcard

_______________________________________________
pkg-discuss mailing list
[email protected]
http://mail.opensolaris.org/mailman/listinfo/pkg-discuss

Reply via email to