May 2018 - Pharo-dev - lists.pharo.org

[Pharo 7.0-dev] Build #831: 21821-Integrate-Iceberg-075

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

There is a new Pharo build available! The status of the build #831 was: SUCCESS. The Pull Request #1287 was integrated: "21821-Integrate-Iceberg-075" Pull request url: https://github.com/pharo-project/pharo/pull/1287 Issue Url: https://pharo.fogbugz.com/f/cases/21821 Build Url: https://ci.inria.fr/pharo-ci-jenkins2/job/Test%20pending%20pull%20request%2…

May 3, 2018

Re: [Pharo-dev] Do we kill the catalog?

by Guillermo Polito

Just for the record https://pharo.fogbugz.com/f/cases/21825/Improve-Catalog-loading https://github.com/pharo-project/pharo/pull/1292 Feel free to review or to propose other PRs, Guille

May 3, 2018

Re: [Pharo-dev] [Ann] Iceberg 0.7.5

by Cyril Ferlicot D.

On 03/05/2018 17:16, Guillermo Polito wrote: > Time for the weekly Iceberg update. > > Iceberg 0.7.5. will be in the next Pharo build. > Thanks to all brave users, issue reporters and contributors :). > > Key improvements:Â > Â - Several improvements in metacello integration. (see #625, #664, #688 > + more tests) > Â - For those using CI, we think this release will also > fixÂ https://github.com/pharo-vcs/iceberg/issues/643 > > Infrastructure improvements: > Â - In 0.7.4 we introduced in the build tests againt pharo 6.1 and 64 bits > Â Â * https://travis-ci.org/pharo-vcs/iceberg/builds/372408795 > Â - In 0.7.5 all iceberg tests run green on Pharo 6.1 32 bits > Â Â * https://travis-ci.org/pharo-vcs/iceberg/builds/374433920 > Â - In the next release we plan to do a pass on 64 bits > > ** Pharo 6.1 backport ** > Â - We plan to backport this version to Pharo 6.1 > Â - Cyril has been using the development version of Iceberg on Pharo 6.1 > for a couple of weeks now. > Â - This plus the green CI encourages us to backport to Pharo 6.1 > > Detailed ChangesLog: > > #625 Non explicit error while loading a project > #758 [Regression] Repair action clone is not setting pharo repository > #756 Iceberg dev-0.7 is broken in Pharo6.1 (Instance of > IceTipURLLabelMorph did not understand #addEmphasis:) > #749 Adding repair actions to the code subdirectory missing problem > #655 Rename extension method buildToolbarItem of CommandActivator > #755 Extracting the URL label behavior as a component that we can reuse > #468 Commiting via Iceberg does not commit package removal > #750 Add possibility to see current commit in Repositories/Package view > #754 Showing and coping the Commit ID > #753 Create branch dialog layout is broken > #664 Package wrongly shown with "uncommited changes" status > #752 Add "invalid ssh" > #747 Cloning a Git repository should change the path with the project name. > #738 Remove "pharo" Repository with "also remove from filesystem" gives > error > #746 Make tests run on pharo 6 > #735 Issue while registering a new project on Pharo 6.1 > #733 Add license file > #740 Fogbugz panel: Improve label, focus order and layout > #549 IceRestoreRepositoryModel should have a class comment > #688 Duplicated project error when loading in a fresh image > > Enjoy, > Guille in behalf of all people that contributed :) Hi, With this integration, SmalltalkCI for Pharo 7 is green again. https://travis-ci.org/TelescopeSt/Telescope/jobs/373994423 Thank you -- Cyril Ferlicot https://ferlicot.fr

May 3, 2018

Aboud CI test retries

by Guillermo Polito

Hi all, Initially the pharo build had tests that were failing randomly. To cope with that, we introduced a retry of the tests. Nowadays, this situation is actually very rare. Tests that fail, fail always, and randomly failing tests are not seen so often... This means however, that in the case that a test is persistently failing, we are (uselessly) retrying it, and making jobs take 10-15 minutes longer for nothing. I propose that we remove the retries. - This will speed up the builds that are green and only penalize those that are not green. - Remove stress from our servers (that we use to have a higher ratio builds/hour :)) - Randomly failing tests will just need to manually retry the build. But since green builds take now ~15-20 minutes, which is in the same order of magnitude of the retries, we only penalize the one that found the hiccup. Guille

May 3, 2018

Re: [Pharo-dev] [Ann] Iceberg 0.7.5

by Guillermo Polito

