Load OB in Pharo 1.2

Previous Topic Next Topic
 
classic Classic list List threaded Threaded
28 messages Options
12
Reply | Threaded
Open this post in threaded view
|

Load OB in Pharo 1.2

Mariano Martinez Peck
Hi Lukas. Trying to evaluate:

(ConfigurationOfOmniBrowser project version: '1.1-baseline') load:  'Core RB'

I am trying to load OB and RB in Pharo 1.2

I found still refernces to Preferences and some other problems.

I attach a changeset to fix the ones I found. If you want to check them lukas would be nice :)

Thanks

Mariano

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

#Users#mariano#Pharo#imagenes#PharoCore-1.2-12092#OBInPharo12.1.cs (1K) Download Attachment
Reply | Threaded
Open this post in threaded view
|

Re: Load OB in Pharo 1.2

Mariano Martinez Peck
Sorry...found another one.

In MenuMorph >> add: label target: anObject selector: aSelector enabled: aBoolean icon: aSymbol
   
you do Preferences menuWithIcons

but should be  UITheme currentSettings menuWithIcons

cheers

mariano

On Tue, Aug 17, 2010 at 3:53 PM, Mariano Martinez Peck <[hidden email]> wrote:
Hi Lukas. Trying to evaluate:

(ConfigurationOfOmniBrowser project version: '1.1-baseline') load:  'Core RB'

I am trying to load OB and RB in Pharo 1.2

I found still refernces to Preferences and some other problems.

I attach a changeset to fix the ones I found. If you want to check them lukas would be nice :)

Thanks

Mariano


_______________________________________________
Pharo-project mailing list
[hidden email]
http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project
Reply | Threaded
Open this post in threaded view
|

Re: Load OB in Pharo 1.2

Lukas Renggli
2010/8/17 Mariano Martinez Peck <[hidden email]>:
> Sorry...found another one.
>
> In MenuMorph >> add: label target: anObject selector: aSelector enabled:
> aBoolean icon: aSymbol
>
> you do Preferences menuWithIcons
>
> but should be  UITheme currentSettings menuWithIcons

Looks like I forgot to commit a whole batch of changes:

Name: OB-Morphic-lr.132
Author: lr
Time: 17 August 2010, 5:38:15 pm
UUID: b03368be-fe23-445b-9586-f16372e5fc8b
Ancestors: OB-Morphic-lr.131

- removed all references to Preferences

Name: OB-Shout-lr.7
Author: lr
Time: 17 August 2010, 5:38:45 pm
UUID: b14c055a-6430-4772-9c89-f5cc890681fe
Ancestors: OB-Shout-lr.6

- removed all references to Preferences

Name: ECompletion-lr.135
Author: lr
Time: 17 August 2010, 5:39:14 pm
UUID: 783fd270-7bd5-43d7-abe6-a750dc6f869b
Ancestors: ECompletion-lr.134

- improved the settings placement

--
Lukas Renggli
www.lukas-renggli.ch

_______________________________________________
Pharo-project mailing list
[hidden email]
http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project
Reply | Threaded
Open this post in threaded view
|

Re: Load OB in Pharo 1.2

Lukas Renggli
In reply to this post by Mariano Martinez Peck
I cannot integrate that change-set because it breaks OB in Pharo 1.1.

PluggableButtonMorphPlus was never deprecated and now you say it just
disappeared? :-(

Lukas

2010/8/17 Mariano Martinez Peck <[hidden email]>:

> Hi Lukas. Trying to evaluate:
>
> (ConfigurationOfOmniBrowser project version: '1.1-baseline') load:  'Core
> RB'
>
> I am trying to load OB and RB in Pharo 1.2
>
> I found still refernces to Preferences and some other problems.
>
> I attach a changeset to fix the ones I found. If you want to check them
> lukas would be nice :)
>
> Thanks
>
> Mariano
>
> _______________________________________________
> Pharo-project mailing list
> [hidden email]
> http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project
>



--
Lukas Renggli
www.lukas-renggli.ch

