October 2013 - Pharo-dev - lists.pharo.org

Re: [Pharo-dev] additional Milestone label for applications

by Camillo Bruni

seems like it works, so by default the pharo-internal projects will have the Area "1. Pharo Image" set. Why the "1."? Well I cannot order the Areas, and the first one is selected by default :P The filter "review" and "integration" now only look at issues that have the proper Area set. On 2013-10-06, at 08:08, StÃ©phane Ducasse <stephane.ducasse(a)inria.fr> wrote: > thanks! > This is a good idea. > > On Oct 6, 2013, at 1:57 AM, Camillo Bruni <camillobruni(a)gmail.com> wrote: > >>>> I think it should just be fine if we define a different workflow (and thus different labels) for external projects... >>>> >>>> >>> The 'Area' field looks unused, having only a single option 'Misc'. Perhaps that could be 'Image' and 'External Project' or similar. Most of an external project's issues would also have an area of 'External 'Project', but sometimes it might be 'Image' if a change there is required to fix an issue reported on a project. Or maybe for external projects you have only a single option 'External Project' for Area. >> >> So we need to have 2 things >> 1. a separate project for each external project that wants to report >> 2. a separate Area for these external projects >> >> For instance, NativeBoost has both a stable version in the image and a development >> version aside. So if a bug appears in the image we can set the Area to "Pharo Image" (the default). >> However, if the bug concerns the development version we set the Area to something project >> specific. >> >> I will try to update the tracker and the filters to reflect that... > >

Oct. 6, 2013

Re: [Pharo-dev] New Critic Rule: Nobody should directly send #methodDict

by StÃ©phane Ducasse

ben this is not that we do not have plan and vision. Just that we have lot of consumers and not enough people thinking that Pharo is theirs :) Stef On Oct 6, 2013, at 2:19 PM, btc(a)openInWorld.com wrote: > StÃ©phane Ducasse wrote: >> Hi >> I'm really happy because >> "New Critic Rule: Nobody should directly send #methodDict" is the way to go. We should document the system design using automatic rules! >> Excellent! >> >> Stef >> >> > So you could also have "Critic Rule: Should not implement system method." Like PharoLauncher got stuck having implemented class-side method #layout [1] > > btw, apart from having a separate critics browser, are the any plans to integrate it further into the system, perhaps as: > * a Critics Panel in Nautilus in the same position as the Comments Panel, or perhaps appended to the Comment. * contextual highlighting > > cheers -ben > > [1] https://pharo.fogbugz.com/default.asp?11783 > > >

Oct. 6, 2013

Re: [Pharo-dev] Starting image gives FileWriteError: File stdout is closed - huh?

by Frank Shearar

On 6 October 2013 11:21, phil(a)highoctane.be <phil(a)highoctane.be> wrote: > Now, when starting my image: > > FileWriteError: File stdout is closed > MultiByteFileStream(StandardFileStream)>>primWrite:from:startingAt:count: > MultiByteFileStream(StandardFileStream)>>next:putAll:startingAt: > MultiByteFileStream>>basicNext:putAll:startingAt: > UTF8TextConverter(TextConverter)>>nextPutByteString:toStream: > UTF8TextConverter(TextConverter)>>nextPutAll:toStream: > MultiByteFileStream>>nextPutAll: > > I was a link while Googling but not resolution. > > What to do? What operating system? Windows? frank > -- > Phil

Oct. 6, 2013

Re: [Pharo-dev] New Critic Rule: Nobody should directly send #methodDict

by btc＠openinworld.com

StÃ©phane Ducasse wrote: > Hi > > I'm really happy because > > "New Critic Rule: Nobody should directly send #methodDict" is the way to go. We should document the system design using > automatic rules! > > Excellent! > > Stef > > So you could also have "Critic Rule: Should not implement system method." Like PharoLauncher got stuck having implemented class-side method #layout [1] btw, apart from having a separate critics browser, are the any plans to integrate it further into the system, perhaps as: * a Critics Panel in Nautilus in the same position as the Comments Panel, or perhaps appended to the Comment. * contextual highlighting cheers -ben [1] https://pharo.fogbugz.com/default.asp?11783

Oct. 6, 2013

Re: [Pharo-dev] [update 3.0] #30455

by Marcus Denker

