- Pharo-dev - lists.pharo.org

Re: [Pharo-project] shortcuts in PluggableTextMorph

by StÃ©phane Ducasse

We should really change that. Benjamin this would a good case for the Summertalk Stef On Jul 14, 2011, at 11:33 AM, Lukas Renggli wrote: > Probably not :-) > > Whenever you want to change something in the default behavior of > PluggableList/Editor you need to subclass many classes. > > Lukas > > On Thursday, 14 July 2011, Tudor Girba <tudor(a)tudorgirba.com> wrote: >> Hi, >> >> Ok, I found a solution (inspired by OB): >> >> - >> GLMPluggableTextMorph subclasses PluggableTextMorph >> - >> GLMTextMorphForEditView subclasses TextMorphForEditView >> - >> GLMPluggableTextMorph>>textMorphClass >> ^ GLMTextMorphForEditView >> - >> GLMTextMorphForEditView>>keyStroke: anEvent >> | isHandledByModel | >> isHandledByModel := self editView model keystroke: anEvent from: self editView. >> isHandledByModel ifFalse: [ super keyStroke: anEvent ] >> >> This is messy, to put it mildly :). Are there other options? >> >> Cheers, >> Doru >> >> >> On 13 Jul 2011, at 13:07, Lukas Renggli wrote: >> >>> What about using the keyDown event then? >>> >>> Lukas >>> >>> On 13 July 2011 13:05, Tudor Girba <tudor(a)tudorgirba.com> wrote: >>>> Yes. But as I said, the value of "self text" is already modified when it gets to the #keystroke:from: >>>> >>>> GLMTextModel>>keystroke: anEvent from: aMorph >>>> self text inspect. >>>> ^ true >>>> >>>> So, I am stuck again :(. >>>> >>>> Doru >>>> >>>> >>>> >>>> On 13 Jul 2011, at 11:22, Lukas Renggli wrote: >>>> >>>>> And you return true, if you handled the request? >>>>> >>>>> On 13 July 2011 11:18, Tudor Girba <tudor(a)tudorgirba.com> wrote: >>>>>> >>>>>> On 13 Jul 2011, at 11:08, Lukas Renggli wrote: >>>>>> >>>>>>>> This is what I am doing right now and it does not work: >>>>>>>> (PluggableTextMorph on: textModel ...) >>>>>>>> onKeyStrokeSend: #keystroke:from: to: textModel >>>>>>>> ... >>>>>>> >>>>>>> You have to attach your handler on the text morph itself, not on the >>>>>>> pluggable morph which is just a wrapper. >>>>>> >>>>>> This is just a helper method in PluggableTextMorph that does what you propose: >>>>>> >>>>>> PluggableTextMorph>>onKeyStrokeSend: sel to: recipient >>>>>> textMorph on: #keyStroke send: sel to: recipient. >>>>>> >>>>>> But this still does not seem to work. >>>>>> >>>>>> Cheers, >>>>>> Doru >>>>>> >>>>>> >>>>>> >>>>>>> Lukas >>>>>>> >>>>>>> -- >>>>>>> Lukas Renggli >>>>>>> www.lukas-renggli.ch >>>>>>> >>>>>> >>>>>> -- >>>>>> www.tudorgirba.com >>>>>> >>>>>> "Speaking louder won't make the point worthier." >>>>>> >>>>>> >>>>>> >>>>> >>>>> >>>>> >>>>> -- >>>>> Lukas Renggli >>>>> www.lukas-renggli.ch >>>>> >>>> >>>> -- >>>> www.tudorgirba.com >>>> >>>> "Be rather willing to give than demanding to get." >>>> >>>> >>>> >>>> >>>> >>> >>> >>> >>> -- >>> Lukas Renggli >>> www.lukas-renggli.ch >>> >> >> -- >> www.tudorgirba.com >> >> "Not knowing how to do something is not an argument for how it cannot be done." >> >> >> > > -- > Lukas Renggli > www.lukas-renggli.ch >

July 14, 2011

Re: [Pharo-project] Tests package is missing in the Pharo14 repository

by Pavel Krivanek

