Seaside 2.8 overrides

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

Seaside 2.8 overrides

Pavel Krivanek-2
Hi,

Seaside 2.8 overrides this four methods in the Kernel of Squeak:

Stream >> closed
Symbol >> asMutator
Color >> asHTMLColor
MessageSend >> valueWithPossibleArgs:

It would great if we could revalue this overrides and remove them. I
have already removed overrides of KomHttpServer in the KernelImage
repository.

If Seaside will have no overrides of other packages, we will be able
to modify and publish basic KernelImage packages using web-based
Seaside environment. It would be really big improvement and the first
step to the independent fully web-based Squeak development
environment.

If some of this overrides are necessary for older versions of Squeak
(3.7), I beseech for moving them into the standalone package.

-- Pavel
_______________________________________________
Seaside mailing list
[hidden email]
http://lists.squeakfoundation.org/cgi-bin/mailman/listinfo/seaside
Reply | Threaded
Open this post in threaded view
|

Re: Seaside 2.8 overrides

Philippe Marschall
2007/2/25, Pavel Krivanek <[hidden email]>:

> Hi,
>
> Seaside 2.8 overrides this four methods in the Kernel of Squeak:
>
> Stream >> closed
> Symbol >> asMutator
> Color >> asHTMLColor
> MessageSend >> valueWithPossibleArgs:
>
> It would great if we could revalue this overrides and remove them. I
> have already removed overrides of KomHttpServer in the KernelImage
> repository.
>
> If Seaside will have no overrides of other packages, we will be able
> to modify and publish basic KernelImage packages using web-based
> Seaside environment. It would be really big improvement and the first
> step to the independent fully web-based Squeak development
> environment.
>
> If some of this overrides are necessary for older versions of Squeak
> (3.7), I beseech for moving them into the standalone package.
All but #valueWithPossibleArgs: are in Squeak 3.8 so we'll probably
need a compatibility package for Squeak 3.7 (which Avi uses).

#valueWithPossibleArgs: is a conflict with OmniBrowser, right?

I'll have to contact Göran about the Kom fix.

Cheers
Philippe

> -- Pavel
> _______________________________________________
> Seaside mailing list
> [hidden email]
> http://lists.squeakfoundation.org/cgi-bin/mailman/listinfo/seaside
>

_______________________________________________
Seaside mailing list
[hidden email]
http://lists.squeakfoundation.org/cgi-bin/mailman/listinfo/seaside
Reply | Threaded
Open this post in threaded view
|

Re: Seaside 2.8 overrides

Avi Bryant-2
On 2/25/07, Philippe Marschall <[hidden email]> wrote:

> All but #valueWithPossibleArgs: are in Squeak 3.8 so we'll probably
> need a compatibility package for Squeak 3.7 (which Avi uses).

Well, certainly no need to keep it in solely on my account - I can
(and do) use a branched version of Seaside anyway.

Others may use 3.7, of course.

We could also pick a different, non-conflicting selector that did the
same thing and use that instead.

Avi
_______________________________________________
Seaside mailing list
[hidden email]
http://lists.squeakfoundation.org/cgi-bin/mailman/listinfo/seaside
Reply | Threaded
Open this post in threaded view
|

Re: Seaside 2.8 overrides

Philippe Marschall
2007/2/26, Avi Bryant <[hidden email]>:

> On 2/25/07, Philippe Marschall <[hidden email]> wrote:
>
> > All but #valueWithPossibleArgs: are in Squeak 3.8 so we'll probably
> > need a compatibility package for Squeak 3.7 (which Avi uses).
>
> Well, certainly no need to keep it in solely on my account - I can
> (and do) use a branched version of Seaside anyway.
>
> Others may use 3.7, of course.
>
> We could also pick a different, non-conflicting selector that did the
> same thing and use that instead.

I decided to go with: #valueWithPossibleArgument:, it has the
following advantages

- BlockContext has it in 3.8.1 as well as 3.9
- I don't see an other package that has it in MessageSend

I put BlockContext >> #valueWithPossibleArgument: in the compatibility
package for Squeak 3.7 called SeasideSqueak37. Additionally there is a
TestSuite (WAPlatformTests) for all these four selectors.

When Kom gets updated we can remove that override as well.

Philippe

> Avi
> _______________________________________________
> Seaside mailing list
> [hidden email]
> http://lists.squeakfoundation.org/cgi-bin/mailman/listinfo/seaside
>
_______________________________________________
Seaside mailing list
[hidden email]
http://lists.squeakfoundation.org/cgi-bin/mailman/listinfo/seaside