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

Re: [Pharo-dev] Change Pharo window icon

by Vincent.Blondeau＠lamresearch.com

Hi, In the way to change dynamically the icon of the main window, I am adding a primitive in the HostWindow VM Plugin. So, I implemented the primitive call on Pharo side: https://github.com/pharo-project/pharo/pull/1198 Modified the C plugin file: https://github.com/OpenSmalltalk/opensmalltalk-vm/pull/248 (only the windows version for now) And implemented the binding in VMMaker, and I would commit this last one. However, I do not have the write access to http://source.squeak.org/VMMaker. Could someone grant me to it? Thanks in advance, Vincent From: Blondeau, Vincent Sent: Tuesday, February 6, 2018 18:36 To: Pharo Development List <pharo-dev(a)lists.pharo.org> Subject: RE: [Pharo-dev] Change Pharo window icon Hi, The fact that he mentioned an ability to change window's title from inside of the running image suggested me that he actually wants to change an icon at runtime too. Indeed, that is what I would like to do. Cheers, Vincent From: Pharo-dev [mailto:pharo-dev-bounces@lists.pharo.org] On Behalf Of Aliaksei Syrel Sent: Saturday, February 3, 2018 8:09 To: Pharo Development List <pharo-dev(a)lists.pharo.org<mailto:pharo-dev@lists.pharo.org>> Subject: Re: [Pharo-dev] Change Pharo window icon Hi Eliot, I thought Vincent wants to change Pharo main window icon (image). I would like to know how to change the main Pharo window icon. The fact that he mentioned an ability to change window's title from inside of the running image suggested me that he actually wants to change an icon at runtime too. DisplayScreen class >> primitiveWindowTitle:string: I may be wrong :) But in any case it would be nice to be able to change window icon from the image. Cheers, Alex On 3 February 2018 at 17:01, Eliot Miranda <eliot.miranda(a)gmail.com<mailto:eliot.miranda@gmail.com>> wrote: Hi Aliaksei, On Feb 3, 2018, at 4:30 AM, Aliaksei Syrel <alex.syrel(a)gmail.com<mailto:alex.syrel@gmail.com>> wrote: Hi Since Pharo uses sdl2 (or should use) https://wiki.libsdl.org/SDL_SetWindowIcon<https://urldefense.proofpoint.com/v2/url?u=https-3A__wiki.libsdl.org_SDL-5F…> Hang on. Changing a window's icon is not at all the same as changing the VM's icon. Which are we talking about here? I thought we were talking about changing the VM icon. Alex

April 13, 2018

Re: [Pharo-dev] [Pharo70 alpha] Store settings raise a DNU primFlush

by Sven Van Caekenberghe

> On 13 Apr 2018, at 22:26, Alistair Grant <akgrant0710(a)gmail.com> wrote: > > Hi Stef & Sven, > > On 13 April 2018 at 20:38, Stephane Ducasse <stepharo.self(a)gmail.com> wrote: >> Hi >> >> I do not know if this is related to the OldStreamCleaningEffort but >> when I press store settings in the Settings browser I get a primitive >> failed. >> >> https://pharo.fogbugz.com/f/cases/21699/Store-settings-raise-a-DNU-primFlush > > > This is because the stream created in > SystemSettingsPersistence>>storeExactStoredSettings: is closed twice, > first in SettingsStonWriter>>store and then in > SystemSettingsPersistence>>storeExactStoredSettings:. > > Previous versions of file streams have allowed the stream #close to be > called multiple times, however ZnBufferedReadWriteStream doesn't allow > this. > > I think that since an already open stream is handed to > SettingsStonWriter, it shouldn't be closing the stream. That will > resolve this particular problem. > > It can also be argued that ZnBufferedReadWriteStream and co. should > allow streams to be closed multiple times for backward compatibility. > Sven? Yes, OK, making #close idempotent is good. If we add #closed to ZnBufferedReadStream we can add a self closed ifFalse: [ .. ] guard in #close. > Cheers, > Alistair >