It was the only one, Marcus probably missed it. -- Pavel On Thu, Jul 14, 2011 at 12:31 PM, StÃ©phane Ducasse <stephane.ducasse(a)inria.fr> wrote: > this is strange because I thought marcus copy them all. > > Stef > > On Jul 14, 2011, at 10:37 AM, Pavel Krivanek wrote: > >> ok, thank you, I hope that no other package is missing. >> >> -- Pavel >> >> On Thu, Jul 14, 2011 at 10:34 AM, StÃ©phane Ducasse >> <stephane.ducasse(a)inria.fr> wrote: >>> ok done. >>> >>> pavel I added you as a commiter to pharo1.4 >>> is it ok for you? >>> >>> Stef >>> >>> On Jul 14, 2011, at 10:16 AM, Pavel Krivanek wrote: >>> >>>> Please, >>>> >>>> can somebody copy latests Tests package from Pharo repository to >>>> Pharo14 repository too? >>>> >>>> -- Pavel >>>> >>> >>> >>> >> > > >

July 14, 2011

Re: [Pharo-project] CI down

by Pavel Krivanek

it is different server, for Pharo 1.4 the server https://ci.lille.inria.fr/pharo/ is used. -- Pavel On Thu, Jul 14, 2011 at 12:29 PM, Tudor Girba <tudor(a)tudorgirba.com> wrote: > It works for me. > https://pharo-ic.lille.inria.fr/hudson/ > > Doru > > > On 14 Jul 2011, at 12:26, Pavel Krivanek wrote: > >> The server is not accessible however building works. I got e-mail >> about finished successful build. >> >> -- Pavel >> >> On Thu, Jul 14, 2011 at 9:40 AM, Pavel Krivanek >> <pavel.krivanek(a)gmail.com> wrote: >>> Jenkins server https://ci.lille.inria.fr/pharo/ is not responding. >>> >>> -- Pavel >>> >> > > -- > www.tudorgirba.com > > "Beauty is where we see it." > > > > >

July 14, 2011

Re: [Pharo-project] Tests package is missing in the Pharo14 repository

by StÃ©phane Ducasse

this is strange because I thought marcus copy them all. Stef On Jul 14, 2011, at 10:37 AM, Pavel Krivanek wrote: > ok, thank you, I hope that no other package is missing. > > -- Pavel > > On Thu, Jul 14, 2011 at 10:34 AM, StÃ©phane Ducasse > <stephane.ducasse(a)inria.fr> wrote: >> ok done. >> >> pavel I added you as a commiter to pharo1.4 >> is it ok for you? >> >> Stef >> >> On Jul 14, 2011, at 10:16 AM, Pavel Krivanek wrote: >> >>> Please, >>> >>> can somebody copy latests Tests package from Pharo repository to >>> Pharo14 repository too? >>> >>> -- Pavel >>> >> >> >> >

July 14, 2011

Re: [Pharo-project] CI down

by Tudor Girba

It works for me. https://pharo-ic.lille.inria.fr/hudson/ Doru On 14 Jul 2011, at 12:26, Pavel Krivanek wrote: > The server is not accessible however building works. I got e-mail > about finished successful build. > > -- Pavel > > On Thu, Jul 14, 2011 at 9:40 AM, Pavel Krivanek > <pavel.krivanek(a)gmail.com> wrote: >> Jenkins server https://ci.lille.inria.fr/pharo/ is not responding. >> >> -- Pavel >> > -- www.tudorgirba.com "Beauty is where we see it."

July 14, 2011

Re: [Pharo-project] CI down

by Pavel Krivanek

The server is not accessible however building works. I got e-mail about finished successful build. -- Pavel On Thu, Jul 14, 2011 at 9:40 AM, Pavel Krivanek <pavel.krivanek(a)gmail.com> wrote: > Jenkins server https://ci.lille.inria.fr/pharo/ is not responding. > > -- Pavel >

July 14, 2011

Re: [Pharo-project] Zinc time outs too fast

by StÃ©phane Ducasse

