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

[Pharo 7] Help needed reviewing Pull requests

by Marcus Denker

Hello, we need more people helping to review and tests Pull Requests. https://github.com/pharo-project/pharo/pulls <https://github.com/pharo-project/pharo/pulls> Especially, it is very important to add a note if you think that something should be merged. Even if it is clear from the context. We need an explicit âthis is good to integrateâ. Marcus

March 20, 2018

Re: [Pharo-dev] Experiment: New Download page based on Pharo Launcher

by Marcus Denker

> On 16 Mar 2018, at 16:41, Esteban Lorenzano <estebanlm(a)gmail.com> wrote: > > but thing is: default download should be the launcher. > because this is the ânewbie-friendlyâ approach. > > All the rest can be in under a link (or a subtitle, doesnât matters). It think that even that is a mistake: To give ânewbiesâ a different download than the âpeople that know better" is *wrong*. Marcus

March 20, 2018

Re: [Pharo-dev] Experiment: New Download page based on Pharo Launcher

by CÃ©drick BÃ©ler

> > > On Sat, Mar 17, 2018 at 1:26 PM, CÃ©drick BÃ©ler <cdrick65(a)gmail.com <mailto:cdrick65@gmail.com>> wrote: > Little feedback too. > > I was surprised by the new download as some students had to install on windowsâ¦ hopefully, I had old win versions but it has disturbed me :) > Still handy and straightforward to just put an image on the VM. > > Butâ¦this being said, I just tried on Mac and this works nicely and this is true that launcher makes the management of image clearer and easier.. > I donât know about windows but I have to care about as 98% of students uses Pharo on windows... > > I was thinking of some adjustments: > 1) maybe a default/advanced mode on the launcher to show only, as default, Mooc/PBE5/Official distributions. > > > did you see that the Launcher already includes this? > There is a Pharo Mooc item. Yes but I was more thinking about hiding most of elements of the tree, especially Jenkins. To me a newcomer only need to see (at first) official distributions + MOOC + PBE5. I canât find the official PBE5 image. It doesnât seem to be the raw Pharo5 ? If not present, is it possible to add it ? I think this is important (if doc cannot be updated) to keep the proper book related images. > > > 2) a proper icon name to display in OSâs task bars. > When several image are opened, they should have different icons, at least different name when hovering, and not only Pharo. > > The PharoLauncher should be named Pharo Launcher :). > Now we should get a nice icon. Yes :) I would go for a circle containing three circles (circle being the pharo logo like the quick pic hack below) :) => image all the way downsâ¦ > > <PastedGraphic-4.png> > > A more general feedback is about cloud drive OSâs (iCloud Drive, Dropbox, â¦). > I have 3 macâs (pro and personal). I put often files in a directory that are synchronized across these devices. > - sometimes I get a message that an image are in different states > - sync image might be long (+changes) depending on network conditions > > I think the question is important as the launcher download in the Document folder (OSX) and this directory is synchronized by default. > What do you think about such common situation ? What are your common practices ? > - lets says on two Macs and works on both > - or I use a dropbox folder > > => cloud drive are cool and my impression is that we could take a better advantage of them for data that could be shared between (personal) images (Play-Cache ? Packages-cache ? â¦ ) > ==> would it be possible to have several files for one image (chunked over time so only the last chunk is modified and then less file would be synchronized) - (I donât know at all if possible) ? > ==> same for changesâ¦ but this one should be easier I think. > > => I also find longer to share pharo system the old ways - zipping (or not) via USB - I think git integration produces a large number of files that relies slows down system copy paste (if only image this is ok)â¦ No idea about that ? I finally resolved to change the default image and VM dirâ¦ Documents is syncing on Macs, and when your connection is slowâ¦ thatâs quite problematics (especially when you do image experiments like trying to graphs more than 2Million objects ^^). I finally got my space low warning (I may retry on a better computer with a 64 bits image) ^^ Side note: ok image grows, but the default is about 200MB in memory for me (Launcher is 115MB here)â¦ when you see safari tab being always more Thant 200MB (and this time I have only one opened tab which is rare)â¦ Safari is not an exception, I find Pharo memory related concern still very reasonable (+ it will improve once the bootstrap is production ready). Cheers, CÃ©drick