On Thu, May 3, 2018 at 5:25 PM, Norbert Hartl <norbert(a)hartl.name> wrote: > Will it be too hard to do 6.2? I encourage the backport but it will break > stuff which is also really annoying. > @esteban? :) I'd really like to have that or a 6.1.1, a way to actually differentiate them... > > Norbert > > > Am 03.05.2018 um 17:16 schrieb Guillermo Polito <guillermopolito(a)gmail.com > >: > > Time for the weekly Iceberg update. > > Iceberg 0.7.5. will be in the next Pharo build. > Thanks to all brave users, issue reporters and contributors :). > > Key improvements: > - Several improvements in metacello integration. (see #625, #664, #688 + > more tests) > - For those using CI, we think this release will also fix > https://github.com/pharo-vcs/iceberg/issues/643 > > Infrastructure improvements: > - In 0.7.4 we introduced in the build tests againt pharo 6.1 and 64 bits > * https://travis-ci.org/pharo-vcs/iceberg/builds/372408795 > - In 0.7.5 all iceberg tests run green on Pharo 6.1 32 bits > * https://travis-ci.org/pharo-vcs/iceberg/builds/374433920 > - In the next release we plan to do a pass on 64 bits > > ** Pharo 6.1 backport ** > - We plan to backport this version to Pharo 6.1 > - Cyril has been using the development version of Iceberg on Pharo 6.1 > for a couple of weeks now. > - This plus the green CI encourages us to backport to Pharo 6.1 > > Detailed ChangesLog: > > #625 Non explicit error while loading a project > #758 [Regression] Repair action clone is not setting pharo repository > #756 Iceberg dev-0.7 is broken in Pharo6.1 (Instance of > IceTipURLLabelMorph did not understand #addEmphasis:) > #749 Adding repair actions to the code subdirectory missing problem > #655 Rename extension method buildToolbarItem of CommandActivator > #755 Extracting the URL label behavior as a component that we can reuse > #468 Commiting via Iceberg does not commit package removal > #750 Add possibility to see current commit in Repositories/Package view > #754 Showing and coping the Commit ID > #753 Create branch dialog layout is broken > #664 Package wrongly shown with "uncommited changes" status > #752 Add "invalid ssh" > #747 Cloning a Git repository should change the path with the project name. > #738 Remove "pharo" Repository with "also remove from filesystem" gives > error > #746 Make tests run on pharo 6 > #735 Issue while registering a new project on Pharo 6.1 > #733 Add license file > #740 Fogbugz panel: Improve label, focus order and layout > #549 IceRestoreRepositoryModel should have a class comment > #688 Duplicated project error when loading in a fresh image > > Enjoy, > Guille in behalf of all people that contributed :) > > > -- Guille Polito Research Engineer Centre de Recherche en Informatique, Signal et Automatique de Lille CRIStAL - UMR 9189 French National Center for Scientific Research - *http://www.cnrs.fr <http://www.cnrs.fr>* *Web:* *http://guillep.github.io* <http://guillep.github.io> *Phone: *+33 06 52 70 66 13

May 3, 2018

[Pharo 7.0-dev] Build #830: 21765-cleanup-interactiveParserFor-

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

There is a new Pharo build available! The status of the build #830 was: SUCCESS. The Pull Request #1284 was integrated: "21765-cleanup-interactiveParserFor-" Pull request url: https://github.com/pharo-project/pharo/pull/1284 Issue Url: https://pharo.fogbugz.com/f/cases/21765 Build Url: https://ci.inria.fr/pharo-ci-jenkins2/job/Test%20pending%20pull%20request%2…

May 3, 2018

Re: [Pharo-dev] [Ann] Iceberg 0.7.5

by Norbert Hartl

