Pharo-dev

By thread

pharo-dev@lists.pharo.org

1 participants
144613 messages

Re: [Pharo-dev] Zamaahh

by Marcus Denker

It is on my TODO to think if we should not send all these automated mails to a dedicated mailinglist. Marcus > On 25 Aug 2019, at 12:10, Laszlo Zsolt Kiss <lzskiss(a)gmail.com> wrote: > > > 2019.08.24., Szo 16:18 dÃ¡tummal <ci-pharo-ci-jenkins2(a)inria.fr <mailto:ci-pharo-ci-jenkins2@inria.fr>> ezt Ãrta: > There is a new Pharo build available! > > The status of the build #671 was: FAILURE. > > The Pull Request #4418 was integrated: "4414-slot-class-defintion-Class-Variables-with-parameters-are-not-printed-correctly" > Pull request url: https://github.com/pharo-project/pharo/pull/4418 <https://github.com/pharo-project/pharo/pull/4418> > > Issue Url: https://github.com/pharo-project/pharo/issues/4414 <https://github.com/pharo-project/pharo/issues/4414> > Build Url: https://ci.inria.fr/pharo-ci-jenkins2/job/Test%20pending%20pull%20request%2… <https://ci.inria.fr/pharo-ci-jenkins2/job/Test%20pending%20pull%20request%2…>

Aug. 25, 2019

Zamaahh

by Laszlo Zsolt Kiss

2019.08.24., Szo 16:18 dÃ¡tummal <ci-pharo-ci-jenkins2(a)inria.fr> ezt Ãrta: > There is a new Pharo build available! > > The status of the build #671 was: FAILURE. > > The Pull Request #4418 was integrated: > "4414-slot-class-defintion-Class-Variables-with-parameters-are-not-printed-correctly" > Pull request url: https://github.com/pharo-project/pharo/pull/4418 > > Issue Url: https://github.com/pharo-project/pharo/issues/4414 > Build Url: > https://ci.inria.fr/pharo-ci-jenkins2/job/Test%20pending%20pull%20request%2… >

Aug. 25, 2019

[Pharo 8.0] Build #672: 4149-instVarNames-is-not-including-instance-variables-coming-from-Stateful-traits

by ci-pharo-ci-jenkins2＠inria.fr

There is a new Pharo build available! The status of the build #672 was: FAILURE. The Pull Request #4367 was integrated: "4149-instVarNames-is-not-including-instance-variables-coming-from-Stateful-traits" Pull request url: https://github.com/pharo-project/pharo/pull/4367 Issue Url: https://github.com/pharo-project/pharo/issues/4149 Build Url: https://ci.inria.fr/pharo-ci-jenkins2/job/Test%20pending%20pull%20request%2…

Aug. 24, 2019

[Pharo 8.0] Build #671: 4414-slot-class-defintion-Class-Variables-with-parameters-are-not-printed-correctly

by ci-pharo-ci-jenkins2＠inria.fr

There is a new Pharo build available! The status of the build #671 was: FAILURE. The Pull Request #4418 was integrated: "4414-slot-class-defintion-Class-Variables-with-parameters-are-not-printed-correctly" Pull request url: https://github.com/pharo-project/pharo/pull/4418 Issue Url: https://github.com/pharo-project/pharo/issues/4414 Build Url: https://ci.inria.fr/pharo-ci-jenkins2/job/Test%20pending%20pull%20request%2…

Aug. 24, 2019

[Pharo 8.0] Build #670: Roassal3Extentions

by ci-pharo-ci-jenkins2＠inria.fr

There is a new Pharo build available! The status of the build #670 was: FAILURE. The Pull Request #4405 was integrated: "Roassal3Extentions" Pull request url: https://github.com/pharo-project/pharo/pull/4405 Issue Url: https://github.com/pharo-project/pharo/issues/Roassal3Extentions Build Url: https://ci.inria.fr/pharo-ci-jenkins2/job/Test%20pending%20pull%20request%2…

Aug. 24, 2019

[Pharo 8.0] Build #669: Class Variables with Default value

by ci-pharo-ci-jenkins2＠inria.fr

There is a new Pharo build available! The status of the build #669 was: FAILURE. The Pull Request #4413 was integrated: "Class Variables with Default value" Pull request url: https://github.com/pharo-project/pharo/pull/4413 Issue Url: https://github.com/pharo-project/pharo/issues/ClassVarWithDefault Build Url: https://ci.inria.fr/pharo-ci-jenkins2/job/Test%20pending%20pull%20request%2…

Aug. 24, 2019

[Pharo 8.0] Build #668: 4403-cannot-change-position-in-a-memory-write-stream

by ci-pharo-ci-jenkins2＠inria.fr

