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

[Pharo 7.0] Build #33: 22755-Reflectivity-cleanups-tests-unify-variable-Naming

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

There is a new Pharo build available! The status of the build #33 was: SUCCESS. The Pull Request #2063 was integrated: "22755-Reflectivity-cleanups-tests-unify-variable-Naming" Pull request url: https://github.com/pharo-project/pharo/pull/2063 Issue Url: https://pharo.fogbugz.com/f/cases/22755 Build Url: https://ci.inria.fr/pharo-ci-jenkins2/job/Test%20pending%20pull%20request%2…

Dec. 12, 2018

[Pharo 7.0] Build #32: 22750-Add-GTPlayground-classopenUrl

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

There is a new Pharo build available! The status of the build #32 was: FAILURE. The Pull Request #2057 was integrated: "22750-Add-GTPlayground-classopenUrl" Pull request url: https://github.com/pharo-project/pharo/pull/2057 Issue Url: https://pharo.fogbugz.com/f/cases/22750 Build Url: https://ci.inria.fr/pharo-ci-jenkins2/job/Test%20pending%20pull%20request%2…

Dec. 12, 2018

Re: [Pharo-dev] Multiline method definition in Tonel

by Esteban Lorenzano

Yes, it should preserve the format. If not, there is bug :) Esteban > On 12 Dec 2018, at 15:59, Guillermo Polito <guillermopolito(a)gmail.com> wrote: > > Usually Esteban was picky to be able to export and import methods maintaining the original indentation and formatting ^^. > > On Mon, Dec 10, 2018 at 9:03 PM Mariano Martinez Peck <marianopeck(a)gmail.com <mailto:marianopeck@gmail.com>> wrote: > > > On Sun, Dec 9, 2018 at 6:28 PM Peter Uhnak <i.uhnak(a)gmail.com <mailto:i.uhnak@gmail.com>> wrote: > That's just whitespace... or more specifically any of these are fine (TonelParser>>extractSelector:) > > ``` > separators := { > Character space. > Character tab. > Character lf. > Character newPage. > Character cr. > $: "not me though"}. > ``` > > You can also try to parse it directly and see the result: > > ``` > def := 'Class { > #name : #Whatever, > #superclass : #Object, > #category : ''Wherever'' > } > > { #category : #protocol } > Whatever >> testWith: arg1 > with: arg2 > with: arg3 [ > ^ 42 > ] > '. > TonelParser parseStream: def readStream. > ``` > > > Thanks Peter! I was afraid that TonelParser would break if tried to parse multiline method declarations. > > Note that exporting it will result in a "normalized" format, so if you keep it in git you will see a change (this also applies to the ordering of methods and some optional stuff) > > > OK. Thanks for letting me know. And for "normalized" you mean a classical oneliner right? > > Thanks! > > > Peter > > On Fri, Dec 7, 2018 at 5:57 PM Mariano Martinez Peck <marianopeck(a)gmail.com <mailto:marianopeck@gmail.com>> wrote: > Hi guys, > > I was reading the Tonel spec here [1] but it is not clear to me the case of the multiline method definitions. Are those allowed or not? Say I have a method like this: > > testWith: arg1 > with: arg2 > with: arg3 > > ^ 42 > > Is that valid as Tonel format? Or tonel expects to have method definition in a one liner like: > > testWith: arg1 with: arg2 with: arg3 > > ^ 42 > > > Thanks in advance, > > > [1] https://github.com/pharo-vcs/tonel <https://github.com/pharo-vcs/tonel> > > -- > Mariano > https://twitter.com/MartinezPeck <https://twitter.com/MartinezPeck> > http://marianopeck.wordpress.com <http://marianopeck.wordpress.com/> > > > -- > Mariano > https://twitter.com/MartinezPeck <https://twitter.com/MartinezPeck> > http://marianopeck.wordpress.com <http://marianopeck.wordpress.com/> > > > -- > > 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

Dec. 12, 2018

Re: [Pharo-dev] Fixed by reboot

by phil＠highoctane.be

Logging out and back in also solves the issue for me. I guess that a dll handle is not released properly. Phil On Wed, Dec 12, 2018, 16:14 Guillermo Polito <guillermopolito(a)gmail.com wrote: > > > On Sun, Dec 9, 2018 at 2:26 AM Ben Coman <btc(a)openinworld.com> wrote: > >> Just reporting a strange occurrence in case anyone sees the same. >> >> I was using PharoLauncher fine, then it wasn't. >> When trying to launch an image it started reporting with "ProcessWrapper >> newProcess failed..." >> > > Has your Pharo launcher been open for many time/hours/days? > > >> >> First time I've ever seen that, and fixed by a reboot, but still strange. >> There are a couple of other troubles that get fixed by reboot. >> Anyone have insight into an underlying cause? >> > > I've started using windows two days a week to spot more issues and be more > efficiente to fix them... > And I've seen many issues regarding DLL opening/loading both by VM plugins > and FFI calls. >

Dec. 12, 2018

[Pharo 7.0] Build #31: 22752-cleanup-preamble-generation--test-for-method-Sequence

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

There is a new Pharo build available! The status of the build #31 was: FAILURE. The Pull Request #2060 was integrated: "22752-cleanup-preamble-generation--test-for-method-Sequence" Pull request url: https://github.com/pharo-project/pharo/pull/2060 Issue Url: https://pharo.fogbugz.com/f/cases/22752 Build Url: https://ci.inria.fr/pharo-ci-jenkins2/job/Test%20pending%20pull%20request%2…

Dec. 12, 2018

Re: [Pharo-dev] Fixed by reboot

by Ben Coman

On Wed, 12 Dec 2018 at 23:14, Guillermo Polito <guillermopolito(a)gmail.com> wrote: > > > On Sun, Dec 9, 2018 at 2:26 AM Ben Coman <btc(a)openinworld.com> wrote: > >> Just reporting a strange occurrence in case anyone sees the same. >> >> I was using PharoLauncher fine, then it wasn't. >> When trying to launch an image it started reporting with "ProcessWrapper >> newProcess failed..." >> > > Has your Pharo launcher been open for many time/hours/days? > Yes, many days. Including across laptop going to sleep morning and evening travelling between home and office. I like it not-quitting on launch. cheers -ben

Dec. 12, 2018

[ANN] Git Thermite integration with Iceberg

by Ronie Salgado

Hello, Some time ago I presented Git Thermite in this mailing. Git Thermite is a tool for visualizing Git commits that I am making for my master thesis. Now I have managed to integrate Git Thermite with Iceberg. GitHub page: https://github.com/ronsaldo/pharo-git-thermite Demo video: https://www.youtube.com/watch?v=TdBHpdyFgZE Visualization learning material: https://github.com/ronsaldo/pharo-git-thermite/blob/master/doc/learning-mat… Feedback: https://docs.google.com/forms/d/e/1FAIpQLSeir6VlE3bR78oRsNAp9eHLkUn2Q016wEl… Pharo 7 installation doIt: Metacello new baseline: 'Thermite'; repository: 'github://ronsaldo/pharo-git-thermite/src'; load Please, I would love to get some usage feedback through the Google form :-) Best regards, Ronie

Dec. 12, 2018

Re: [Pharo-dev] Fixed by reboot

by Guillermo Polito

On Sun, Dec 9, 2018 at 2:26 AM Ben Coman <btc(a)openinworld.com> wrote: > Just reporting a strange occurrence in case anyone sees the same. > > I was using PharoLauncher fine, then it wasn't. > When trying to launch an image it started reporting with "ProcessWrapper > newProcess failed..." > Has your Pharo launcher been open for many time/hours/days? > > First time I've ever seen that, and fixed by a reboot, but still strange. > There are a couple of other troubles that get fixed by reboot. > Anyone have insight into an underlying cause? > I've started using windows two days a week to spot more issues and be more efficiente to fix them... And I've seen many issues regarding DLL opening/loading both by VM plugins and FFI calls.

Dec. 12, 2018

Re: [Pharo-dev] OSSubprocess equivalent in Pharo7?

by Guillermo Polito

Hi Hans, On Thu, Dec 6, 2018 at 12:14 AM Hans-Martin <pharo-dev(a)fam-mosner.de> wrote: > Hello, > I noticed that theOSSubprocess functionality for communicating with OS > subprocesses depends on OSSAttachableFileStream which is a subclass of > deprecated FileStream OSSubprocess works in Pharo7. Yes, we should find a replacement/solution for the deprecation of FileStream, but it's not deprecated because it does not work anymore but because the new version is cleaner, clearer and more flexible. So all existing code should continue working. > (incidentally, the project is not marked as being > compatible with Pharo7). > Are you talking about the catalog? Well, since Pharo7 is not officially released OSSubprocess has not been marked yet, that's all. > The popen()/fgetc() based examples in UNIXProcess are blocking, which is > unusable for any but the most basic external command executions. > Are there plans to implement a proper replacement in Pharo7? > Can you check if OSSubproces tests are running green for you? Are there tests cases for cases similar to popen/fgetc? I'm interested in knowing what are exactly the problems you've found. Thanks, Guille > Cheers, > Hans-Martin > > > > -- > Sent from: http://forum.world.st/Pharo-Smalltalk-Developers-f1294837.html > > -- 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

Dec. 12, 2018

[Pharo 7.0] Build #30: 22742 Package ManifestHiedra should be tagged

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

There is a new Pharo build available! The status of the build #30 was: SUCCESS. The Pull Request #2051 was integrated: "22742 Package ManifestHiedra should be tagged" Pull request url: https://github.com/pharo-project/pharo/pull/2051 Issue Url: https://pharo.fogbugz.com/f/cases/22742 Build Url: https://ci.inria.fr/pharo-ci-jenkins2/job/Test%20pending%20pull%20request%2…

Dec. 12, 2018