March 20, 2018

Re: [Pharo-dev] Released versions in Pharo Bootstrap

by Stephan Eggermont

Cyril Ferlicot D. <cyril.ferlicot(a)gmail.com> wrote: > Le 19/03/2018 Ã 18:49, Stephan Eggermont a Ã©crit : >> Then why bother releasing? >> > > Hi, > > For people using Pharo 7 to not get the same bugs for more than 1 year, > to get the new functionalities and to be able to detect new bugs > introduced in releases. Forgive me for sounding like a broken record. How exactly is this style of releasing going to help with these points? Stephan

March 20, 2018

Re: [Pharo-dev] New Files in Pharo - Migration Guide, How To's and examples

by Stephane Ducasse

Ah if this is that then I agree with you denis :) This is why having good names is important. On Mon, Mar 19, 2018 at 9:21 PM, Denis Kudriashov <dionisiydk(a)gmail.com> wrote: > > > 2018-03-19 20:39 GMT+01:00 Esteban A. Maringolo <emaringolo(a)gmail.com>: >> >> 2018-03-19 16:32 GMT-03:00 Denis Kudriashov <dionisiydk(a)gmail.com>: >> > >> > Hi Guille. >> >> > What you think to add helpfull converting methods? >> >> I'd like them. Anything that saves me from passing a string literal as >> parameter is good. :) >> >> > aStream buffered. >> >> I'd only change this one, to #beBuffered. > > > But #be sounds like it modifies receiver. > But idea is same as #reversed or #sorted for collections. > >> >> >> > aStream withLineEnding: String cr. >> > aStream withPlatformLineEnding. >> >> These two seem ambiguous to me, #with prefix signals something else in my >> head. >> >> >> I'd remove the "with" and use instead: >> >> aStream lineEnding: String cr. >> aStream usePlatformLineEnding. > > > Same here. They will return new instances wrapping receiver. And #with is > like #copyWith: for arrays. > Maybe something like: > > aStream forcedLineEnding: String cr. > aStream forcedPlatformLineEnding. > > >> >> >> Regards! >> >> >> >> Esteban A. Maringolo >> >

March 19, 2018

Re: [Pharo-dev] New Files in Pharo - Migration Guide, How To's and examples

by Esteban A. Maringolo

2018-03-19 17:42 GMT-03:00 Nicolas Cellier <nicolas.cellier.aka.nice(a)gmail.com>: > 2018-03-19 21:21 GMT+01:00 Denis Kudriashov <dionisiydk(a)gmail.com>: >> >> 2018-03-19 20:39 GMT+01:00 Esteban A. Maringolo <emaringolo(a)gmail.com>: >>> >>> 2018-03-19 16:32 GMT-03:00 Denis Kudriashov <dionisiydk(a)gmail.com>: >>> > aStream buffered. >>> I'd only change this one, to #beBuffered. >> But #be sounds like it modifies receiver. >> But idea is same as #reversed or #sorted for collections. > -1 for beBuffered, it's not about adding states to the stream. > As Denis said, it's a transform producing a new stream. I didn't understand it was returning a transformed instance, I actually thought it was the opposite, so if that is the case then Denis' suggested selectors are fine. Thanks for the clarification. Esteban A. Maringolo

March 19, 2018

Re: [Pharo-dev] New Files in Pharo - Migration Guide, How To's and examples

by Nicolas Cellier

