June 2008 - Pharo-dev - lists.pharo.org

Re: [Pharo-project] test run

by Norbert Hartl

On Sun, 2008-06-22 at 15:51 +0200, Adrian Lienhard wrote: > On Jun 22, 2008, at 15:45 , Norbert Hartl wrote: > > > On Sun, 2008-06-22 at 12:42 +0200, Adrian Lienhard wrote: > >> I made 3 more tests pass: > >> > >> - Fixed Monticello to pass MCSnapshotBrowserTest>>testButtonMutex > >> (Monticello-al.317) > >> - Fix FloatTest to not depend on platform for representation of NaN > >> bit pattern (KernelTests-al.67) > >> - Remove test testUnimplementedNonPrimitiveCalls (Tests-al.25) > >> > > Hmmm I'm not sure if it is a good idea to delete this test. It is > > a kind of test like ReleaseTest>>testUndeclared. It isn't critical > > but it helps moving to a clean target. I would opt to keep the test. > > At least I would turn it into a "expected failure" as long as we > > aren't far enough to state this to be totally clean. > > I'm fine with making this test an expected failure if we want to make > it pass in the future (in this case the test should exclude some cases > where it is ok that there are no implementors, like for tests that > compile those methods at runtime). Hmm, I wasn't sure myself if a test case is the right place to check. Maybe those tests are better suitable in a class where they can be exectued. Norbert

June 22, 2008

Re: [Pharo-project] random cleanups

by Norbert Hartl

On Sun, 2008-06-22 at 15:45 +0200, Adrian Lienhard wrote: > On Jun 22, 2008, at 15:36 , Norbert Hartl wrote: > > > On Sat, 2008-06-21 at 16:03 +0200, Adrian Lienhard wrote: > >> The package is in the inbox but as it turns out, the slice refers > >> to a > >> version with another UUID (the file name does not matter for > >> resolving > >> the dependency). Maybe this is because you committed again with the > >> same name, so it could be that the original problem was a different > >> one. > >> > >> Norbert, I suggest you take a fresh image, load the 5 packages from > >> the inbox and then create a new slice package and commit that too. > >> > > I took my old image and saved the slice once more. I tested to loead > > with a fresh image and it works now. > > It does not work... It probably works for you because you have those > packages in your local cache. Can you try what I suggested above? I > think it will work like this. > That is strange as I used I complete fresh directory without any package cache to load it back. Anyway I did all changes again created a new SLICE-RandomRemoval and uploaded that together. Norbert > Adrian > > > > > > > > Norbert > >> HTH, > >> Adrian > >> > >> > >> On Jun 21, 2008, at 13:43 , StÃ©phane Ducasse wrote: > >> > >>> Norbert may be the package is in your local cache but failed to be > >>> uploaded. > >>> > >>> Stef > >>> > >>> On Jun 21, 2008, at 10:12 AM, Norbert Hartl wrote: > >>> > >>>> On Sat, 2008-06-21 at 09:28 +0200, Alexandre Bergel wrote: > >>>>> Hi Norbert, > >>>>> > >>>>> I tried to load your slice, but I got "can't find dependency > >>>>> MorphicExtras-NorbertHartl.38" > >>>>> > >>>>> How such a situation may occur? > >>>>> > >>>> I don't know. I didn't get an error while uploading. I saved > >>>> the package again with the same name. > >>>> > >>>> Norbert > >>>>> Cheers, > >>>>> Alexandre > >>>>> > >>>>> On 20 Jun 2008, at 22:54, Norbert Hartl wrote: > >>>>> > >>>>>> I browsed a little bit through the image and found > >>>>>> odd things. I deleted them in > >>>>>> > >>>>>> SLICE-RandomCleanup > >>>>>> > >>>>>> I think all of them a pretty harmless changes. > >>>>>> > >>>>>> Norbert > >>>>>> > >>>>>> > >>>>>> _______________________________________________ > >>>>>> Pharo-project mailing list > >>>>>> Pharo-project(a)lists.gforge.inria.fr > >>>>>> http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project > >>>>>> > >>>>> > >>>> > >>>> > >>>> _______________________________________________ > >>>> Pharo-project mailing list > >>>> Pharo-project(a)lists.gforge.inria.fr > >>>> http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project > >>>> > >>> > >>> > >>> _______________________________________________ > >>> Pharo-project mailing list > >>> Pharo-project(a)lists.gforge.inria.fr > >>> http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project > >> > >> > >> _______________________________________________ > >> Pharo-project mailing list > >> Pharo-project(a)lists.gforge.inria.fr > >> http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project > > > > > > _______________________________________________ > > Pharo-project mailing list > > Pharo-project(a)lists.gforge.inria.fr > > http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project > > > _______________________________________________ > Pharo-project mailing list > Pharo-project(a)lists.gforge.inria.fr > http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project