On Oct 6, 2013, at 12:38 AM, Camillo Bruni <camillobruni(a)gmail.com> wrote: > > On 2013-10-06, at 00:06, Marcus Denker <marcus.denker(a)inria.fr> wrote: > >> 11792 First simplistic file system tree inspector >> https://pharo.fogbugz.com/f/cases/11792 >> >> >> actually a FileReference tree inspector... lots of improvments possible >> inspect >> >> FileSystem workingDirectory >> >> and select the FileReference Tree Inspector > > ha! nice :) actually we could reuse the file list here, no? Yesâ¦ that would be best. Marcus

Oct. 6, 2013

External Semaphores leaking in Socket

by Norbert Hartl

I took some time to analyze my current problem with external semaphores. I was just reluctant to raise the limit in my image because I want the problem solved. I logged the management of external semaphores and discovered that the table fills if a connection times out. The problem turns out to be in Socket>>#connectTo: hostAddress port: port waitForConnectionFor: timeout "Initiate a connection to the given port at the given host address. Waits until the connection is established or time outs." self connectNonBlockingTo: hostAddress port: port. self waitForConnectionFor: timeout ifTimedOut: [ConnectionTimedOut signal: 'Cannot connect to ' , (NetNameResolver stringFromAddress: hostAddress) , ':' , port asString] When a socket is created three external semaphores are registered in the ExternalSemaphoreTable. If a connection times out the exception is thrown but the Socket still has his resources attached. So e.g. in SocketStream class>>#openConnectionToHost: hostIP port: portNumber timeout: timeout | socket | socket := Socket new. socket connectTo: hostIP port: portNumber waitForConnectionFor: timeout. ^self on: socket it holds locally a socket (with semaphores registered) but on exception time the reference to the socket gets lost and the semaphores stay registered. The only way to unregister is on finalization time but I think it should work better. So I would add a destroy before the exception is raised. Socket>>#connectTo: hostAddress port: port waitForConnectionFor: timeout "Initiate a connection to the given port at the given host address. Waits until the connection is established or time outs." self connectNonBlockingTo: hostAddress port: port. self waitForConnectionFor: timeout ifTimedOut: [ self destroy. ConnectionTimedOut signal: 'Cannot connect to ' , (NetNameResolver stringFromAddress: hostAddress) , ':' , port asString] I opened a ticket at https://pharo.fogbugz.com/f/cases/11797 but I'm not sure how I am supposed to provide fixes made against a pharo2.0 image. Probably I should fix this againt 3.0 but then I'm still a 2.0 user :) Norbert

Oct. 6, 2013

Re: [Pharo-dev] What is WeakActionSequence?

by HernÃ¡n Morales Durand

El 04/10/2013 14:27, Camillo Bruni escribiÃ³: > What is that class? No comments, two single methods on Object that refer to it? > A WeakActionSequence is used as container for MessageSend's. It looks like is not even really weak, there is no weakSubclass: definition... You probably don't see users because it was (tipically) only used in OmniBrowser in Pharo <= 1.4. Users of the SASE (a.k.a triggerEvent) mechanism also are (or were) users. The question is, why every object should be able to register events? Think of magnitudes registering events. Object subclasses could include all the observer pattern implementations you like. HernÃ¡n

Oct. 6, 2013

Starting image gives FileWriteError: File stdout is closed - huh?

by phil＠highoctane.be

Now, when starting my image: FileWriteError: File stdout is closed MultiByteFileStream(StandardFileStream)>>primWrite:from:startingAt:count: MultiByteFileStream(StandardFileStream)>>next:putAll:startingAt: MultiByteFileStream>>basicNext:putAll:startingAt: UTF8TextConverter(TextConverter)>>nextPutByteString:toStream: UTF8TextConverter(TextConverter)>>nextPutAll:toStream: MultiByteFileStream>>nextPutAll: I was a link while Googling but not resolution. What to do? -- Phil

Oct. 6, 2013

full version of a Lint Chapter for Entreprise Pharo forthcoming book :)

by StÃ©phane Ducasse

Hi Pharoers since we believe in quality and automation, here is a new chapter on Lint rule support in Pharo :) https://ci.inria.fr/pharo-contribution/job/PharoForTheEnterprise/ws/Lint/Li… Stef

Oct. 6, 2013

New Critic Rule: Nobody should directly send #methodDict

by StÃ©phane Ducasse

Hi I'm really happy because "New Critic Rule: Nobody should directly send #methodDict" is the way to go. We should document the system design using automatic rules! Excellent! Stef

Oct. 6, 2013