Re: [Pharo-project] SocketStream enh

2010-08-22 Thread Stéphane Ducasse
I will integrate another socket related fix because apparently the tests still 
break the image 
with these ones loaded.

Stef

On Aug 22, 2010, at 1:53 PM, Philippe Marschall wrote:

> On 18.08.2010 12:56, Stéphane Ducasse wrote:
>> 
>> I will integrate the following SocketStream tests and before I would 
>> love to see if this
>> does not impact seaside
>> 
>> http://code.google.com/p/pharo/issues/detail?id=2767
>> 
>> Does anybody have the time to give feedback on this important network 
>> changes?
> 
> I did a short test with Seaside and everything still seems to work. And
> if it fixes bugs, I'm all tits forward about it.
> 
> The deprecation of SmalltalkImage >> #platformName is a PITA though. It
> requires us to have different versions of Comanche and WebClient for
> different versions of Pharo.
> 
> Cheers
> Philippe
> 
> 
> ___
> Pharo-project mailing list
> Pharo-project@lists.gforge.inria.fr
> http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project


___
Pharo-project mailing list
Pharo-project@lists.gforge.inria.fr
http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project


Re: [Pharo-project] SocketStream enh

2010-08-22 Thread Stéphane Ducasse
Thanks!

I really think that we should systematically use a dependency management 
system. Of course it 
does not fix the problems but we can move on.
I think that we can benefit of the idea of a platform classes hierarchy, this 
is why we push the change.


>>> I will integrate the following SocketStream tests and before I would 
>>> love to see if this
>>> does not impact seaside
>>> 
>>> http://code.google.com/p/pharo/issues/detail?id=2767
>>> 
>>> Does anybody have the time to give feedback on this important network 
>>> changes?
>> 
>> I did a short test with Seaside and everything still seems to work. And
>> if it fixes bugs, I'm all tits forward about it.

Ok I will integrate it now.
This was the next item on the list.

>> The deprecation of SmalltalkImage >> #platformName is a PITA though. It
>> requires us to have different versions of Comanche and WebClient for
>> different versions of Pharo.

I will integrate now webClient. May be I will need to know what I should load 
in. :)

> Pharo 1.0 is long outdated, Pharo 1.2 is not stable yet. We should
> only target Pharo 1.1.
> 
> Also the latest development tools only work in Pharo 1.1. There is no
> point in making Seaside work in all versions.
> 
> Lukas
> 
> -- 
> Lukas Renggli
> www.lukas-renggli.ch
> 
> ___
> Pharo-project mailing list
> Pharo-project@lists.gforge.inria.fr
> http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project


___
Pharo-project mailing list
Pharo-project@lists.gforge.inria.fr
http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project


Re: [Pharo-project] SocketStream enh

2010-08-22 Thread Lukas Renggli
On 22 August 2010 13:53, Philippe Marschall  wrote:
> On 18.08.2010 12:56, Stéphane Ducasse wrote:
>>
>> I will integrate the following SocketStream tests and before I would 
>> love to see if this
>> does not impact seaside
>>
>> http://code.google.com/p/pharo/issues/detail?id=2767
>>
>> Does anybody have the time to give feedback on this important network 
>> changes?
>
> I did a short test with Seaside and everything still seems to work. And
> if it fixes bugs, I'm all tits forward about it.
>
> The deprecation of SmalltalkImage >> #platformName is a PITA though. It
> requires us to have different versions of Comanche and WebClient for
> different versions of Pharo.

Pharo 1.0 is long outdated, Pharo 1.2 is not stable yet. We should
only target Pharo 1.1.

Also the latest development tools only work in Pharo 1.1. There is no
point in making Seaside work in all versions.

Lukas

-- 
Lukas Renggli
www.lukas-renggli.ch

___
Pharo-project mailing list
Pharo-project@lists.gforge.inria.fr
http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project


Re: [Pharo-project] SocketStream enh

2010-08-22 Thread Philippe Marschall
On 18.08.2010 12:56, Stéphane Ducasse wrote:
> 
> I will integrate the following SocketStream tests and before I would love 
> to see if this
> does not impact seaside
>
> http://code.google.com/p/pharo/issues/detail?id=2767
> 
> Does anybody have the time to give feedback on this important network changes?

I did a short test with Seaside and everything still seems to work. And
if it fixes bugs, I'm all tits forward about it.

The deprecation of SmalltalkImage >> #platformName is a PITA though. It
requires us to have different versions of Comanche and WebClient for
different versions of Pharo.

Cheers
Philippe