Thanks I will integrate them now. > I'll add a unit test for ZnConnectionTimeout behavior tomorrow, just to make sure it works. > > I am pretty sure it works, the lastest Zn commits: > > Name: Zinc-HTTP-SvenVanCaekenberghe.174 > Author: SvenVanCaekenberghe > Time: 14 July 2011, 9:54:57 am > UUID: bec35859-b638-42c1-9689-3f1d7a540c8b > Ancestors: Zinc-HTTP-SvenVanCaekenberghe.173 > > ZnDefaultServerDelegate>>#echoRequest: added option to delay the response to /echo with a specified number of seconds, as in echo?delay=60 > > Name: Zinc-Tests-SvenVanCaekenberghe.87 > Author: SvenVanCaekenberghe > Time: 14 July 2011, 9:55:47 am > UUID: d1f2d440-8420-4b11-84ec-d79bdc48e16b > Ancestors: Zinc-Tests-SvenVanCaekenberghe.86 > > added ZnClientTests>>#testTimeout to test the correct working of ZnConnectionTimeout > > > > Both making the timeout shorter and forcing a timeout works, as in the test: > > testTimeout > | server | > server := ZnServer on: 1701. > [ > server start. > self assert: server isRunning & server isListening. > self > should: [ ZnConnectionTimeout value: 1 during: [ ZnClient get: 'http://localhost:1701/echo?delay=2' ] ] > raise: ConnectionTimedOut > ] > ensure: [ server stop ] > > As well as the other way around, waiting longer than the default timeout (I can't put this in the tests as it takes way too long for a unit test): > > ZnConnectionTimeout value: 50 during: [ ^ ZnClient get: 'http://localhost:1701/echo?delay=40' ] > > There could still be some deep problem somewhere (you say the default timeout of 30 shows up in stack), but as far as I can see this works. > > I have said this before, ConnectionTimedOut should in 99% of the case be interpreted as ResourceUnavailable, 30 seconds is an eternity for all but a small number of data or processing heavy requests. > > Sven > > PS: There is also always the option of extending the global default timeout > > ZnNetworkingUtils defaultSocketStreamTimeout: 360. > > >

July 14, 2011

[Pharo-project] Reloaded Core tests result

by Pavel Krivanek

Hi, the CI server is currently not working but on my local Jenkins copy I got this result for the Pharo Kernel Reload job: All Failed Tests: >>> Ring.Tests.Kernel.RGClassDefinitionTest.testAddingMethods 0.001 1 >>> Ring.Tests.Kernel.RGClassDefinitionTest.testAsClassDefinition 0.001 1 >>> Ring.Tests.Kernel.RGClassDefinitionTest.testExistingClass 0.0 1 >>> Ring.Tests.Kernel.RGClassDefinitionTest.testNonExistingClass 0.0 1 >>> Ring.Tests.Kernel.RGClassDefinitionTest.testWithComment 0.001 1 >>> Ring.Tests.Kernel.RGClassDefinitionTest.testWithDefaultNamespace 0.0 1 >>> Ring.Tests.Kernel.RGMethodDefinitionTest.testExistingMethodWithClass 0.0 1 >>> Ring.Tests.Kernel.RGMethodDefinitionTest.testExistingMethodWithPointer 0.001 1 >>> Ring.Tests.Kernel.RGMethodDefinitionTest.testExistingMethodWithoutClass 0.0 1 >>> Ring.Tests.Kernel.RGMethodDefinitionTest.testNonExistingMethodWithClass 0.0 1 >>> Ring.Tests.Kernel.RGTraitDefinitionTest.testAddingMethods 0.001 1 >>> Ring.Tests.Kernel.RGTraitDefinitionTest.testExistingTrait 0.0 1 >>> Ring.Tests.Kernel.RGTraitDefinitionTest.testNonExistingClass 0.001 1 On the first sight it is not very good result however it is the same set of Ring related failing tests as in case of the original Pharo image. So we are able to shrink Pharo to small headless image and then load everything back without new failing tests. Cheers, -- Pavel

July 14, 2011

Re: [Pharo-project] shortcuts in PluggableTextMorph

by Lukas Renggli

