Issue 6055 in pharo: [BUG] New System Progress doesn't work when headless

Status: Accepted
Owner: [hidden email]
Labels: Type-Bug

New issue 6055 by [hidden email]: [BUG] New System Progress doesn't  
work when headless
http://code.google.com/p/pharo/issues/detail?id=6055

2.0

When headless and displaying progress, with used to be a no-op, now there  
is a BlockClosure DNU label:. This is because progress used to use a block  
that was pretending to be a class, so #value: would get sent everywhere,  
with the "selector" as the argument (e.g. value: #label). Now that we have  
a real progress object, we are sending #label: explicitly. Since  
CommandLineUIManager wasn't updated, it's still passing a block ([:bar |]),  
which obviously doesn't understand #label:


_______________________________________________
Pharo-bugtracker mailing list
[hidden email]
http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-bugtracker

Updates:
        Status: FixReviewNeeded

Comment #1 on issue 6055 by [hidden email]: [BUG] New System  
Progress doesn't work when headless
http://code.google.com/p/pharo/issues/detail?id=6055

The attached st files fix CommandLineUIManager to use a  
DummySystemProgressItem instead of a block.

There will be more refactoring of the entire system progress framework, but  
let's get it working, one step at a time.

N.B. The DummySystemProgressItem was temporarily placed in its own category  
to avoid making Morphic-Widgets dirty, which would be wiped out as the  
updates to 20119 are loaded

Attachments:
        CommandLineUIManager-progressInitiationExceptionDefaultAction.st  351 bytes
        DummySystemProgressItem.st  369 bytes


_______________________________________________
Pharo-bugtracker mailing list
[hidden email]
http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-bugtracker

Comment #2 on issue 6055 by [hidden email]: [BUG] New System  
Progress doesn't work when headless
http://code.google.com/p/pharo/issues/detail?id=6055

Fix in inbox:  
SLICE-Issue-6055-BUG-New-System-Progress-doesnt-work-when-headless-SeanDeNigris.1

* Create dummy DummySystemProgressItem, which accepts and ignores all  
messages, as a temporary measure to avoid errors while headless
* Patch CommandLineUIManager to use above instead of [:bar| ], which causes  
DNU when updating Pharo headlessly


_______________________________________________
Pharo-bugtracker mailing list
[hidden email]
http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-bugtracker

Updates:
        Status: FixToInclude

Comment #3 on issue 6055 by [hidden email]: [BUG] New System Progress  
doesn't work when headless
http://code.google.com/p/pharo/issues/detail?id=6055

(No comment was entered for this change.)


_______________________________________________
Pharo-bugtracker mailing list
[hidden email]
http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-bugtracker

Updates:
        Labels: Milestone-2.0

Comment #4 on issue 6055 by [hidden email]: [BUG] New System  
Progress doesn't work when headless
http://code.google.com/p/pharo/issues/detail?id=6055

(No comment was entered for this change.)


_______________________________________________
Pharo-bugtracker mailing list
[hidden email]
http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-bugtracker

Updates:
        Status: Integrated

Comment #5 on issue 6055 by [hidden email]: [BUG] New System Progress  
doesn't work when headless
http://code.google.com/p/pharo/issues/detail?id=6055

I forgot to close this


_______________________________________________
Pharo-bugtracker mailing list
[hidden email]
http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-bugtracker