June 22, 2008

Re: [Pharo-project] test run

by Adrian Lienhard

On Jun 22, 2008, at 15:45 , Norbert Hartl wrote: > On Sun, 2008-06-22 at 12:42 +0200, Adrian Lienhard wrote: >> I made 3 more tests pass: >> >> - Fixed Monticello to pass MCSnapshotBrowserTest>>testButtonMutex >> (Monticello-al.317) >> - Fix FloatTest to not depend on platform for representation of NaN >> bit pattern (KernelTests-al.67) >> - Remove test testUnimplementedNonPrimitiveCalls (Tests-al.25) >> > Hmmm I'm not sure if it is a good idea to delete this test. It is > a kind of test like ReleaseTest>>testUndeclared. It isn't critical > but it helps moving to a clean target. I would opt to keep the test. > At least I would turn it into a "expected failure" as long as we > aren't far enough to state this to be totally clean. I'm fine with making this test an expected failure if we want to make it pass in the future (in this case the test should exclude some cases where it is ok that there are no implementors, like for tests that compile those methods at runtime). Adrian

June 22, 2008

Re: [Pharo-project] Re: Missing PluggableTextController andPluggableTextView

by Damien Cassou

On Fri, Jun 20, 2008 at 8:49 PM, Andrew Tween <amtween(a)hotmail.com> wrote: > Hi Alain & Damien, > Thanks to both of you. > > I have merged in the changes... > > http://source.wiresong.ca/ob/OB-Morphic-tween.48.mcz > > http://www.squeaksource.com/shout/Shout.3.15-tween.72.mcz > > Damien, please could you update the Universe Shout entry like you did > before? Thanks. Done. Thank you for the fixes. -- Damien Cassou Peter von der AhÃ©: Â«I'm beginning to see why Gilad wished us good luckÂ». (http://blogs.sun.com/ahe/entry/override_snafu)

June 22, 2008

Re: [Pharo-project] test run

by Norbert Hartl

On Sun, 2008-06-22 at 15:07 +0200, StÃ©phane Ducasse wrote: > Ok I was not aware that alain published packages in the inbox. I will > go over them. > Did norbert published his packages finally? > Yes, the necessary ones weren't problematic. Only the RandomCleanup one which is not essential. Norbert > Stef > > On Jun 22, 2008, at 12:42 PM, Adrian Lienhard wrote: > > > I made 3 more tests pass: > > > > - Fixed Monticello to pass MCSnapshotBrowserTest>>testButtonMutex > > (Monticello-al.317) > > - Fix FloatTest to not depend on platform for representation of NaN > > bit pattern (KernelTests-al.67) > > - Remove test testUnimplementedNonPrimitiveCalls (Tests-al.25) > > > > So now, using the other fixes of Norbert, the only test failing is > > #testUnwindDebuggerWithStep > > > > Cheers, > > Adrian > > > > PS: Stef, it seems that the package versions from the last update > > you made are still in PharoInbox. > > > > On Jun 20, 2008, at 23:11 , Norbert Hartl wrote: > > > >> I applied all changes from the inbox and the test > >> run showed expected results, 2 expected, 4 failures. > >> > >> Norbert > >> > >> > >> _______________________________________________ > >> Pharo-project mailing list > >> Pharo-project(a)lists.gforge.inria.fr > >> http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project > > > > > > _______________________________________________ > > Pharo-project mailing list > > Pharo-project(a)lists.gforge.inria.fr > > http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project > > > > > _______________________________________________ > Pharo-project mailing list > Pharo-project(a)lists.gforge.inria.fr > http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project

June 22, 2008

Re: [Pharo-project] test run

by Norbert Hartl

