- Pharo-dev - lists.pharo.org

Re: [Pharo-dev] Rules

by Yuriy Tymchuk

> On 24 Nov 2015, at 14:10, Ben Coman <btc(a)openinworld.com> wrote: > > On Tue, Nov 24, 2015 at 4:32 PM, Nicolas Anquetil > <nicolas.anquetil(a)inria.fr <mailto:nicolas.anquetil@inria.fr>> wrote: >> >> >> On 23/11/2015 22:14, stepharo wrote: >>> >>> >>> >>> Le 22/11/15 22:11, Yuriy Tymchuk a Ã©crit : >>>> >>>> Hi. >>>> >>>> The rule is correct. Sending a different message to super may be >>>> confusing. In this case you are in âprintOn:â method and you are sending >>>> âprintOn:nextPutAll:â to super. The idea is that you have to send >>>> âprintOn:nextPutAll:â to self and then either it will invoke the super one, >>>> you there will be an overridden method in your current class. >>> >>> >>> exactly but do you expect a newbie to understand anything with the current >>> explanation? >>> I do not. >>> >>> " >>> >>> Rewrite super messages to self messages when both refer to same method" >> >> What about : > > Sorry :) both of these make my head hurt. I think Stef's was clearer. > The useful part there is that it reports the actual identifier causing > the problem, which makes it more obvious this is not what was meant, > from which even a newbie may infer the missing period. > >> "'super' should call the same selector. Send different selector to 'self' >> then use 'super' in this one." >> ? > > btw, is this just commonly how super is used, or is it really bad to > call a different super method? > I'm sure I've done that on occasion. It makes harder to extend your code. Because if method `a` calls method `b`, then even if method `b` is not defined in your class, I can subclass and redefine method b. Otherwise I have to subclass and redefine `a` although `a` is fine, but I want to run my `b` and not the one of superclass. Uko > > cheers -ben > >> >> or: >> >> "should not send a different message to super (send different message to >> self and then call super from it)" >> ? >> >> >> >> nicolas >> >>> >>> Sucks >>> >>>> >>>> Secondly, there is an issue with âsends unkown selectorâ. Iâve also >>>> noticed it recently. I think that it happens because the rule has a special >>>> cace, and I messed up something with refresh. This is the next thing on my >>>> TODO list. >>>> >>>> Cheers. >>>> Uko >>>> >>>> >>>>> On 22 Nov 2015, at 16:33, Marcus Denker <marcus.denker(a)inria.fr> wrote: >>>>> >>>>> >>>>>> On 22 Nov 2015, at 12:20, stepharo <stepharo(a)free.fr> wrote: >>>>>> >>>>>> With the following (broken) definition >>>>>> >>>>>> >>>>>> : aStream >>>>>> super printOn: aStream >>>>>> aStream nextPutAll: '(', faces printString , ')' >>>>>> >>>>>> I got this strange rule: >>>>>> >>>>>> Rewrite super messages to self messages when both refer to same >>>>>> method >>>>>> >>>>>> I do not get it and it looks really suspicious to me. >>>>>> Any suggestions? >>>>> >>>>> The rule is meant to catch the case when when âsuperâ can be âselfâ as >>>>> there is no #printOn: >>>>> defined in the class. >>>>> >>>>> But here it looks like it is wrong: it would be an infinite loop. >>>>> >>>>> I think it could be related to this other bug: I have seen that the >>>>> rules are not correctly updated >>>>> when you add methods. (I wanted to add an issue for that but did not >>>>> yet). E.g. I often get >>>>> a âsends unkown selectorâ for self sends and then I check â> method is >>>>> there, I just added it! >>>>> >>>>> So here this means that you added a new printOn:, the rules do not >>>>> update, and it thinks >>>>> that there is no printOn:. >>>>> >>>>> So the real bug is that added methods need to be taken into account by >>>>> QA. >>>>> >>>>> Marcus >>>> >>>> >>>> >>> >>> >> >> -- >> Nicolas Anquetil >> RMod team -- Inria Lille

Nov. 24, 2015

Seaside REST on Pharo 50

by Franck Warlouzet

