January 2014 - Pharo-dev - lists.pharo.org

Re: [Pharo-dev] Relative path

by Tudor Girba

Please open an issue and propose the suggestions as slices. Doru On Tue, Jan 14, 2014 at 12:42 PM, Sebastian Sastre < sebastian(a)flowingconcept.com> wrote: > suggestion, would be nice to have > > > AnstractPath>>asPath > > ^ self > > > > > On Jan 13, 2014, at 11:07 PM, Sebastian Sastre < > SEBASTIAN(a)FLOWINGCONCEPT.COM> wrote: > > Hi guys, > > nothing terrible, just some comments on paths and file references > > I found today myself doing in a method: > > ^ (AbsolutePath withAll: anAbsolutePath path segments, > aRelativePathFilename segments) asFileReference exists. > To put a relative path on top of an absolute path because lack of a better > option > > But shouldn't this be doable? > > absolutePath / relativePath / aFilename > > We're half an inch of that syntax sugar > > I saw that / expects only strings, but we can take advantage of file > references, wouldn't make more sense to expect both strings or file > references? > > and then there is this > > '' asFileReference = '/' asFileReference. true > > doesn't feel right > > thoughts? > > > > sebastian <https://about.me/sebastianconcept> > > o/ > > > > > > > -- www.tudorgirba.com "Every thing has its own flow"

Jan. 14, 2014

[pharo-project/pharo-core] a2cac4: 30693

by GitHub

Branch: refs/heads/3.0 Home: https://github.com/pharo-project/pharo-core Commit: a2cac44d8fe3d456455bab051c5179fb18bededb https://github.com/pharo-project/pharo-core/commit/a2cac44d8fe3d456455bab05… Author: Jenkins Build Server <board(a)pharo-project.org> Date: 2014-01-14 (Tue, 14 Jan 2014) Changed paths: A Kernel.package/Class.class/instance/subclass creation/newAnonymousSubclass.st M Polymorph-Widgets.package/CheckboxMorph.class/instance/initialization/initialize.st A ScriptLoader30.package/ScriptLoader.class/instance/pharo - scripts/script346.st A ScriptLoader30.package/ScriptLoader.class/instance/pharo - updates/update30693.st M ScriptLoader30.package/ScriptLoader.class/instance/public/commentForCurrentUpdate.st Log Message: ----------- 30693 7542 NB system example Part1: add Class>>#newAnonymousSubclass https://pharo.fogbugz.com/f/cases/7542 12605 promptForCancel is sometimes unclosable https://pharo.fogbugz.com/f/cases/12605 http://files.pharo.org/image/30/30693.zip

Jan. 14, 2014

[pharo-project/pharo-core]

by GitHub

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

Jan. 14, 2014

Re: [Pharo-dev] Pharo3 and MetacelloPreview

by Johan Brichau