On Sun, 2008-06-22 at 12:42 +0200, Adrian Lienhard wrote: > I made 3 more tests pass: > > - Fixed Monticello to pass MCSnapshotBrowserTest>>testButtonMutex > (Monticello-al.317) > - Fix FloatTest to not depend on platform for representation of NaN > bit pattern (KernelTests-al.67) > - Remove test testUnimplementedNonPrimitiveCalls (Tests-al.25) > Hmmm I'm not sure if it is a good idea to delete this test. It is a kind of test like ReleaseTest>>testUndeclared. It isn't critical but it helps moving to a clean target. I would opt to keep the test. At least I would turn it into a "expected failure" as long as we aren't far enough to state this to be totally clean. > So now, using the other fixes of Norbert, the only test failing is > #testUnwindDebuggerWithStep > I wrote an email to squeak-dev and got response from Andreas. I'll try to dig into it und come up with a solution. Great what you did anyway. Now we are on the (how is "Zielgerade" called in english ?? :) ) Norbert > Cheers, > Adrian > > PS: Stef, it seems that the package versions from the last update you > made are still in PharoInbox. > > On Jun 20, 2008, at 23:11 , Norbert Hartl wrote: > > > I applied all changes from the inbox and the test > > run showed expected results, 2 expected, 4 failures. > > > > Norbert > > > > > > _______________________________________________ > > Pharo-project mailing list > > Pharo-project(a)lists.gforge.inria.fr > > http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project > > > _______________________________________________ > Pharo-project mailing list > Pharo-project(a)lists.gforge.inria.fr > http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project

June 22, 2008

Re: [Pharo-project] random cleanups

by Adrian Lienhard

On Jun 22, 2008, at 15:36 , Norbert Hartl wrote: > On Sat, 2008-06-21 at 16:03 +0200, Adrian Lienhard wrote: >> The package is in the inbox but as it turns out, the slice refers >> to a >> version with another UUID (the file name does not matter for >> resolving >> the dependency). Maybe this is because you committed again with the >> same name, so it could be that the original problem was a different >> one. >> >> Norbert, I suggest you take a fresh image, load the 5 packages from >> the inbox and then create a new slice package and commit that too. >> > I took my old image and saved the slice once more. I tested to loead > with a fresh image and it works now. It does not work... It probably works for you because you have those packages in your local cache. Can you try what I suggested above? I think it will work like this. Adrian > > > Norbert >> HTH, >> Adrian >> >> >> On Jun 21, 2008, at 13:43 , StÃ©phane Ducasse wrote: >> >>> Norbert may be the package is in your local cache but failed to be >>> uploaded. >>> >>> Stef >>> >>> On Jun 21, 2008, at 10:12 AM, Norbert Hartl wrote: >>> >>>> On Sat, 2008-06-21 at 09:28 +0200, Alexandre Bergel wrote: >>>>> Hi Norbert, >>>>> >>>>> I tried to load your slice, but I got "can't find dependency >>>>> MorphicExtras-NorbertHartl.38" >>>>> >>>>> How such a situation may occur? >>>>> >>>> I don't know. I didn't get an error while uploading. I saved >>>> the package again with the same name. >>>> >>>> Norbert >>>>> Cheers, >>>>> Alexandre >>>>> >>>>> On 20 Jun 2008, at 22:54, Norbert Hartl wrote: >>>>> >>>>>> I browsed a little bit through the image and found >>>>>> odd things. I deleted them in >>>>>> >>>>>> SLICE-RandomCleanup >>>>>> >>>>>> I think all of them a pretty harmless changes. >>>>>> >>>>>> Norbert >>>>>> >>>>>> >>>>>> _______________________________________________ >>>>>> Pharo-project mailing list >>>>>> Pharo-project(a)lists.gforge.inria.fr >>>>>> http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project >>>>>> >>>>> >>>> >>>> >>>> _______________________________________________ >>>> Pharo-project mailing list >>>> Pharo-project(a)lists.gforge.inria.fr >>>> http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project >>>> >>> >>> >>> _______________________________________________ >>> Pharo-project mailing list >>> Pharo-project(a)lists.gforge.inria.fr >>> http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project >> >> >> _______________________________________________ >> Pharo-project mailing list >> Pharo-project(a)lists.gforge.inria.fr >> http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project > > > _______________________________________________ > Pharo-project mailing list > Pharo-project(a)lists.gforge.inria.fr > http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project

June 22, 2008

Re: [Pharo-project] random cleanups

by Norbert Hartl