_______________________________________________
Pharo-project mailing list
[hidden email]
http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project
Reply | Threaded
Open this post in threaded view
|

Re: Load OB in Pharo 1.2

Lukas Renggli
Anyway, don't bother. I fixed it in OB-Morphic-lr.133. This should
work in any version:

Name: OB-Morphic-lr.133
Author: lr
Time: 17 August 2010, 6:04:59 pm
UUID: 728befb5-2f47-4d92-a6c3-3f1ca44bc21f
Ancestors: OB-Morphic-lr.132

- don't use PluggableButtonMorphPlus, as it doesn't work anymore in
upcoming versions

On 17 August 2010 17:41, Lukas Renggli <[hidden email]> wrote:

> I cannot integrate that change-set because it breaks OB in Pharo 1.1.
>
> PluggableButtonMorphPlus was never deprecated and now you say it just
> disappeared? :-(
>
> Lukas
>
> 2010/8/17 Mariano Martinez Peck <[hidden email]>:
>> Hi Lukas. Trying to evaluate:
>>
>> (ConfigurationOfOmniBrowser project version: '1.1-baseline') load:  'Core
>> RB'
>>
>> I am trying to load OB and RB in Pharo 1.2
>>
>> I found still refernces to Preferences and some other problems.
>>
>> I attach a changeset to fix the ones I found. If you want to check them
>> lukas would be nice :)
>>
>> Thanks
>>
>> Mariano
>>
>> _______________________________________________
>> Pharo-project mailing list
>> [hidden email]
>> http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project
>>
>
>
>
> --
> Lukas Renggli
> www.lukas-renggli.ch
>



--
Lukas Renggli
www.lukas-renggli.ch

_______________________________________________
Pharo-project mailing list
[hidden email]
http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project
Reply | Threaded
Open this post in threaded view
|

Re: Load OB in Pharo 1.2

Stéphane Ducasse
In reply to this post by Lukas Renggli
Yes lukas we cleaned and merged all the *Plus classes are now in their superclass.
and for classes we do not really have a nice deprecate mechanism.

Stef



> I cannot integrate that change-set because it breaks OB in Pharo 1.1.
>
> PluggableButtonMorphPlus was never deprecated and now you say it just
> disappeared? :-(
>
> Lukas
>
> 2010/8/17 Mariano Martinez Peck <[hidden email]>:
>> Hi Lukas. Trying to evaluate:
>>
>> (ConfigurationOfOmniBrowser project version: '1.1-baseline') load:  'Core
>> RB'
>>
>> I am trying to load OB and RB in Pharo 1.2
>>
>> I found still refernces to Preferences and some other problems.
>>
>> I attach a changeset to fix the ones I found. If you want to check them
>> lukas would be nice :)
>>
>> Thanks
>>
>> Mariano
>>
>> _______________________________________________
>> Pharo-project mailing list
>> [hidden email]
>> http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project
>>
>
>
>
> --
> Lukas Renggli
> www.lukas-renggli.ch
>
> _______________________________________________
> Pharo-project mailing list
> [hidden email]
> http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project


_______________________________________________
Pharo-project mailing list
[hidden email]
http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project
Reply | Threaded
Open this post in threaded view
|

Re: Load OB in Pharo 1.2

Lukas Renggli
On 17 August 2010 18:43, Stéphane Ducasse <[hidden email]> wrote:
> Yes lukas we cleaned and merged all the *Plus classes are now in their superclass.
> and for classes we do not really have a nice deprecate mechanism.

Why can't you put a deprecated warning into basicNew? That would be
really helpful, because otherwise this will always cause troubles like
this.

Lukas

--
Lukas Renggli
www.lukas-renggli.ch

_______________________________________________
Pharo-project mailing list
[hidden email]
http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project
Reply | Threaded
Open this post in threaded view
|

Re: Load OB in Pharo 1.2

Mariano Martinez Peck
In reply to this post by Lukas Renggli