Probably not :-) Whenever you want to change something in the default behavior of PluggableList/Editor you need to subclass many classes. Lukas On Thursday, 14 July 2011, Tudor Girba <tudor(a)tudorgirba.com> wrote: > Hi, > > Ok, I found a solution (inspired by OB): > > - > GLMPluggableTextMorph subclasses PluggableTextMorph > - > GLMTextMorphForEditView subclasses TextMorphForEditView > - > GLMPluggableTextMorph>>textMorphClass > Â Â Â Â ^ GLMTextMorphForEditView > - > GLMTextMorphForEditView>>keyStroke: anEvent > Â Â Â Â | isHandledByModel | > Â Â Â Â isHandledByModel := self editView model keystroke: anEvent from: self editView. > Â Â Â Â isHandledByModel ifFalse: [ super keyStroke: anEvent ] > > This is messy, to put it mildly :). Are there other options? > > Cheers, > Doru > > > On 13 Jul 2011, at 13:07, Lukas Renggli wrote: > >> What about using the keyDown event then? >> >> Lukas >> >> On 13 July 2011 13:05, Tudor Girba <tudor(a)tudorgirba.com> wrote: >>> Yes. But as I said, the value of "self text" is already modified when it gets to the #keystroke:from: >>> >>> GLMTextModel>>keystroke: anEvent from: aMorph >>> Â Â Â Â self text inspect. >>> Â Â Â Â ^ true >>> >>> So, I am stuck again :(. >>> >>> Doru >>> >>> >>> >>> On 13 Jul 2011, at 11:22, Lukas Renggli wrote: >>> >>>> And you return true, if you handled the request? >>>> >>>> On 13 July 2011 11:18, Tudor Girba <tudor(a)tudorgirba.com> wrote: >>>>> >>>>> On 13 Jul 2011, at 11:08, Lukas Renggli wrote: >>>>> >>>>>>> This is what I am doing right now and it does not work: >>>>>>> Â (PluggableTextMorph on: textModel ...) >>>>>>> Â Â Â Â Â Â Â onKeyStrokeSend: #keystroke:from: to: textModel >>>>>>> Â Â Â Â Â Â Â Â ... >>>>>> >>>>>> You have to attach your handler on the text morph itself, not on the >>>>>> pluggable morph which is just a wrapper. >>>>> >>>>> This is just a helper method in PluggableTextMorph that does what you propose: >>>>> >>>>> PluggableTextMorph>>onKeyStrokeSend: sel to: recipient >>>>> Â Â Â Â textMorph on: #keyStroke send: sel to: recipient. >>>>> >>>>> But this still does not seem to work. >>>>> >>>>> Cheers, >>>>> Doru >>>>> >>>>> >>>>> >>>>>> Lukas >>>>>> >>>>>> -- >>>>>> Lukas Renggli >>>>>> www.lukas-renggli.ch >>>>>> >>>>> >>>>> -- >>>>> www.tudorgirba.com >>>>> >>>>> "Speaking louder won't make the point worthier." >>>>> >>>>> >>>>> >>>> >>>> >>>> >>>> -- >>>> Lukas Renggli >>>> www.lukas-renggli.ch >>>> >>> >>> -- >>> www.tudorgirba.com >>> >>> "Be rather willing to give than demanding to get." >>> >>> >>> >>> >>> >> >> >> >> -- >> Lukas Renggli >> www.lukas-renggli.ch >> > > -- > www.tudorgirba.com > > "Not knowing how to do something is not an argument for how it cannot be done." > > > -- Lukas Renggli www.lukas-renggli.ch

July 14, 2011

Re: [Pharo-project] News about the consortium :)

by Tudor Girba

+1 Doru On 14 Jul 2011, at 11:17, laurent laffont wrote: > Great ! > > Laurent. > > On Thu, Jul 14, 2011 at 10:29 AM, StÃ©phane Ducasse <stephane.ducasse(a)inria.fr> wrote: > Hi guys > > just to let you know that INRIA validated our idea to build a consortium. > We are checking the details with them. The discussion will probably continue in September but this is a good news > since they are ready to finance an engineer. Now we have to sort out everything. > > Stef > -- www.tudorgirba.com "Don't give to get. Just give."

July 14, 2011