On Sat, 2008-06-21 at 16:03 +0200, Adrian Lienhard wrote: > The package is in the inbox but as it turns out, the slice refers to a > version with another UUID (the file name does not matter for resolving > the dependency). Maybe this is because you committed again with the > same name, so it could be that the original problem was a different one. > > Norbert, I suggest you take a fresh image, load the 5 packages from > the inbox and then create a new slice package and commit that too. > I took my old image and saved the slice once more. I tested to loead with a fresh image and it works now. Norbert > HTH, > Adrian > > > On Jun 21, 2008, at 13:43 , StÃ©phane Ducasse wrote: > > > Norbert may be the package is in your local cache but failed to be > > uploaded. > > > > Stef > > > > On Jun 21, 2008, at 10:12 AM, Norbert Hartl wrote: > > > >> On Sat, 2008-06-21 at 09:28 +0200, Alexandre Bergel wrote: > >>> Hi Norbert, > >>> > >>> I tried to load your slice, but I got "can't find dependency > >>> MorphicExtras-NorbertHartl.38" > >>> > >>> How such a situation may occur? > >>> > >> I don't know. I didn't get an error while uploading. I saved > >> the package again with the same name. > >> > >> Norbert > >>> Cheers, > >>> Alexandre > >>> > >>> On 20 Jun 2008, at 22:54, Norbert Hartl wrote: > >>> > >>>> I browsed a little bit through the image and found > >>>> odd things. I deleted them in > >>>> > >>>> SLICE-RandomCleanup > >>>> > >>>> I think all of them a pretty harmless changes. > >>>> > >>>> Norbert > >>>> > >>>> > >>>> _______________________________________________ > >>>> Pharo-project mailing list > >>>> Pharo-project(a)lists.gforge.inria.fr > >>>> http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project > >>>> > >>> > >> > >> > >> _______________________________________________ > >> Pharo-project mailing list > >> Pharo-project(a)lists.gforge.inria.fr > >> http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project > >> > > > > > > _______________________________________________ > > Pharo-project mailing list > > Pharo-project(a)lists.gforge.inria.fr > > http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project > > > _______________________________________________ > Pharo-project mailing list > Pharo-project(a)lists.gforge.inria.fr > http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project

June 22, 2008

Re: [Pharo-project] For milestone 2

by StÃ©phane Ducasse

On Jun 22, 2008, at 2:11 PM, Lukas Renggli wrote: >> I do not get your last sentence. What kind of integration? > > Most prominent, Monticello dispatches through PackageInfo to find out > what parts of the system belong to that package. Earlier versions of > Seaside used this mechanism to have overlapping packages (e.g. for > specific platforms). Ok but I think that this part should be change in the future to get first class package. > Furthermore PackageInfo defines a mechanism to > define pre-load, post-load, pre-unload and post-unload scripts. So I will hook into packageInfo. This should be one line change for what I have done (nothing exciting for now). > > > Cheers, > Lukas > > -- > Lukas Renggli > http://www.lukas-renggli.ch > > _______________________________________________ > Pharo-project mailing list > Pharo-project(a)lists.gforge.inria.fr > http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project >

June 22, 2008

Re: [Pharo-project] test run

by StÃ©phane Ducasse

Ok I was not aware that alain published packages in the inbox. I will go over them. Did norbert published his packages finally? Stef On Jun 22, 2008, at 12:42 PM, Adrian Lienhard wrote: > I made 3 more tests pass: > > - Fixed Monticello to pass MCSnapshotBrowserTest>>testButtonMutex > (Monticello-al.317) > - Fix FloatTest to not depend on platform for representation of NaN > bit pattern (KernelTests-al.67) > - Remove test testUnimplementedNonPrimitiveCalls (Tests-al.25) > > So now, using the other fixes of Norbert, the only test failing is > #testUnwindDebuggerWithStep > > Cheers, > Adrian > > PS: Stef, it seems that the package versions from the last update > you made are still in PharoInbox. > > On Jun 20, 2008, at 23:11 , Norbert Hartl wrote: > >> I applied all changes from the inbox and the test >> run showed expected results, 2 expected, 4 failures. >> >> Norbert >> >> >> _______________________________________________ >> Pharo-project mailing list >> Pharo-project(a)lists.gforge.inria.fr >> http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project > > > _______________________________________________ > Pharo-project mailing list > Pharo-project(a)lists.gforge.inria.fr > http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project >

June 22, 2008