Re: [Pharo-dev] 'VOMongoConnectionError' when dowloading mcz from smalltalkhub.com

2020-08-29 Thread Dale Henrichs
So close, the interactive tests are passing, but there is a use case 
that is popping up in a travis job[1] where this url 
http://smalltalkhub.com/mc/Swazoo/Swazoo/main is being used (no trailing 
slash) and this form of the url still produces the `raw` listing, on 
SqueakSource (and other sites) the trailing slash is not required to get 
the html-based listing ...


Again, I appreciate your effort, I hopeful that this is the last detail,

Dale

[1] https://travis-ci.org/github/GsDevKit/GsDevKit_home/jobs/721523220

On 8/29/20 10:12 AM, Christophe Demarey wrote:

I hope this time it will be fine. I removed the newline characters.
I will also check if I can keep the original url in the browser rather 
than the redirected one.


Regards,
Christophe.

Le 29 août 2020 à 18:41, Dale Henrichs 
> a écrit :


Christophe,

Again, I appreciate your effort, but you are not quite there.

If you look at SqueakSource or GemSource html (attached), the html 
uses  to create newlines on the page and in the Smalltalkhub 
page, you've inserted a newline as part of the href filename, while 
SqueakSource and GemSource (and the other monticello sites) do not 
include a newline in the href, so for better or worse, in GemStone 
the method MCReader>>canReadFileNamed: expects the filename to end 
with `.mcz` not a newline and all of the filenames parsed from the 
SmalltalkHub page are discarded as non-Monticello files.


So if you eliminate the trailing newline in the href, the GemStone 
code should finally be happy.


Thanks for your effort!

Dale


On 8/28/20 3:05 PM, Christophe Demarey wrote:
I understand that he new site is unusable if tools do not get the 
expected input. Sorry for the inconvenience. I was just not aware of 
this as Pharo uses the raw format.

I had to set up a CGI to produce an html listing from the raw file.
It should now be ok. Could you tell me if it works in GemStone now?

Christophe

Le 28 août 2020 à 19:21, Dale Henrichs 
> a écrit :


Christophe,

I appreciate your effort, but I assume that you aren't finished yet?

When I use the url from your message I get a `Not Found` error 
(note that I explicitly included a trailing slash in the url):




This is a bug (or omission) as well, since other Monticello sites 
produce the expected HTML output with or without a trailing slash.


Without the trailing slash I get the same (non-html) response that 
was causing a problem before ... although something must have 
changed, because the GemStone command is now producing an error 
instead of an empty list ... Once you get the static site to 
produce the expected output, I'm confident that the GemStone errors 
will go away (this code has been running for about a decade on all 
Monticello html repositories).