___
Pharo-project mailing list
Pharo-project@lists.gforge.inria.fr
http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project

Re: [Pharo-project] SocketStream enh

2010-08-19 Thread Stéphane Ducasse
Tx
I will forward that to squeakers because they may be interested too.

On Aug 19, 2010, at 3:04 PM, LUIS E QUIJANO wrote:

> Stef,
> 
> I have been testing your SocketStream code.  So far so good.
> 
> I noticed that if accidentally a SmallInteger versus a CharacterOrByte 
> parameter is used in the method SocketStream>>upTo:, the environment goes 
> into a degraded state that persists until the original instance of 
> SocketStream gets closed.  For example, no more new Socket connects.  After 
> closing the original instance of SocketStream, the environment "seems" to be 
> back to "normal".  A simple example for reproducing the "bad parameter" 
> situation is below.
> 
> Hope this helps,
> 
> Luis
> -=-=-
> Luis Quijano
> quij...@panasoft.com
> -=-=-
> | hostName portNumber hostIP ss data |
> hostName := '10.10.10.100'.
> portNumber := 10001.
> hostIP := NetNameResolver addressForName: hostName timeout: 20.
> ss := SocketStream openConnectionToHost: hostIP port: portNumber.
> Transcript
>show: '-- Connecting --'; cr;
>show: 'hostName = ' , hostName; cr;
>show: 'hostIP = ' , hostIP printString; cr;
>show: 'portNumber = ' , portNumber printString;
>cr.
> "data := ss upTo: (Character value: 68)." "Good parameter"
> data := ss upTo: 68. "Bad parameter"
> Transcript
>show: 'Data received: ';
>show: data;
>cr.
> ss close.
> Transcript
>show: '-- Connection Closed --';
>cr.
> data inspect.
> =
> At 11:43 AM 8/8/2010, you wrote:
>> Hi lukas and others
>> 
>> I will integrate the following SocketStream tests and before I would love to 
>> see if this
>> does not impact seaside
>> 
>> http://code.google.com/p/pharo/issues/detail?id=2767
>> 
>> Stef
>> ___
>> Pharo-project mailing list
>> Pharo-project@lists.gforge.inria.fr
>> http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project
> 
> 
> 
> 
> ___
> Pharo-project mailing list
> Pharo-project@lists.gforge.inria.fr
> http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project


___
Pharo-project mailing list
Pharo-project@lists.gforge.inria.fr
http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project


Re: [Pharo-project] SocketStream enh

2010-08-19 Thread LUIS E QUIJANO

Stef,

I have been testing your SocketStream code.  So far so good.

I noticed that if accidentally a SmallInteger versus a 
CharacterOrByte parameter is used in the method SocketStream>>upTo:, 
the environment goes into a degraded state that persists until the 
original instance of SocketStream gets closed.  For example, no more 
new Socket connects.  After closing the original instance of 
SocketStream, the environment "seems" to be back to "normal".  A 
simple example for reproducing the "bad parameter" situation is below.


Hope this helps,

Luis
-=-=-
Luis Quijano
quij...@panasoft.com
-=-=-
| hostName portNumber hostIP ss data |
hostName := '10.10.10.100'.
portNumber := 10001.
hostIP := NetNameResolver addressForName: hostName timeout: 20.
ss := SocketStream openConnectionToHost: hostIP port: portNumber.
Transcript
show: '-- Connecting --'; cr;
show: 'hostName = ' , hostName; cr;
show: 'hostIP = ' , hostIP printString; cr;
show: 'portNumber = ' , portNumber printString;
cr.
"data := ss upTo: (Character value: 68)." "Good parameter"
data := ss upTo: 68. "Bad parameter"
Transcript
show: 'Data received: ';
show: data;
cr.
ss close.
Transcript
show: '-- Connection Closed --';
cr.
data inspect.
=
At 11:43 AM 8/8/2010, you wrote:

Hi lukas and others

I will integrate the following SocketStream tests and before I would 
love to see if this

does not impact seaside

http://code.google.com/p/pharo/issues/detail?id=2767

Stef
___
Pharo-project mailing list
Pharo-project@lists.gforge.inria.fr
http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project





___
Pharo-project mailing list
Pharo-project@lists.gforge.inria.fr
http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project


Re: [Pharo-project] SocketStream enh

2010-08-18 Thread Noury Bouraqadi

On 18 août 2010, at 12:56, Stéphane Ducasse wrote:

> 
> I will integrate the following SocketStream tests and before I would love 
> to see if this
> does not impact seaside
> 
> http://code.google.com/p/pharo/issues/detail?id=2767
> 
> Does anybody have the time to give feedback on this important network changes?
> 
I tested the referenced code and it fixes issue 2765 (SocketStreamTest hangs).
I put a detailed comment on GoogleCode.

Noury

___
Pharo-project mailing list
Pharo-project@lists.gforge.inria.fr
http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project


Re: [Pharo-project] SocketStream enh

2010-08-18 Thread Philippe Marschall
On 08/18/2010 12:56 PM, Stéphane Ducasse wrote:
> 
> I will integrate the following SocketStream tests and before I would love 
> to see if this
> does not impact seaside
>
> http://code.google.com/p/pharo/issues/detail?id=2767
> 
> Does anybody have the time to give feedback on this important network changes?

Sorry, didn't yet have time.

Cheers
Philippe


___
Pharo-project mailing list
Pharo-project@lists.gforge.inria.fr
http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project

Re: [Pharo-project] SocketStream enh

2010-08-18 Thread Stéphane Ducasse

 I will integrate the following SocketStream tests and before I would love 
 to see if this
 does not impact seaside
 
 http://code.google.com/p/pharo/issues/detail?id=2767

Does anybody have the time to give feedback on this important network changes?

Stef
___
Pharo-project mailing list
Pharo-project@lists.gforge.inria.fr
http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project


Re: [Pharo-project] SocketStream enh

2010-08-10 Thread Stéphane Ducasse
enjoy them

Stef

On Aug 9, 2010, at 10:56 PM, Lukas Renggli wrote:

> I am on holidays, I cannot test this week. Philippe, Julian?
> 
> Lukas
> 
> On 9 August 2010 10:57, Stéphane Ducasse  wrote:
>> May be you missed this call but this is an important one.
>> 
>> On Aug 8, 2010, at 6:43 PM, Stéphane Ducasse wrote:
>> 
>>> Hi lukas and others
>>> 
>>> I will integrate the following SocketStream tests and before I would love 
>>> to see if this
>>> does not impact seaside
>>> 
>>> http://code.google.com/p/pharo/issues/detail?id=2767
>> 
>> and related fixes.
>> 
>> 
>>> 
>>> Stef
>>> ___
>>> Pharo-project mailing list
>>> Pharo-project@lists.gforge.inria.fr
>>> http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project
>> 
>> 
>> ___
>> Pharo-project mailing list
>> Pharo-project@lists.gforge.inria.fr
>> http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project
>> 
> 
> 
> 
> -- 
> Lukas Renggli
> www.lukas-renggli.ch
> 
> ___
> Pharo-project mailing list
> Pharo-project@lists.gforge.inria.fr
> http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project


___
Pharo-project mailing list
Pharo-project@lists.gforge.inria.fr
http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project


Re: [Pharo-project] SocketStream enh

2010-08-09 Thread Lukas Renggli
I am on holidays, I cannot test this week. Philippe, Julian?

Lukas

On 9 August 2010 10:57, Stéphane Ducasse  wrote:
> May be you missed this call but this is an important one.
>
> On Aug 8, 2010, at 6:43 PM, Stéphane Ducasse wrote:
>
>> Hi lukas and others
>>
>> I will integrate the following SocketStream tests and before I would love to 
>> see if this
>> does not impact seaside
>>
>> http://code.google.com/p/pharo/issues/detail?id=2767
>
> and related fixes.
>
>
>>
>> Stef
>> ___
>> Pharo-project mailing list
>> Pharo-project@lists.gforge.inria.fr
>> http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project
>
>
> ___
> Pharo-project mailing list
> Pharo-project@lists.gforge.inria.fr
> http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project
>



-- 
Lukas Renggli
www.lukas-renggli.ch

___
Pharo-project mailing list
Pharo-project@lists.gforge.inria.fr
http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project


Re: [Pharo-project] SocketStream enh

2010-08-09 Thread Stéphane Ducasse
May be you missed this call but this is an important one. 

On Aug 8, 2010, at 6:43 PM, Stéphane Ducasse wrote:

> Hi lukas and others
> 
> I will integrate the following SocketStream tests and before I would love to 
> see if this 
> does not impact seaside 
> 
> http://code.google.com/p/pharo/issues/detail?id=2767

and related fixes.


> 
> Stef
> ___
> Pharo-project mailing list
> Pharo-project@lists.gforge.inria.fr
> http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project


___
Pharo-project mailing list
Pharo-project@lists.gforge.inria.fr
http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project