On Tue, Aug 17, 2010 at 5:41 PM, Lukas Renggli <[hidden email]> wrote:
I cannot integrate that change-set because it breaks OB in Pharo 1.1.

PluggableButtonMorphPlus was never deprecated and now you say it just
disappeared? :-(


Thanks Lukas. I wasn't who remove that class neither who didn't deprecated. I am just trying to start building Dev images from now...otherwise, we will have to wait a lot of time once the core is released.

thanks for your commits
 
Lukas

2010/8/17 Mariano Martinez Peck <[hidden email]>:
> Hi Lukas. Trying to evaluate:
>
> (ConfigurationOfOmniBrowser project version: '1.1-baseline') load:  'Core
> RB'
>
> I am trying to load OB and RB in Pharo 1.2
>
> I found still refernces to Preferences and some other problems.
>
> I attach a changeset to fix the ones I found. If you want to check them
> lukas would be nice :)
>
> Thanks
>
> Mariano
>
> _______________________________________________
> Pharo-project mailing list
> [hidden email]
> http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project
>



--
Lukas Renggli
www.lukas-renggli.ch

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


_______________________________________________
Pharo-project mailing list
[hidden email]
http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project
Reply | Threaded
Open this post in threaded view
|

Re: Load OB in Pharo 1.2

Lukas Renggli
> Thanks Lukas. I wasn't who remove that class neither who didn't deprecated.
> I am just trying to start building Dev images from now...otherwise, we will
> have to wait a lot of time once the core is released.

I think there must be a delay between the release of the core and the
non-core images. I don't see how that can otherwise work. Of course it
makes sense to have some incremental adaption already upfront ...

Lukas



>
> thanks for your commits
>
>>
>> Lukas
>>
>> 2010/8/17 Mariano Martinez Peck <[hidden email]>:
>> > Hi Lukas. Trying to evaluate:
>> >
>> > (ConfigurationOfOmniBrowser project version: '1.1-baseline') load:
>> > 'Core
>> > RB'
>> >
>> > I am trying to load OB and RB in Pharo 1.2
>> >
>> > I found still refernces to Preferences and some other problems.
>> >
>> > I attach a changeset to fix the ones I found. If you want to check them
>> > lukas would be nice :)
>> >
>> > Thanks
>> >
>> > Mariano
>> >
>> > _______________________________________________
>> > Pharo-project mailing list
>> > [hidden email]
>> > http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project
>> >
>>
>>
>>
>> --
>> Lukas Renggli
>> www.lukas-renggli.ch
>>
>> _______________________________________________
>> Pharo-project mailing list
>> [hidden email]
>> http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project
>
>
> _______________________________________________
> Pharo-project mailing list
> [hidden email]
> http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project
>



--
Lukas Renggli
www.lukas-renggli.ch

_______________________________________________
Pharo-project mailing list
[hidden email]
http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project
Reply | Threaded
Open this post in threaded view
|

Re: Load OB in Pharo 1.2

Marcus Denker-4

On Aug 17, 2010, at 7:31 PM, Lukas Renggli wrote:

>> Thanks Lukas. I wasn't who remove that class neither who didn't deprecated.
>> I am just trying to start building Dev images from now...otherwise, we will
>> have to wait a lot of time once the core is released.
>
> I think there must be a delay between the release of the core and the
> non-core images. I don't see how that can otherwise work. Of course it
> makes sense to have some incremental adaption already upfront ...

Hmm... one thing is clear: I want to use Refactoring and especially the
Code Critique tool on 1.2. So I need to somehow load it.

(2 pages of philosophical ramblings deleted... I should start a blog..)

        Marcus


--
Marcus Denker  -- http://www.marcusdenker.de
INRIA Lille -- Nord Europe. Team RMoD.


_______________________________________________
Pharo-project mailing list
[hidden email]
http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project
Reply | Threaded
Open this post in threaded view
|

Re: Load OB in Pharo 1.2

Lukas Renggli
On Tuesday, August 17, 2010, Marcus Denker <[hidden email]> wrote:

>
> On Aug 17, 2010, at 7:31 PM, Lukas Renggli wrote:
>
>>> Thanks Lukas. I wasn't who remove that class neither who didn't deprecated.
>>> I am just trying to start building Dev images from now...otherwise, we will
>>> have to wait a lot of time once the core is released.
>>
>> I think there must be a delay between the release of the core and the
>> non-core images. I don't see how that can otherwise work. Of course it
>> makes sense to have some incremental adaption already upfront ...
>
> Hmm... one thing is clear: I want to use Refactoring and especially the
> Code Critique tool on 1.2. So I need to somehow load it.

Yeah that make sense, but I cannot maintain code that I can't use.

Lukas

>
> (2 pages of philosophical ramblings deleted... I should start a blog..)
>
>         Marcus
>
>
> --
> Marcus Denker  -- http://www.marcusdenker.de
> INRIA Lille -- Nord Europe. Team RMoD.
>
>
> _______________________________________________
> Pharo-project mailing list
> [hidden email]
> http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project
>

--
Lukas Renggli
www.lukas-renggli.ch

_______________________________________________
Pharo-project mailing list
[hidden email]
http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project
Reply | Threaded
Open this post in threaded view
|

Re: Load OB in Pharo 1.2

Stéphane Ducasse
In reply to this post by Marcus Denker-4

On Aug 17, 2010, at 7:55 PM, Marcus Denker wrote:

>
> On Aug 17, 2010, at 7:31 PM, Lukas Renggli wrote:
>
>>> Thanks Lukas. I wasn't who remove that class neither who didn't deprecated.
>>> I am just trying to start building Dev images from now...otherwise, we will
>>> have to wait a lot of time once the core is released.
>>
>> I think there must be a delay between the release of the core and the
>> non-core images. I don't see how that can otherwise work. Of course it
>> makes sense to have some incremental adaption already upfront ...
>
> Hmm... one thing is clear: I want to use Refactoring and especially the
> Code Critique tool on 1.2. So I need to somehow load it.
>
> (2 pages of philosophical ramblings deleted... I should start a blog..)


No
rambling blog are boring :)

>
> Marcus
>
>
> --
> Marcus Denker  -- http://www.marcusdenker.de
> INRIA Lille -- Nord Europe. Team RMoD.
>
>
> _______________________________________________
> Pharo-project mailing list
> [hidden email]
> http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project


_______________________________________________
Pharo-project mailing list
[hidden email]
http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project
Reply | Threaded
Open this post in threaded view
|

Re: Load OB in Pharo 1.2

Gary Chambers-4
In reply to this post by Lukas Renggli
Sorry, that was Stef and I at Camp Smalltalk. Did feel good to be rid of
XYZPlus classes though ;-)

Regards, Gary

----- Original Message -----
From: "Lukas Renggli" <[hidden email]>
To: <[hidden email]>
Sent: Tuesday, August 17, 2010 7:07 PM
Subject: Re: [Pharo-project] Load OB in Pharo 1.2


On Tuesday, August 17, 2010, Marcus Denker <[hidden email]> wrote:

>
> On Aug 17, 2010, at 7:31 PM, Lukas Renggli wrote:
>
>>> Thanks Lukas. I wasn't who remove that class neither who didn't
>>> deprecated.
>>> I am just trying to start building Dev images from now...otherwise, we
>>> will
>>> have to wait a lot of time once the core is released.
>>
>> I think there must be a delay between the release of the core and the
>> non-core images. I don't see how that can otherwise work. Of course it
>> makes sense to have some incremental adaption already upfront ...
>
> Hmm... one thing is clear: I want to use Refactoring and especially the
> Code Critique tool on 1.2. So I need to somehow load it.

Yeah that make sense, but I cannot maintain code that I can't use.

Lukas

>
> (2 pages of philosophical ramblings deleted... I should start a blog..)
>
>  Marcus
>
>
> --
> Marcus Denker -- http://www.marcusdenker.de
> INRIA Lille -- Nord Europe. Team RMoD.
>
>
> _______________________________________________
> Pharo-project mailing list
> [hidden email]
> http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project
>