I suggest that you look at what is produced on SqueakSource 
(http://www.squeaksource.com/MooseSQL/) as an example of expected 
output of the mcz file listing ... this html page format has been 
used since 2003 for ALL (valid) monticello repositories.


If it is not clear, the `?format=raw` option is a recent Pharo only 
option and when the dynamic site was running and the `?format-raw` 
option was OMITTED it produced output compatible with 
(http://www.squeaksource.com/MooseSQL/) ...


Restoring HTML is *REQUIRED* for the static site to be a faithful 
replacement of the dynamic site... the format that is produced 
today is only compatible with the Pharo only `?format=raw` option.


Bug or omission ... the static site is currently in worse shape 
than the dynamic site before the swap ...


Dale

On 8/28/20 2:12 AM, Christophe Demarey wrote:

Le 28 août 2020 à 10:15, Sven Van Caekenberghe  a écrit :




On 28 Aug 2020, at 10:13, Christophe Demarey  
wrote:

Smalltalkhub is now to able to distinguish between raw and not raw mcz listing 
requests.
Ex:
http://smalltalkhub.com/mc/Seaside/Seaside30LGPL/main?format=raw
http://smalltalkhub.com/mc/Seaside/Seaside30LGPL/main/

I already use the apache directory index for another page so I will not be able 
to modify this one. I guess the current index will «  not work » because glass 
expect some HTML structure.
Could you confirm and tell me what is the expected structure?

See my earlier mail with code.

Thanks Sven










Re: [Pharo-dev] 'VOMongoConnectionError' when dowloading mcz from smalltalkhub.com

2020-08-29 Thread Christophe Demarey
I hope this time it will be fine. I removed the newline characters.
I will also check if I can keep the original url in the browser rather than the 
redirected one.

Regards,
Christophe.

> Le 29 août 2020 à 18:41, Dale Henrichs  a 
> écrit :
> 
> Christophe,
> 
> Again, I appreciate your effort, but you are not quite there. 
> 
> If you look at SqueakSource or GemSource html (attached), the html uses  /> to create newlines on the page and in the Smalltalkhub page, you've 
> inserted a newline as part of the href filename, while SqueakSource and 
> GemSource (and the other monticello sites) do not include a newline in the 
> href, so for better or worse, in GemStone the method 
> MCReader>>canReadFileNamed: expects the filename to end with `.mcz` not a 
> newline and all of the filenames parsed from the SmalltalkHub page are 
> discarded as non-Monticello files.
> 
> So if you eliminate the trailing newline in the href, the GemStone code 
> should finally be happy.
> 
> Thanks for your effort!
> 
> Dale
> 
> 
> On 8/28/20 3:05 PM, Christophe Demarey wrote:
>> I understand that he new site is unusable if tools do not get the expected 
>> input. Sorry for the inconvenience. I was just not aware of this as Pharo 
>> uses the raw format.
>> I had to set up a CGI to produce an html listing from the raw file.
>> It should now be ok. Could you tell me if it works in GemStone now?
>> 
>> Christophe
>> 
>>> Le 28 août 2020 à 19:21, Dale Henrichs >> > a écrit :
>>> 
>>> Christophe,
>>> 
>>> I appreciate your effort, but I assume that you aren't finished yet?
>>> 
>>> When I use the url from your message I get a `Not Found` error (note that I 
>>> explicitly included a trailing slash in the url):
>>> 
>>> 
>>> 
>>> This is a bug (or omission) as well, since other Monticello sites produce 
>>> the expected HTML output with or without a trailing slash.
>>> 
>>> Without the trailing slash I get the same (non-html) response that was 
>>> causing a problem before ... although something must have changed, because 
>>> the GemStone command is now producing an error instead of an empty list ... 
>>> Once you get the static site to produce the expected output, I'm confident 
>>> that the GemStone errors will go away (this code has been running for about 
>>> a decade on all Monticello html repositories). 
>>> 
>>> I suggest that you look at what is produced on SqueakSource 
>>> (http://www.squeaksource.com/MooseSQL/ 
>>> ) as an example of expected output 
>>> of the mcz file listing ... this html page format has been used since 2003 
>>> for ALL (valid) monticello repositories.
>>> 
>>> If it is not clear, the `?format=raw` option is a recent Pharo only option 
>>> and when the dynamic site was running and the `?format-raw` option was 
>>> OMITTED it produced output compatible with 
>>> (http://www.squeaksource.com/MooseSQL/ 
>>> ) ... 
>>> 
>>> Restoring HTML is *REQUIRED* for the static site to be a faithful 
>>> replacement of the dynamic site... the format that is produced today is 
>>> only compatible with the Pharo only `?format=raw` option.
>>> 
>>> Bug or omission ... the static site is currently in worse shape than the 
>>> dynamic site before the swap ...
>>> 
>>> Dale
>>> 
>>> On 8/28/20 2:12 AM, Christophe Demarey wrote:
> Le 28 août 2020 à 10:15, Sven Van Caekenberghe  
>  a écrit :
> 
> 
> 
>> On 28 Aug 2020, at 10:13, Christophe Demarey 
>>   wrote:
>> 
>> Smalltalkhub is now to able to distinguish between raw and not raw mcz 
>> listing requests.
>> Ex: 
>>  http://smalltalkhub.com/mc/Seaside/Seaside30LGPL/main?format=raw 
>> 
>>  http://smalltalkhub.com/mc/Seaside/Seaside30LGPL/main/ 
>> 
>> 
>> I already use the apache directory index for another page so I will not 
>> be able to modify this one. I guess the current index will «  not work » 
>> because glass expect some HTML structure.
>> Could you confirm and tell me what is the expected structure?
> See my earlier mail with code.
 Thanks Sven
 
 
>> 
> 



[Pharo-dev] [Pharo 9.0] Build #630: hasLiteralSuchThat-do-not-look-into-Pragmas

2020-08-29 Thread ci-pharo-ci-jenkins2
There is a new Pharo build available!
  
The status of the build #630 was: FAILURE.

The Pull Request #7183 was integrated: 
"hasLiteralSuchThat-do-not-look-into-Pragmas"
Pull request url: https://github.com/pharo-project/pharo/pull/7183

Issue Url: https://github.com/pharo-project/pharo/issues/hasLiteralSuchThat
Build Url: 
https://ci.inria.fr/pharo-ci-jenkins2/job/Test%20pending%20pull%20request%20and%20branch%20Pipeline/job/Pharo9.0/630/


[Pharo-dev] [Pharo 9.0] Build #629: TestCase-use-classCommentBlank

2020-08-29 Thread ci-pharo-ci-jenkins2
There is a new Pharo build available!
  
The status of the build #629 was: FAILURE.

The Pull Request #7205 was integrated: "TestCase-use-classCommentBlank"
Pull request url: https://github.com/pharo-project/pharo/pull/7205

Issue Url: https://github.com/pharo-project/pharo/issues/TestCase
Build Url: 
https://ci.inria.fr/pharo-ci-jenkins2/job/Test%20pending%20pull%20request%20and%20branch%20Pipeline/job/Pharo9.0/629/