Hi, We are currently using seaside, magritte, reef etc to do a project at school. We also have to do a mobile app (written in Java :(), so we are doing a REST service to access the database from the mobile app. We are working on Pharo 50, and we noticed that Seaside-REST is not working anymore. For simple get, it is ok, for example : test <get> ^ 'foo' but if you want to have a complete path like that : test: aValue <get> <path: '/foo?value={aValue}'> ^ 'foo' You get a DNU RBVariableNode>>key during the route creating (Same on pharo 40 is working). Any idea why ? In the meantime, we are using Teapot, it seems to be working on Pharo 50. Btw, I cannot subscribe to seaside mailing list (I never get the confirmation mail), so if someone could forward this email on seaside mailing list, it would be nice. Franck

Nov. 24, 2015

Re: [Pharo-dev] Improving API of Date, Month....

by Esteban A. Maringolo

2015-11-24 12:14 GMT-03:00 Skip Lentz <skip.lentz(a)inria.fr>: > Hi, > > I also had an idea for a method which returns the most significant unit of a Duration. > > For example, you have a duration of 432 days, 3 hours, 21 minutes, 5 seconds, etc., and > it would return you "432 days" (or just â1 yearâ). I encountered this when wanting to create > a time indication on e.g. a commit or a comment. Itâs much more readable and user-friendly > than a timestamp. > > Would this be a nice addition to the Duration API? It would be nice to have. But why not something like a DurationFormatter? that in turn collaborates with the Locale to ge the words for "Year, Month, Week" in the current language. Regards!

Nov. 24, 2015

Re: [Pharo-dev] Improving API of Date, Month....

by Esteban A. Maringolo

2015-11-24 11:59 GMT-03:00 stepharo <stepharo(a)free.fr>: > Hi > > I was reading the code of Date and I was expecting to get some methods such > as > monthNames > dayNames > daysInMonths > > Ok they are stored in ChronologyConstants but they are not accessible. > There is no monthNames nor > > Do I imply something strange or silly? > I think that it would make sense to add them. It makes sense, maybe the class Date is not the best for those methods, but there isn't a "Calendar" like class either. Whatever is chosen, it should collaborate with the Locale, but be able to use any other locale to retrieve the names. Esteban A. Maringolo

Nov. 24, 2015

Re: [Pharo-dev] Improving API of Date, Month....

by Skip Lentz

Hi, I also had an idea for a method which returns the most significant unit of a Duration. For example, you have a duration of 432 days, 3 hours, 21 minutes, 5 seconds, etc., and it would return you "432 days" (or just â1 yearâ). I encountered this when wanting to create a time indication on e.g. a commit or a comment. Itâs much more readable and user-friendly than a timestamp. Would this be a nice addition to the Duration API? > On Nov 24, 2015, at 3:59 PM, stepharo <stepharo(a)free.fr> wrote: > > Hi > > I was reading the code of Date and I was expecting to get some methods such as > monthNames > dayNames > daysInMonths > > Ok they are stored in ChronologyConstants but they are not accessible. > There is no monthNames nor > > Do I imply something strange or silly? > I think that it would make sense to add them. > > Stef >

Nov. 24, 2015

Re: [Pharo-dev] Improving API of Date, Month....

by stepharo

https://pharo.fogbugz.com/f/cases/17098/Improve-Date-class-API Le 24/11/15 15:59, stepharo a Ã©crit : > Hi > > I was reading the code of Date and I was expecting to get some methods > such as > monthNames > dayNames > daysInMonths > > Ok they are stored in ChronologyConstants but they are not accessible. > There is no monthNames nor > > Do I imply something strange or silly? > I think that it would make sense to add them. > > Stef > >

Nov. 24, 2015

Improving API of Date, Month....

by stepharo

Hi I was reading the code of Date and I was expecting to get some methods such as monthNames dayNames daysInMonths Ok they are stored in ChronologyConstants but they are not accessible. There is no monthNames nor Do I imply something strange or silly? I think that it would make sense to add them. Stef

Nov. 24, 2015

Re: [Pharo-dev] problem using curl and wget in zero-conf

by stepharo

I'm bad with scripts :) but this is important to improve them. Can you open a bug entry so that this discussion is not lost? Le 24/11/15 00:25, Dale Henrichs a Ã©crit : > I've been using zero-conf scripts to download the pharo vm for a while > and occasionally the download fails to install the vm in the pharo-vm > directory .... silently ..... with no clue as to what happened. even > worse the exit status is zero so the failure isn't noticed until much > later on in my install when an attempt is made to actually use the vm > .... I do know that when I see these failures the pharo-vm does get > created but is empty ... > > I got another failure today and figured that I would dig into this on > my end to see if I could understand what might be going on .... > > The section of the zero-conf script that is suspect is the following: > > mkdir $VM_DIR > $DOWNLOAD_TO$VM_DIR/vm.zip $VM_URL > > > After the $VM_DIR is created (which I know happens), the $DOWNLOAD_TO > script is run. Recently, I learned that a side effect of using the > --silent option for curl is that error messages are also suppressed > and I see that for curl, the `--silent` option is being used: > > if [[ `which curl 2> /dev/null` ]]; then > DOWNLOAD="curl --silent --location --compressed "; > DOWNLOAD_TO="$DOWNLOAD --output "; > elif [[ `which wget 2> /dev/null` ]]; then > DOWNLOAD_TO="wget --quiet --output-document="; > DOWNLOAD="$DOWNLOAD_TO-"; > else > echo "Please install curl or wget on your machine"; > exit 1 > fi > > > For curl if you want to turn off the progress meter (a good thing) and > continue to report errors, you need to use --show-error in conjunction > with --silent (as described in the man page) > > I think the same thing applies to wget, --quiet turns off error > messages and progress .... you probably should be using --no-verbose > to continue to get error messages ... > > Dale > > >

Nov. 24, 2015

Install Pharo launcher on MAC

by Trussardi Dario Romano

Ciao, i have a macbook with Mac OS X 10.7.5 and some Pharo x.y applications. Now i begin a new development work and i think to started with the Pharo launcher to manage the last and the future pharo image. Into https://ci.inria.fr/pharo/view/Launcher/job/Launcher-Mac/ i found this link: latest.dmg It's the right dmg image to install ? I need to do something for the old pharo applications? After installed the new pharo launcher, the old Pharo application is load and works without problem? ( how they work now: open with specific VM ) In the Mac App Store i found this : https://itunes.apple.com/it/app/pharo-launcher/id503025789?mt=12 It has the same name: 'pharo-launcher' but it's another think or i wrong? It can interact with the above? Thanks for any considerations. Dario

Nov. 24, 2015

[pharo-project/pharo-core]

by GitHub

Branch: refs/tags/50464 Home: https://github.com/pharo-project/pharo-core

Nov. 24, 2015