--
Lukas Renggli
www.lukas-renggli.ch

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


_______________________________________________
Pharo-project mailing list
[hidden email]
http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project
Reply | Threaded
Open this post in threaded view
|

Re: Load OB in Pharo 1.2

Lukas Renggli
Removing them is cool, but there should be a grace period where they
are deprecated. Now I had to duplicate the whole class in OB with a
uniue name.

On Tuesday, August 17, 2010, Gary Chambers <[hidden email]> wrote:

> Sorry, that was Stef and I at Camp Smalltalk. Did feel good to be rid of XYZPlus classes though ;-)
>
> Regards, Gary
>
> ----- Original Message ----- From: "Lukas Renggli" <[hidden email]>
> To: <[hidden email]>
> Sent: Tuesday, August 17, 2010 7:07 PM
> Subject: Re: [Pharo-project] Load OB in Pharo 1.2
>
>
> On Tuesday, August 17, 2010, Marcus Denker <[hidden email]> wrote:
>
>
> On Aug 17, 2010, at 7:31 PM, Lukas Renggli wrote:
>
>
> Thanks Lukas. I wasn't who remove that class neither who didn't deprecated.
> I am just trying to start building Dev images from now...otherwise, we will
> have to wait a lot of time once the core is released.
>
>
> I think there must be a delay between the release of the core and the
> non-core images. I don't see how that can otherwise work. Of course it
> makes sense to have some incremental adaption already upfront ...
>
>
> Hmm... one thing is clear: I want to use Refactoring and especially the
> Code Critique tool on 1.2. So I need to somehow load it.
>
>
> Yeah that make sense, but I cannot maintain code that I can't use.
>
> Lukas
>
>
>
> (2 pages of philosophical ramblings deleted... I should start a blog..)
>
>  Marcus
>
>
> --
> Marcus Denker -- http://www.marcusdenker.de
> INRIA Lille -- Nord Europe. Team RMoD.
>
>
> _______________________________________________
> Pharo-project mailing list
> [hidden email]
> http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project
>
>
>
> --
> Lukas Renggli
> www.lukas-renggli.ch
>
> _______________________________________________
> Pharo-project mailing list
> [hidden email]
> http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project
>
> _______________________________________________
> Pharo-project mailing list
> [hidden email]
> http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project
>

--
Lukas Renggli
www.lukas-renggli.ch

_______________________________________________
Pharo-project mailing list
[hidden email]
http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project
Reply | Threaded
Open this post in threaded view
|

Re: Load OB in Pharo 1.2

Marcus Denker-4

On Aug 17, 2010, at 8:28 PM, Lukas Renggli wrote:

> Removing them is cool, but there should be a grace period where they
> are deprecated. Now I had to duplicate the whole class in OB with a
> uniue name.
>
can it be that this class has a var that we pushed up the hierarchy? At least
when I try to load OB in 1.2, I get an error message like that...