Hi Christophe, That indeed explains why things go so wrong... Thank you! Johan On 13 Jan 2014, at 10:47, Christophe Demarey <christophe.demarey(a)inria.fr> wrote: > Hi Johan, > > You should not do that because Metacello Preview is already loaded in Pharo 3.0. > Loading the preview bootstrap is only needed to bootstrap the load of Metacello Preview. > > Regards, > Christophe. > > Le 13 janv. 2014 Ã 09:22, Johan Brichau a Ã©crit : > >> Sorry, I meant the loading of the following: >> >> ((Smalltalk at: #ConfigurationOfMetacello) >> new >> project >> version: #previewBootstrap) >> load. >> >> On 13 Jan 2014, at 09:18, Johan Brichau <johan(a)inceptive.be> wrote: >> >>> Hi all, >>> >>> When I load the Metacello Preview in Pharo3, I get a very weird loading of the Phantom-Core-dgg.24 package (which yields a warning because of unexisting classes in the image). >>> >>> Nowhere in my image (thus all loaded configurations) I can find the name 'Phantom'. So where is this coming from? >>> What I can make of the execution stack in Metacello is that 'Metacello-ProfStef-dkh.13' strangely resolves to a loading of 'Phantom-Core-dgg.24' >>> >>> Anyone using the Metacello Preview in Pharo3? >>> Anyone experiencing the same? >>> >>> Johan >> >> >

Jan. 14, 2014

Re: [Pharo-dev] Problems with 3.0

by Marcus Denker

On 13 Jan 2014, at 18:12, Norbert Hartl <norbert(a)hartl.name> wrote: > Today I started developing something new with 3.0. I downloaded a fresh 3.0 and worked with that. Inside the debugger I changed some code and accepted. I got back the > > âMethod for block not found on stack , canât edit and continueâ > > dialog with the âbrowseâ button. Pressing that button leads to a DNU in > > DebugContext>>#blockNotFoundDialog:with: > > saying the > > DebugContext>>#codeTextMorph is not understood strangeâ¦ the code is "let's browse the given method with the edited contents" browser := aMethod browse. browser contents: aText; changed: #contents. browser codeTextMorph setSelection: self codeTextMorph selectionInterval; hasUnacceptedEdits: true > > Furthermore do I use old style keybindings and cmd-f in the systembrowser does not open the class find dialog. The context menu (showing f a s shortcut) does. > > Known problems? No, I think we need issue tracker entries for bothâ¦ Marcus

Jan. 14, 2014

Re: [Pharo-dev] Relative path

by Sebastian Sastre

suggestion, would be nice to have AnstractPath>>asPath ^ self On Jan 13, 2014, at 11:07 PM, Sebastian Sastre <SEBASTIAN(a)FLOWINGCONCEPT.COM> wrote: > Hi guys, > > nothing terrible, just some comments on paths and file references > > I found today myself doing in a method: > > ^ (AbsolutePath withAll: anAbsolutePath path segments, aRelativePathFilename segments) asFileReference exists. > > To put a relative path on top of an absolute path because lack of a better option > > But shouldn't this be doable? > > absolutePath / relativePath / aFilename > > We're half an inch of that syntax sugar > > I saw that / expects only strings, but we can take advantage of file references, wouldn't make more sense to expect both strings or file references? > > and then there is this > > '' asFileReference = '/' asFileReference. true > > doesn't feel right > > thoughts? > > > > sebastian > > o/ > > > > >

Jan. 14, 2014

Re: [Pharo-dev] [Pharo3] The last test failing...

by Marcus Denker

maybe we should add an issue tracker entry to not lose this? And no, sadly I have no time to do anythingâ¦ On 11 Jan 2014, at 17:48, btc(a)openinworld.com wrote: > Marcus Denker wrote: >> >>> >>> On 09 Jan 2014, at 17:29, btc(a)openinworld.com wrote: >>> >>>> Marcus Denker wrote: >>>>> >>>>> 12377 ZnHTTPSTests.>>testGetPharoVersion fails on Windows slave >>>>> https://pharo.fogbugz.com/f/cases/12377 >>>>> >>>>> And this is just a build server artefactâ¦ (windows slave firewall problem). >>>>> >>>>> I wonder what to doâ¦ maybe we could just skip it on windows when running on the build slave ;-) >>>>> >>>>> Marcus >>>>> >>>>> >>>> Was the DNS lookup error cannot resolve âci.inria.fr' the only problem? or would access to the server also be blocked? >>>> If only the former, did you consider preloading the DNS resolver cache from hosts file per [1], which provides a number of options: >>>> * refer to ci.inria.fr server via its a private subnet IP address, rather than the public facing IP address >>>> * fake up ci.inria.fr as a second IP address on the slave to which web server can be bound and configured to deliver the require file >>>> Of course the additional effort for these versus the gain needs to be balanced. >>>> >>>> btw, useful commands for testing these are [2]: >>>> * ipconfig /displaydns >>>> * ipconfig /flushdns >>>> >>>> [1] http://technet.microsoft.com/en-us/library/cc780585(v=ws.10).aspx >>>> [2] http://www.microsoft.com/resources/documentation/windows/xp/all/proddocs/en… >>>> >>> >>> No, the thing is: Does it make sense to invest a lot of time into this or are there not more important things? >>> >>> >> >> There is a point where I get endlessly tiredâ¦ so many many many things, sometimes a simple workaround is just the easiest. >> >> Marcus > Very true. Hence my disclaimer "the additional effort for these versus the gain needs to be balanced". > > Actually it turns out my suggestions were over the top, since it seems the root cause is not the firewall but a difference in DNS resolver configuration between Windows and Linux. I had a poke around the pharo-contribution slaves since they are presumably similar to the pharo slaves. Slave pharo-contribution-linux64 was using nameserver 172.21.8.87 with a working 'ping ci.inria.fr', while slave pharo-contribution-win7 was using nameserver 172.21.32.8 (obtained from DHCP) with a non-working 'ping ci.inria.fr'. Forcing win7 DNS manually to use 172.21.8.87 allowed 'ping ci.inria.fr' to work and further the URL [1] worked fine from the Chrome browser returning [2]. > > Now it seems that bumping the network config has fixed this, since after undoing the manual win7 DNS setting, the DNS server (obtained from DHCP) is now 172.21.8.87. Someone with access to the Pharo Windows slaves might like to try the same. > > cheers -ben > > [1] https://ci.inria.fr/pharo/job/Pharo-2.0/lastSuccessfulBuild/api/xml?xpath=/… > [2] <description>2.0 #20628</description> > > _Appendix - Detailed Transcript_ > > On slave pharo-contribution-linux64... > # ping ci.inria.fr > PING ci.inria.fr (193.51.193.223) 56(84) bytes of data. > 64 bytes from ci.inria.fr (193.51.193.223): icmp_req=1 ttl=62 time=0.569 ms > > #cat /etc/resolv.conf > nameserver 172.21.8.87 > nameserver 193.51.196.130 > nameserver 193.51.196.131 > search cs1cloud.internal > > > On slave pharo-contribution-win7... > C:\Users\ci> ping ci.inria.fr > Ping request could not find host ci.inria.fr. Please check the name and try again. > > C:\Users\ci> ipconfig /all > Ethernet adapter Local Area Connection: > Description . . . . . . . . . . . : Intel(R) PRO/1000 MT Network Connection > DHCP Server . . . . . . . . . . . : 172.21.8.87 > DNS Servers . . . . . . . . . . . : 172.21.38.8 193.51.196.130 193.51.196.131 > > Through the GUI manually forced pharo-contribution-win7 to use nameserver 172.21.8.87, then... > C:\Users\ci> ping ci.inria.fr > Pinging ci.inria.fr [193.51.193.223] with 32 bytes of data: > Reply from 193.51.193.223: bytes=32 time<1ms TTL=62 > > And after undoing that manual force... > C:\Users\ci> ipconfig /all > Ethernet adapter Local Area Connection: > Description . . . . . . . . . . . : Intel(R) PRO/1000 MT Network Connection > DHCP Server . . . . . . . . . . . : 172.21.8.87 > DNS Servers . . . . . . . . . . . : 172.21.8.87 193.51.196.130 193.51.196.131 > > C:\Users\ci> ping ci.inria.fr > Pinging ci.inria.fr [193.51.193.223] with 32 bytes of data: > Reply from 193.51.193.223: bytes=32 time<1ms TTL=62

Jan. 14, 2014

Re: [Pharo-dev] [squeak-dev] The Trunk: Kernel-nice.831.mcz

by Marcus Denker

I copy and pasted this mail to the bug tracker (took 1 second): https://pharo.fogbugz.com/f/cases/12642/bug-in-NumberParser-when-reading-a-… On 14 Jan 2014, at 03:48, Nicolas Cellier <nicolas.cellier.aka.nice(a)gmail.com> wrote: > No time to open a bug, but this should better be fixed before 3.0 release > > ---------- Forwarded message ---------- > From: <commits(a)source.squeak.org> > Date: 2014/1/14 > Subject: [squeak-dev] The Trunk: Kernel-nice.831.mcz > To: squeak-dev(a)lists.squeakfoundation.org, packages(a)lists.squeakfoundation.org > > > Nicolas Cellier uploaded a new version of Kernel to project The Trunk: > http://source.squeak.org/trunk/Kernel-nice.831.mcz > > ==================== Summary ==================== > > Name: Kernel-nice.831 > Author: nice > Time: 14 January 2014, 3:43:29.851 am > UUID: 5c62dca4-bf8e-4330-9cb1-9cd24047aac6 > Ancestors: Kernel-nice.830 > > 1) Fix awfull bug in NumberParser when reading a number with fraction part > (the trailing digits of the fraction might be skipped when there's many of them because of incorrect update of lastNonZero position) > > This made this test fail: > self assert: (Float readFrom: (((1<<1075) reciprocal + (1<<1400) reciprocal) printShowingDecimalPlaces: 2000)) = Float fmin. > > I just wanted to check NumberParser w.r.t. a former bug of dtoa, but it failed for different reasons http://www.exploringbinary.com/gays-strtod-returns-zero-for-inputs-just-abo… > > 2) Classify some as yet unclassified (or miss-classified) > > =============== Diff against Kernel-nice.830 =============== > > Item was changed: > + ----- Method: IllegalResumeAttempt>>defaultAction (in category 'handling') ----- > - ----- Method: IllegalResumeAttempt>>defaultAction (in category 'comment') ----- > defaultAction > "No one has handled this error, but now give them a chance to decide how to debug it. If none handle this either then open debugger (see UnhandedError-defaultAction)" > > UnhandledError signalForException: self! > > Item was changed: > + ----- Method: IllegalResumeAttempt>>isResumable (in category 'handling') ----- > - ----- Method: IllegalResumeAttempt>>isResumable (in category 'comment') ----- > isResumable > > ^ false! > > Item was changed: > + ----- Method: InMidstOfFileinNotification>>defaultAction (in category 'handling') ----- > - ----- Method: InMidstOfFileinNotification>>defaultAction (in category 'as yet unclassified') ----- > defaultAction > > self resume: false! > > Item was changed: > ----- Method: NumberParser>>nextLargeIntegerBase:nPackets: (in category 'parsing-large int') ----- > nextLargeIntegerBase: aRadix nPackets: nPackets > "Form a Large integer with incoming digits from sourceStream. > Return this integer, or zero if no digits found. > Stop reading when no more digits or when nPackets elementary LargeInteger have been encountered. > Count the number of digits and the lastNonZero digit and store them in instVar" > > + | high nDigitsHigh lastNonZeroHigh low nDigitsLow halfPackets | > - | high nDigitsHigh low nDigitsLow halfPackets | > halfPackets := nPackets bitShift: -1. > halfPackets = 0 ifTrue: [^self nextElementaryLargeIntegerBase: aRadix]. > high := self nextLargeIntegerBase: aRadix nPackets: halfPackets. > high isLarge ifFalse: [^high]. > nDigitsHigh := nDigits. > + lastNonZeroHigh := lastNonZero. > low := self nextLargeIntegerBase: aRadix nPackets: halfPackets. > nDigitsLow := nDigits. > nDigits := nDigitsHigh + nDigitsLow. > + lastNonZero := lastNonZero = 0 > + ifTrue: [lastNonZeroHigh] > + ifFalse: [lastNonZero + nDigitsHigh]. > - lastNonZero = 0 ifFalse: [lastNonZero := lastNonZero + nDigitsHigh]. > ^high * (aRadix raisedToInteger: nDigitsLow) + low! > > >

Jan. 14, 2014

Re: [Pharo-dev] Cyclic MC dependencies in Pharo packages

by Marcus Denker

Yes, we need to seriously tackle thisâ¦ without safe-guards (tests, for example), this will never work. Even tools would be nice to show dependencies... On 13 Jan 2014, at 15:17, Pavel Krivanek <pavel.krivanek(a)gmail.com> wrote: > Hi, > > if we remove Monticello package X, we should firstly remove packages that extend the package X. I tried to play with a code that could give me such order. Here it is: > > all := (MCWorkingCopy allManagers collect: #packageName). > > deps := Dictionary new. > all do: [:packageName | > | package | > package := (RPackageSet named: packageName). > deps at: packageName put: (package extensionMethods groupedBy: [ :m | (Smalltalk classOrTraitNamed: m className) package packageName ]) keys ]. > > depsList := deps associations asOrderedCollection. > resolveBlock := [:a | a value isEmpty ]. > hasResolved := depsList select: resolveBlock. > notResolved := depsList reject: resolveBlock. > > ordered := hasResolved collect: [:a | a key ]. > > [ foundDeps := notResolved select: [ :a | ordered includesAll: a value ]. > notResolved := notResolved copyWithoutAll: foundDeps. > ordered addAll: (foundDeps collect: [:a | a key]). > foundDeps isEmpty ] whileFalse. > > ordered. > > > However, a lot of packages (156) cannot be removed because of cyclic dependencies. You can see them together with count of references from other packages with this code: > > notResolvedNames := (notResolved collect: [:a | a key ]). > ((notResolved collect: [:a | a key -> > (notResolved select: [:b | b value includes: a key]) size]) asSortedCollection:[:a :b | a value <= b value]) > > The result is that we need to remove cyclic dependencies for this packages: > 'Kernel' 'Morphic-Base' 'Polymorph-Widgets' 'Tools' > Then the removing order can be computed. To check that try to modify previous code: > > resolveBlock := [:a | a value isEmpty or:[#('Kernel' 'Morphic-Base' 'Polymorph-Widgets' 'Tools') includes: a key] ]. > > I think we should add it as a release test too. > > Cheers, > -- Pavel > > > >

Jan. 14, 2014

Re: [Pharo-dev] new compiler and semantic analyser warnings

by Marcus Denker

(I will answer all the compiler related question later this weekâ¦ I am a bit snowed under and need to finish some thing beforeâ¦ in general: the whole compiler API needs work, but we need to get rid of the old compiler before, else it is very hard to doâ¦ so this is for Pharo4) On 14 Jan 2014, at 00:46, Nicolai Hess <nicolaihess(a)web.de> wrote: > Am I right that with the new compiler, the way > how analyser warnings work changed too? > see the description in > case 12512 SemanticAnalyzer ignores answer from UnknownSelectorWarning > and > case 12514 > OCSemanticWarnings ignore "cancel" from a confirmation dialog > > I tried to fix this but I can not figure out how the answers from an > OCSemanticWarning dialog supposed to interact with the analyser/parser/compiler. > > Especially "Cancel" should cancel the current evaluation. > I tried this with evaluationg > compilationContext failBlock value > That works but I don't know if this is the way it is supposed to be. > > For those answers that would change the code > (OCUnusedVariableWarning) is another problem, that the > compiler does not handle ReparseAfterSourceEditing. > (see case 12442 Nautilus brakes code when removing more than one unused local variables) > > Nicolai

Jan. 14, 2014