There is a new Pharo build available! The status of the build #668 was: SUCCESS. The Pull Request #4404 was integrated: "4403-cannot-change-position-in-a-memory-write-stream" Pull request url: https://github.com/pharo-project/pharo/pull/4404 Issue Url: https://github.com/pharo-project/pharo/issues/4403 Build Url: https://ci.inria.fr/pharo-ci-jenkins2/job/Test%20pending%20pull%20request%2…

Aug. 23, 2019

[Pharo 8.0] Build #667: 4406-fix-class-def-parser-for-simple-definitions

by ci-pharo-ci-jenkins2＠inria.fr

There is a new Pharo build available! The status of the build #667 was: FAILURE. The Pull Request #4409 was integrated: "4406-fix-class-def-parser-for-simple-definitions" Pull request url: https://github.com/pharo-project/pharo/pull/4409 Issue Url: https://github.com/pharo-project/pharo/issues/4406 Build Url: https://ci.inria.fr/pharo-ci-jenkins2/job/Test%20pending%20pull%20request%2…

Aug. 23, 2019

Re: [Pharo-dev] Class definition with slot template is broken

by Marcus Denker

> On 23 Aug 2019, at 12:43, Marcus Denker <marcus.denker(a)inria.fr> wrote: > > > >> On 23 Aug 2019, at 12:13, Denis Kudriashov <dionisiydk(a)gmail.com <mailto:dionisiydk@gmail.com>> wrote: >> >> Hi >> >> Ð¿Ñ, 23 Ð°Ð²Ð³. 2019 Ð³., 11:06 Marcus Denker <marcus.denker(a)inria.fr <mailto:marcus.denker@inria.fr>>: >> >> >> > On 23 Aug 2019, at 11:49, Cyril Ferlicot <cyril.ferlicot(a)gmail.com <mailto:cyril.ferlicot@gmail.com>> wrote: >> > >> > Hi, >> > >> > Since yesterday I get a lot of bugs when I try to edit class >> > definitions while having the slot template in class definition >> > enabled. (See comment: >> > https://github.com/pharo-project/pharo/pull/4391 <https://github.com/pharo-project/pharo/pull/4391>) >> > This make it a little hard to develop :( >> > >> > Is there an easy fix? Else maybe we should revert the change until we >> > have the fix? >> > >> >> I have a fix, I will commit when the internet connection is good enough >> >> That's good. > > https://github.com/pharo-project/pharo/pull/4409 <https://github.com/pharo-project/pharo/pull/4409> > > >> >> I guess it was a class parser bug? I only tested browser with classic definition. > > yes, even with this fix I need to do some (larger) change to support Slots with parameters, e.g. > > {#lazy => LazySlot default: 5 . > #computed => ComputedSlot with: [:object | object calculateSomething] } > > I updated the fix: now it should work, there will be just no menu shown for these kinds of slots for now (to be fixed in a second step, we should add tests, too) Marcus

Aug. 23, 2019

Re: [Pharo-dev] Class definition with slot template is broken

by Marcus Denker

> On 23 Aug 2019, at 12:13, Denis Kudriashov <dionisiydk(a)gmail.com> wrote: > > Hi > > Ð¿Ñ, 23 Ð°Ð²Ð³. 2019 Ð³., 11:06 Marcus Denker <marcus.denker(a)inria.fr <mailto:marcus.denker@inria.fr>>: > > > > On 23 Aug 2019, at 11:49, Cyril Ferlicot <cyril.ferlicot(a)gmail.com <mailto:cyril.ferlicot@gmail.com>> wrote: > > > > Hi, > > > > Since yesterday I get a lot of bugs when I try to edit class > > definitions while having the slot template in class definition > > enabled. (See comment: > > https://github.com/pharo-project/pharo/pull/4391 <https://github.com/pharo-project/pharo/pull/4391>) > > This make it a little hard to develop :( > > > > Is there an easy fix? Else maybe we should revert the change until we > > have the fix? > > > > I have a fix, I will commit when the internet connection is good enough > > That's good. https://github.com/pharo-project/pharo/pull/4409 <https://github.com/pharo-project/pharo/pull/4409> > > I guess it was a class parser bug? I only tested browser with classic definition. yes, even with this fix I need to do some (larger) change to support Slots with parameters, e.g. {#lazy => LazySlot default: 5 . #computed => ComputedSlot with: [:object | object calculateSomething] } This need some change in how the class parser models slots, thoughâ¦ CDSlotNode is just modelling a name and the class of the slot, but not the fact that slot definitions can contain parameters which can contain even blocksâ¦ and with composition (not yet in Pharo8), we need to support things like #lazyAndObservable => SpObservableSlot + LazySlot default: 5 . but I think it is not hard to do. Marcus

Aug. 23, 2019

Jump to page: