Hi nicolas I tried and it loads :)
Delay startTimerEventLoop. (ScriptLoader new buildConfigurationMapFor: #('Kernel-nice.renggli.301.mcz' 'KernelTests-nice.96.mcz' 'System-Support-nice.11.mcz')) load.
Now I will wait a bit to know what we do with the event rewrite because may be this is too premature and I would like to avoid to have to undo too much work. Stef On Apr 19, 2009, at 9:54 PM, Nicolas Cellier wrote:
My 16 KernelTests-Processes are green :)
Forget all the SLICE-DelayFixFromMantis7321 stuff, it only add pain and complexity... I checked that entire changes related to mantis 7321 list can be loaded with following instructions:
Delay startTimerEventLoop. (ScriptLoader new buildConfigurationMapFor: #('Kernel-nice.renggli.301.mcz' 'KernelTests-nice.96.mcz' 'System-Support-nice.11.mcz')) load.
If ever a problem occur, try again in several steps:
(ScriptLoader new buildConfigurationMapFor: #('Kernel-nice.renggli.299.mcz' 'KernelTests-nice.95.mcz' 'System-Support-nice.11.mcz')) load.
Delay startTimerEventLoop. (ScriptLoader new buildConfigurationMapFor: #('Kernel-nice.renggli.301.mcz' 'KernelTests-nice.96.mcz')) load.
Or even in smaller grain...
Cheers
Nicolas
2009/4/19 Nicolas Cellier <nicolas.cellier.aka.nice@gmail.com>:
OK, I checked that mantis 6576 is superseded by 7321.
To install DelayCleanup.1.cs it is necessary to write a ScriptLoader script like: script363 Delay startTimerEventLoop. (self buildConfigurationMapFor: #('Kernel-nice.renggli.300.mcz')) load
Now, last thing to fix is remove SmallInteger maxVal//2 protection and test.
2009/4/19 Nicolas Cellier <nicolas.cellier.aka.nice@gmail.com>:
Andreas answered that the SmallInteger limit is obsolete since http://bugs.squeak.org/view.php?id=6576 which was included in 3.10
So I will check that mantis 6576+7321, and change the DelayTests>>#testBounds
Then only the PREAMBLE problem will remain. But I see this preamble can simply go in a ScriptLoader script.
2009/4/19 Nicolas Cellier <nicolas.cellier.aka.nice@gmail.com>:
I took a look at http://bugs.squeak.org/view.php?id=7321 and http://code.google.com/p/pharo/issues/detail?id=688
I started publishing PharoInbox/SLICE-DelayFixFromMantis7321- nice to integrate Andreas' changes.
Now I did integrate all but DelayCleanup.1.cs This change set does cure the failing testSemaphore in Pharo, but has two problems: 1) It requires a preamble, and I don't know how they are handled in Monticello 2) It probably does not include change about LargeInteger delays included in 3.9 (Dave kindly pointed me to http://bugs.squeak.org/view.php?id=854) 3) it makes DelayTests>>#testBounds fail
I think that the limit of (SmallInteger maxVal//2) is not necessary... if LargeInteger delay fix is in the image. I can try to fix 2) mixing Andreas and Tim fixes. What about 1) ?
_______________________________________________ Pharo-project mailing list Pharo-project@lists.gforge.inria.fr http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project