Will it be too hard to do 6.2? I encourage the backport but it will break stuff which is also really annoying. Norbert > Am 03.05.2018 um 17:16 schrieb Guillermo Polito <guillermopolito(a)gmail.com>: > > Time for the weekly Iceberg update. > > Iceberg 0.7.5. will be in the next Pharo build. > Thanks to all brave users, issue reporters and contributors :). > > Key improvements: > - Several improvements in metacello integration. (see #625, #664, #688 + more tests) > - For those using CI, we think this release will also fix https://github.com/pharo-vcs/iceberg/issues/643 <https://github.com/pharo-vcs/iceberg/issues/643> > > Infrastructure improvements: > - In 0.7.4 we introduced in the build tests againt pharo 6.1 and 64 bits > * https://travis-ci.org/pharo-vcs/iceberg/builds/372408795 <https://travis-ci.org/pharo-vcs/iceberg/builds/372408795> > - In 0.7.5 all iceberg tests run green on Pharo 6.1 32 bits > * https://travis-ci.org/pharo-vcs/iceberg/builds/374433920 <https://travis-ci.org/pharo-vcs/iceberg/builds/374433920> > - In the next release we plan to do a pass on 64 bits > > ** Pharo 6.1 backport ** > - We plan to backport this version to Pharo 6.1 > - Cyril has been using the development version of Iceberg on Pharo 6.1 for a couple of weeks now. > - This plus the green CI encourages us to backport to Pharo 6.1 > > Detailed ChangesLog: > > #625 Non explicit error while loading a project > #758 [Regression] Repair action clone is not setting pharo repository > #756 Iceberg dev-0.7 is broken in Pharo6.1 (Instance of IceTipURLLabelMorph did not understand #addEmphasis:) > #749 Adding repair actions to the code subdirectory missing problem > #655 Rename extension method buildToolbarItem of CommandActivator > #755 Extracting the URL label behavior as a component that we can reuse > #468 Commiting via Iceberg does not commit package removal > #750 Add possibility to see current commit in Repositories/Package view > #754 Showing and coping the Commit ID > #753 Create branch dialog layout is broken > #664 Package wrongly shown with "uncommited changes" status > #752 Add "invalid ssh" > #747 Cloning a Git repository should change the path with the project name. > #738 Remove "pharo" Repository with "also remove from filesystem" gives error > #746 Make tests run on pharo 6 > #735 Issue while registering a new project on Pharo 6.1 > #733 Add license file > #740 Fogbugz panel: Improve label, focus order and layout > #549 IceRestoreRepositoryModel should have a class comment > #688 Duplicated project error when loading in a fresh image > > Enjoy, > Guille in behalf of all people that contributed :)

May 3, 2018

Re: [Pharo-dev] Failing tests

by Guillermo Polito

Exactly that :) https://pharo.fogbugz.com/f/cases/21822/Fix-Zn-Tests-due-to-encrypted-googl… https://github.com/pharo-project/pharo/pull/1290 On Thu, May 3, 2018 at 5:18 PM, Sven Van Caekenberghe <sven(a)stfx.eu> wrote: > > > > On 3 May 2018, at 17:12, Guillermo Polito <guillermopolito(a)gmail.com> > wrote: > > > > Hi all, > > > > There are a couple of Zn tests that are failing not even in the CI but > locally in my machine. > > This happens because the tests are trying to do a request to > > > > encrypted.google.com > > > > and expecting a 200, but they are instead receiving: > > > > 301 Moved Permanently text/html > > > > From the response, I guess we should do the request to > > > > https://www.google.com/search?q=Smalltalk > > > > instead. > > > > This solves the failures on my side. Sven, is that Ok if we push those > patches to the tests to Pharo? > > Right, these would be the changed versions then: > > testPlain > | query stream request response | > self isNativeSSLPluginPresent ifFalse: [ ^ self ]. > query := 'Smalltalk'. > stream := ZdcSecureSocketStream openConnectionToHostNamed: ' > www.google.com' port: 443. > [ > request := String streamContents: [ :out | > out << 'GET /search?q=' << query << ' HTTP/1.1' << > String crlf. > out << 'Host: www.google.com' << String crlf. > out << 'Connection: close' << String crlf. > out << String crlf ]. > stream connect. > stream nextPutAll: request asByteArray. > stream flush. > response := stream upToEnd asString. > ] ensure: [ stream close ]. > self assert: (response includesSubstring: '200 OK'). > self assert: (response includesSubstring: 'Google'). > self assert: (response includesSubstring: 'Smalltalk'). > > testRequestResponse > | query stream request response | > self ensureSocketStreamFactory. > self isNativeSSLPluginPresent ifFalse: [ ^ self ]. > query := 'Smalltalk'. > stream := ZdcSecureSocketStream openConnectionToHostNamed: ' > www.google.com' port: 443. > [ > request := ZnRequest get: 'https://www.google.com/ > search?q=', query. > stream connect. > request writeOn: stream. > stream flush. > response := ZnResponse readFrom: stream. > ] ensure: [ stream close ]. > self assert: response isSuccess. > self assert: (response contents includesSubstring: 'Google'). > self assert: (response contents includesSubstring: 'Smalltalk'). > > I will apply them upstream as well. > > > -- > > > > Guille Polito > > Research Engineer > > > > Centre de Recherche en Informatique, Signal et Automatique de Lille > > CRIStAL - UMR 9189 > > French National Center for Scientific Research - http://www.cnrs.fr > > > > Web: http://guillep.github.io > > Phone: +33 06 52 70 66 13 > > > -- Guille Polito Research Engineer Centre de Recherche en Informatique, Signal et Automatique de Lille CRIStAL - UMR 9189 French National Center for Scientific Research - *http://www.cnrs.fr <http://www.cnrs.fr>* *Web:* *http://guillep.github.io* <http://guillep.github.io> *Phone: *+33 06 52 70 66 13