> On Tuesday, August 17, 2010, Gary Chambers <[hidden email]> wrote:
>> Sorry, that was Stef and I at Camp Smalltalk. Did feel good to be rid of XYZPlus classes though ;-)
>>
>> Regards, Gary
>>
>> ----- Original Message ----- From: "Lukas Renggli" <[hidden email]>
>> To: <[hidden email]>
>> Sent: Tuesday, August 17, 2010 7:07 PM
>> Subject: Re: [Pharo-project] Load OB in Pharo 1.2
>>
>>
>> On Tuesday, August 17, 2010, Marcus Denker <[hidden email]> wrote:
>>
>>
>> On Aug 17, 2010, at 7:31 PM, Lukas Renggli wrote:
>>
>>
>> Thanks Lukas. I wasn't who remove that class neither who didn't deprecated.
>> I am just trying to start building Dev images from now...otherwise, we will
>> have to wait a lot of time once the core is released.
>>
>>
>> I think there must be a delay between the release of the core and the
>> non-core images. I don't see how that can otherwise work. Of course it
>> makes sense to have some incremental adaption already upfront ...
>>
>>
>> Hmm... one thing is clear: I want to use Refactoring and especially the
>> Code Critique tool on 1.2. So I need to somehow load it.
>>
>>
>> Yeah that make sense, but I cannot maintain code that I can't use.
>>
>> Lukas
>>
>>
>>
>> (2 pages of philosophical ramblings deleted... I should start a blog..)
>>
>>  Marcus
>>
>>
>> --
>> Marcus Denker -- http://www.marcusdenker.de
>> INRIA Lille -- Nord Europe. Team RMoD.
>>
>>
>> _______________________________________________
>> Pharo-project mailing list
>> [hidden email]
>> http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project
>>
>>
>>
>> --
>> Lukas Renggli
>> www.lukas-renggli.ch
>>
>> _______________________________________________
>> Pharo-project mailing list
>> [hidden email]
>> http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project
>>
>> _______________________________________________
>> Pharo-project mailing list
>> [hidden email]
>> http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project
>>
>
> --
> Lukas Renggli
> www.lukas-renggli.ch
>
> _______________________________________________
> Pharo-project mailing list
> [hidden email]
> http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project

--
Marcus Denker  -- http://www.marcusdenker.de
INRIA Lille -- Nord Europe. Team RMoD.


_______________________________________________
Pharo-project mailing list
[hidden email]
http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project
Reply | Threaded
Open this post in threaded view
|

Re: Load OB in Pharo 1.2

Marcus Denker-4
In reply to this post by Stéphane Ducasse

On Aug 17, 2010, at 8:08 PM, Stéphane Ducasse wrote:
>>
>> (2 pages of philosophical ramblings deleted... I should start a blog..)
>
>
> No
> rambling blog are boring :)

No need to read it... maybe I could just write them and then delete them?
Would help, too.

        Marcus


--
Marcus Denker  -- http://www.marcusdenker.de
INRIA Lille -- Nord Europe. Team RMoD.


_______________________________________________
Pharo-project mailing list
[hidden email]
http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project
Reply | Threaded
Open this post in threaded view
|

Re: Load OB in Pharo 1.2

Marcus Denker-4
In reply to this post by Lukas Renggli

On Aug 17, 2010, at 8:07 PM, Lukas Renggli wrote:

> On Tuesday, August 17, 2010, Marcus Denker <[hidden email]> wrote:
>>
>> On Aug 17, 2010, at 7:31 PM, Lukas Renggli wrote:
>>
>>>> Thanks Lukas. I wasn't who remove that class neither who didn't deprecated.
>>>> I am just trying to start building Dev images from now...otherwise, we will
>>>> have to wait a lot of time once the core is released.
>>>
>>> I think there must be a delay between the release of the core and the
>>> non-core images. I don't see how that can otherwise work. Of course it
>>> makes sense to have some incremental adaption already upfront ...
>>
>> Hmm... one thing is clear: I want to use Refactoring and especially the
>> Code Critique tool on 1.2. So I need to somehow load it.
>
> Yeah that make sense, but I cannot maintain code that I can't use.
>

So I guess I should maintain a branch for 1.2... I really want to use it.
And we need at some point to think about what to do for the browser. do we
really maintain two? Even if OB is not "the browser to end all browsers", isn't it
good enough to replace the old one for now?

        Marcus


--
Marcus Denker  -- http://www.marcusdenker.de
INRIA Lille -- Nord Europe. Team RMoD.


_______________________________________________
Pharo-project mailing list
[hidden email]
http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project
Reply | Threaded
Open this post in threaded view
|

Re: Load OB in Pharo 1.2

Stéphane Ducasse
In reply to this post by Marcus Denker-4
>
>>>
>>> (2 pages of philosophical ramblings deleted... I should start a blog..)
>>
>>
>> No
>> rambling blog are boring :)
>
> No need to read it... maybe I could just write them and then delete them?
> Would help, too.