April 13, 2018

Re: [Pharo-dev] [Pharo70 alpha] Store settings raise a DNU primFlush

by Alistair Grant

Hi Stef & Sven, On 13 April 2018 at 20:38, Stephane Ducasse <stepharo.self(a)gmail.com> wrote: > Hi > > I do not know if this is related to the OldStreamCleaningEffort but > when I press store settings in the Settings browser I get a primitive > failed. > > https://pharo.fogbugz.com/f/cases/21699/Store-settings-raise-a-DNU-primFlush This is because the stream created in SystemSettingsPersistence>>storeExactStoredSettings: is closed twice, first in SettingsStonWriter>>store and then in SystemSettingsPersistence>>storeExactStoredSettings:. Previous versions of file streams have allowed the stream #close to be called multiple times, however ZnBufferedReadWriteStream doesn't allow this. I think that since an already open stream is handed to SettingsStonWriter, it shouldn't be closing the stream. That will resolve this particular problem. It can also be argued that ZnBufferedReadWriteStream and co. should allow streams to be closed multiple times for backward compatibility. Sven? Cheers, Alistair

April 13, 2018

Re: [Pharo-dev] Iceberg2.0 crash test passed :)

by Serge Stinckwich

On Fri, Apr 13, 2018 at 8:54 PM, Esteban Lorenzano <estebanlm(a)gmail.com> wrote: > > > On 13 Apr 2018, at 21:43, Serge Stinckwich <serge.stinckwich(a)gmail.com> > wrote: > > Yes I was able to commit also. > Very slick interface :-) > > > But now, every time I want to commit, my image crash and I have: > > Assertion failed: (git_atomic_get(&git__n_inits) > 0), function > git__global_state, file /Users/travis/build/OpenSmalltalk/opensmalltalk- > vm/build.macos64x64/pharo.cog.spur/build/third-party/libgit2-0.25.1/src/global.c, > line 322. > > > there is a current bug on initialisation (no idea why, still > investigating). > > this is fixed by executing: > > LGitLibrary initialize. > > âThank you Esteban, you save my time :-) â âApparently this crash happens after I relaunch my image with Iceberg running.â -- Serge Stinckwich UMI UMMISCO 209 (SU/IRD/UY1) "Programs must be written for people to read, and only incidentally for machines to execute."http://www.doesnotunderstand.org/

April 13, 2018

Re: [Pharo-dev] Iceberg2.0 crash test passed :)

by Esteban Lorenzano

> On 13 Apr 2018, at 21:43, Serge Stinckwich <serge.stinckwich(a)gmail.com> wrote: > > Yes I was able to commit also. > Very slick interface :-) > > > But now, every time I want to commit, my image crash and I have: > > Assertion failed: (git_atomic_get(&git__n_inits) > 0), function git__global_state, file /Users/travis/build/OpenSmalltalk/opensmalltalk-vm/build.macos64x64/pharo.cog.spur/build/third-party/libgit2-0.25.1/src/global.c, line 322. there is a current bug on initialisation (no idea why, still investigating). this is fixed by executing: LGitLibrary initialize. cheers! Esteban > > > On Fri, Apr 13, 2018 at 8:02 PM, Stephane Ducasse <stepharo.self(a)gmail.com <mailto:stepharo.self@gmail.com>> wrote: > I could commit my first commit with iceberg 2.0 > Well done > > It feels nicer :). > > Stef > > > > > -- > Serge Stinckwich > UMI UMMISCO 209 (SU/IRD/UY1) > "Programs must be written for people to read, and only incidentally for machines to execute." > http://www.doesnotunderstand.org/ <http://www.doesnotunderstand.org/>

April 13, 2018

Re: [Pharo-dev] Iceberg2.0 crash test passed :)

by Serge Stinckwich