2018-03-19 21:21 GMT+01:00 Denis Kudriashov <dionisiydk(a)gmail.com>: > > > 2018-03-19 20:39 GMT+01:00 Esteban A. Maringolo <emaringolo(a)gmail.com>: > >> 2018-03-19 16:32 GMT-03:00 Denis Kudriashov <dionisiydk(a)gmail.com>: >> > >> > Hi Guille. >> >> > What you think to add helpfull converting methods? >> >> I'd like them. Anything that saves me from passing a string literal as >> parameter is good. :) >> >> > aStream buffered. >> >> I'd only change this one, to #beBuffered. >> > > But #be sounds like it modifies receiver. > But idea is same as #reversed or #sorted for collections. > > -1 for beBuffered, it's not about adding states to the stream. As Denis said, it's a transform producing a new stream. > >> > aStream withLineEnding: String cr. >> > aStream withPlatformLineEnding. >> >> These two seem ambiguous to me, #with prefix signals something else in my >> head. > > >> I'd remove the "with" and use instead: >> >> aStream lineEnding: String cr. >> aStream usePlatformLineEnding. >> > > Same here. They will return new instances wrapping receiver. And #with is > like #copyWith: for arrays. > Maybe something like: > > aStream forcedLineEnding: String cr. > aStream forcedPlatformLineEnding. > > > >> >> Regards! >> >> >> >> Esteban A. Maringolo >> >> >

March 19, 2018

Re: [Pharo-dev] New Files in Pharo - Migration Guide, How To's and examples

by Denis Kudriashov

2018-03-19 20:39 GMT+01:00 Esteban A. Maringolo <emaringolo(a)gmail.com>: > 2018-03-19 16:32 GMT-03:00 Denis Kudriashov <dionisiydk(a)gmail.com>: > > > > Hi Guille. > > > What you think to add helpfull converting methods? > > I'd like them. Anything that saves me from passing a string literal as > parameter is good. :) > > > aStream buffered. > > I'd only change this one, to #beBuffered. > But #be sounds like it modifies receiver. But idea is same as #reversed or #sorted for collections. > > > aStream withLineEnding: String cr. > > aStream withPlatformLineEnding. > > These two seem ambiguous to me, #with prefix signals something else in my > head. > I'd remove the "with" and use instead: > > aStream lineEnding: String cr. > aStream usePlatformLineEnding. > Same here. They will return new instances wrapping receiver. And #with is like #copyWith: for arrays. Maybe something like: aStream forcedLineEnding: String cr. aStream forcedPlatformLineEnding. > > Regards! > > > > Esteban A. Maringolo > >

March 19, 2018

Re: [Pharo-dev] New Files in Pharo - Migration Guide, How To's and examples

by Stephane Ducasse

+ one to Esteban gret suggestions! On Mon, Mar 19, 2018 at 8:39 PM, Esteban A. Maringolo <emaringolo(a)gmail.com> wrote: > 2018-03-19 16:32 GMT-03:00 Denis Kudriashov <dionisiydk(a)gmail.com>: >> >> Hi Guille. > >> What you think to add helpfull converting methods? > > I'd like them. Anything that saves me from passing a string literal as > parameter is good. :) > >> aStream buffered. > > I'd only change this one, to #beBuffered. > >> aStream withLineEnding: String cr. >> aStream withPlatformLineEnding. > > These two seem ambiguous to me, #with prefix signals something else in my head. > > I'd remove the "with" and use instead: > > aStream lineEnding: String cr. > aStream usePlatformLineEnding. > > Regards! > > > > Esteban A. Maringolo >

March 19, 2018

Re: [Pharo-dev] New Files in Pharo - Migration Guide, How To's and examples

by Esteban A. Maringolo

2018-03-19 16:32 GMT-03:00 Denis Kudriashov <dionisiydk(a)gmail.com>: > > Hi Guille. > What you think to add helpfull converting methods? I'd like them. Anything that saves me from passing a string literal as parameter is good. :) > aStream buffered. I'd only change this one, to #beBuffered. > aStream withLineEnding: String cr. > aStream withPlatformLineEnding. These two seem ambiguous to me, #with prefix signals something else in my head. I'd remove the "with" and use instead: aStream lineEnding: String cr. aStream usePlatformLineEnding. Regards! Esteban A. Maringolo

March 19, 2018