Yes people call that catharsis :)
It is good to express stuff.
I delete also mails I would like to send :)

Stef

_______________________________________________
Pharo-project mailing list
[hidden email]
http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project
Reply | Threaded
Open this post in threaded view
|

Re: Load OB in Pharo 1.2

Stéphane Ducasse
In reply to this post by Lukas Renggli

On Aug 17, 2010, at 8:28 PM, Lukas Renggli wrote:

> Removing them is cool, but there should be a grace period where they
> are deprecated.

But how? We could let the subclass empty in this case  (is it what you mean) but
when we are doing refactoring like changing iv it may not work.
We could not really let the classes in the system.

I have the impression that having branch at the tool level is something that we cannot avoid.


> Now I had to duplicate the whole class in OB with a
> uniue name.


??



>
> On Tuesday, August 17, 2010, Gary Chambers <[hidden email]> wrote:
>> Sorry, that was Stef and I at Camp Smalltalk. Did feel good to be rid of XYZPlus classes though ;-)
>>
>> Regards, Gary
>>
>> ----- Original Message ----- From: "Lukas Renggli" <[hidden email]>
>> To: <[hidden email]>
>> Sent: Tuesday, August 17, 2010 7:07 PM
>> Subject: Re: [Pharo-project] Load OB in Pharo 1.2
>>
>>
>> On Tuesday, August 17, 2010, Marcus Denker <[hidden email]> wrote:
>>
>>
>> On Aug 17, 2010, at 7:31 PM, Lukas Renggli wrote:
>>
>>
>> Thanks Lukas. I wasn't who remove that class neither who didn't deprecated.
>> I am just trying to start building Dev images from now...otherwise, we will
>> have to wait a lot of time once the core is released.
>>
>>
>> I think there must be a delay between the release of the core and the
>> non-core images. I don't see how that can otherwise work. Of course it
>> makes sense to have some incremental adaption already upfront ...
>>
>>
>> Hmm... one thing is clear: I want to use Refactoring and especially the
>> Code Critique tool on 1.2. So I need to somehow load it.
>>
>>
>> Yeah that make sense, but I cannot maintain code that I can't use.
>>
>> Lukas
>>
>>
>>
>> (2 pages of philosophical ramblings deleted... I should start a blog..)
>>
>>  Marcus
>>
>>
>> --
>> Marcus Denker -- http://www.marcusdenker.de
>> INRIA Lille -- Nord Europe. Team RMoD.
>>
>>
>> _______________________________________________
>> Pharo-project mailing list
>> [hidden email]
>> http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project
>>
>>
>>
>> --
>> Lukas Renggli
>> www.lukas-renggli.ch
>>
>> _______________________________________________
>> Pharo-project mailing list
>> [hidden email]
>> http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project
>>
>> _______________________________________________
>> Pharo-project mailing list
>> [hidden email]
>> http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project
>>
>
> --
> Lukas Renggli
> www.lukas-renggli.ch
>
> _______________________________________________
> Pharo-project mailing list
> [hidden email]
> http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project


_______________________________________________
Pharo-project mailing list
[hidden email]
http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project
Reply | Threaded
Open this post in threaded view
|

Re: Load OB in Pharo 1.2

Henrik Sperre Johansen
  On 17.08.2010 21:14, Stéphane Ducasse wrote:
> On Aug 17, 2010, at 8:28 PM, Lukas Renggli wrote:
>
>> Removing them is cool, but there should be a grace period where they
>> are deprecated.
> But how? We could let the subclass empty in this case  (is it what you mean) but
> when we are doing refactoring like changing iv it may not work.
> We could not really let the classes in the system.
>
> I have the impression that having branch at the tool level is something that we cannot avoid.
You can do the same as for CrLFFileStream, keep the empty class, but
reroute any instantiations to super which now contains its functionality.
Like Lukas said, deprecate Pluggable*Plus class >> basicNew in 1.2. Then
remove them in 1.3.

Cheers,
Henry

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