Yes Pharo 7.0, latest VM 64 bits On Fri, Apr 13, 2018 at 8:46 PM, Guillermo Polito <guillermopolito(a)gmail.com > wrote: > Is that Pharo 7? > > On Fri, Apr 13, 2018 at 9:43 PM, Serge Stinckwich < > serge.stinckwich(a)gmail.com> wrote: > >> Yes I was able to commit also. >> Very slick interface :-) >> >> >> But now, every time I want to commit, my image crash and I have: >> >> Assertion failed: (git_atomic_get(&git__n_inits) > 0), function >> git__global_state, file /Users/travis/build/OpenSmallt >> alk/opensmalltalk-vm/build.macos64x64/pharo.cog.spur/ >> build/third-party/libgit2-0.25.1/src/global.c, line 322. >> >> >> On Fri, Apr 13, 2018 at 8:02 PM, Stephane Ducasse < >> stepharo.self(a)gmail.com> wrote: >> >>> I could commit my first commit with iceberg 2.0 >>> Well done >>> >>> It feels nicer :). >>> >>> Stef >>> >>> >> >> >> -- >> Serge Stinckwich >> UMI UMMISCO 209 (SU/IRD/UY1) >> "Programs must be written for people to read, and only incidentally for >> machines to execute."http://www.doesnotunderstand.org/ >> > > > > -- > > > > 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 > -- Serge Stinckwich UMI UMMISCO 209 (SU/IRD/UY1) "Programs must be written for people to read, and only incidentally for machines to execute."http://www.doesnotunderstand.org/

April 13, 2018

Re: [Pharo-dev] Iceberg2.0 crash test passed :)

by Guillermo Polito

Is that Pharo 7? On Fri, Apr 13, 2018 at 9:43 PM, Serge Stinckwich < serge.stinckwich(a)gmail.com> wrote: > Yes I was able to commit also. > Very slick interface :-) > > > But now, every time I want to commit, my image crash and I have: > > Assertion failed: (git_atomic_get(&git__n_inits) > 0), function > git__global_state, file /Users/travis/build/OpenSmalltalk/opensmalltalk- > vm/build.macos64x64/pharo.cog.spur/build/third-party/libgit2-0.25.1/src/global.c, > line 322. > > > On Fri, Apr 13, 2018 at 8:02 PM, Stephane Ducasse <stepharo.self(a)gmail.com > > wrote: > >> I could commit my first commit with iceberg 2.0 >> Well done >> >> It feels nicer :). >> >> Stef >> >> > > > -- > Serge Stinckwich > UMI UMMISCO 209 (SU/IRD/UY1) > "Programs must be written for people to read, and only incidentally for > machines to execute."http://www.doesnotunderstand.org/ > -- 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

April 13, 2018

Re: [Pharo-dev] Iceberg2.0 crash test passed :)

by Serge Stinckwich

Yes I was able to commit also. Very slick interface :-) But now, every time I want to commit, my image crash and I have: Assertion failed: (git_atomic_get(&git__n_inits) > 0), function git__global_state, file /Users/travis/build/OpenSmalltalk/opensmalltalk-vm/build.macos64x64/pharo.cog.spur/build/third-party/libgit2-0.25.1/src/global.c, line 322. On Fri, Apr 13, 2018 at 8:02 PM, Stephane Ducasse <stepharo.self(a)gmail.com> wrote: > I could commit my first commit with iceberg 2.0 > Well done > > It feels nicer :). > > Stef > > -- Serge Stinckwich UMI UMMISCO 209 (SU/IRD/UY1) "Programs must be written for people to read, and only incidentally for machines to execute."http://www.doesnotunderstand.org/

April 13, 2018

Re: [Pharo-dev] Iceberg2.0 crash test passed :)

by Guillermo Polito

I want your bug reports! Le ven. 13 avr. 2018 Ã 21:09, Stephane Ducasse <stepharo.self(a)gmail.com> a Ã©crit : > I could commit my first commit with iceberg 2.0 > Well done > > It feels nicer :). > > Stef > > -- 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

April 13, 2018

Iceberg2.0 crash test passed :)

by Stephane Ducasse

I could commit my first commit with iceberg 2.0 Well done It feels nicer :). Stef

April 13, 2018