May 3, 2018

Re: [Pharo-dev] Failing tests

by Sven Van Caekenberghe

> On 3 May 2018, at 17:12, Guillermo Polito <guillermopolito(a)gmail.com> wrote: > > Hi all, > > There are a couple of Zn tests that are failing not even in the CI but locally in my machine. > This happens because the tests are trying to do a request to > > encrypted.google.com > > and expecting a 200, but they are instead receiving: > > 301 Moved Permanently text/html > > From the response, I guess we should do the request to > > https://www.google.com/search?q=Smalltalk > > instead. > > This solves the failures on my side. Sven, is that Ok if we push those patches to the tests to Pharo? Right, these would be the changed versions then: testPlain | query stream request response | self isNativeSSLPluginPresent ifFalse: [ ^ self ]. query := 'Smalltalk'. stream := ZdcSecureSocketStream openConnectionToHostNamed: 'www.google.com' port: 443. [ request := String streamContents: [ :out | out << 'GET /search?q=' << query << ' HTTP/1.1' << String crlf. out << 'Host: www.google.com' << String crlf. out << 'Connection: close' << String crlf. out << String crlf ]. stream connect. stream nextPutAll: request asByteArray. stream flush. response := stream upToEnd asString. ] ensure: [ stream close ]. self assert: (response includesSubstring: '200 OK'). self assert: (response includesSubstring: 'Google'). self assert: (response includesSubstring: 'Smalltalk'). testRequestResponse | query stream request response | self ensureSocketStreamFactory. self isNativeSSLPluginPresent ifFalse: [ ^ self ]. query := 'Smalltalk'. stream := ZdcSecureSocketStream openConnectionToHostNamed: 'www.google.com' port: 443. [ request := ZnRequest get: 'https://www.google.com/search?q=', query. stream connect. request writeOn: stream. stream flush. response := ZnResponse readFrom: stream. ] ensure: [ stream close ]. self assert: response isSuccess. self assert: (response contents includesSubstring: 'Google'). self assert: (response contents includesSubstring: 'Smalltalk'). I will apply them upstream as well. > -- > > Guille Polito > Research Engineer > > Centre de Recherche en Informatique, Signal et Automatique de Lille > CRIStAL - UMR 9189 > French National Center for Scientific Research - http://www.cnrs.fr > > Web: http://guillep.github.io > Phone: +33 06 52 70 66 13

May 3, 2018

[Ann] Iceberg 0.7.5

by Guillermo Polito

Time for the weekly Iceberg update. Iceberg 0.7.5. will be in the next Pharo build. Thanks to all brave users, issue reporters and contributors :). Key improvements: - Several improvements in metacello integration. (see #625, #664, #688 + more tests) - For those using CI, we think this release will also fix https://github.com/pharo-vcs/iceberg/issues/643 Infrastructure improvements: - In 0.7.4 we introduced in the build tests againt pharo 6.1 and 64 bits * https://travis-ci.org/pharo-vcs/iceberg/builds/372408795 - In 0.7.5 all iceberg tests run green on Pharo 6.1 32 bits * https://travis-ci.org/pharo-vcs/iceberg/builds/374433920 - In the next release we plan to do a pass on 64 bits ** Pharo 6.1 backport ** - We plan to backport this version to Pharo 6.1 - Cyril has been using the development version of Iceberg on Pharo 6.1 for a couple of weeks now. - This plus the green CI encourages us to backport to Pharo 6.1 Detailed ChangesLog: #625 Non explicit error while loading a project #758 [Regression] Repair action clone is not setting pharo repository #756 Iceberg dev-0.7 is broken in Pharo6.1 (Instance of IceTipURLLabelMorph did not understand #addEmphasis:) #749 Adding repair actions to the code subdirectory missing problem #655 Rename extension method buildToolbarItem of CommandActivator #755 Extracting the URL label behavior as a component that we can reuse #468 Commiting via Iceberg does not commit package removal #750 Add possibility to see current commit in Repositories/Package view #754 Showing and coping the Commit ID #753 Create branch dialog layout is broken #664 Package wrongly shown with "uncommited changes" status #752 Add "invalid ssh" #747 Cloning a Git repository should change the path with the project name. #738 Remove "pharo" Repository with "also remove from filesystem" gives error #746 Make tests run on pharo 6 #735 Issue while registering a new project on Pharo 6.1 #733 Add license file #740 Fogbugz panel: Improve label, focus order and layout #549 IceRestoreRepositoryModel should have a class comment #688 Duplicated project error when loading in a fresh image Enjoy, Guille in behalf of all people that contributed :)

May 3, 2018