- Pharo-dev - lists.pharo.org

Slice dependent package log message
by Ben Coman Jan. 28, 2017

Jan. 28, 2017

This is a simple by slightly invasive idea I'd like to run past the community. Sometimes when I look at packages in Pharo/main and PharoInbox/main repositories, I wonder which Issues particular package versions belongs to. So I'd like to add an automatic reference to their log message back to the Slice. I think the following could be a nice way to do it... MCWorkingCopy>>newVersionWithName: nameString message: messageString in: aRepository | info deps depsMsgString | info := ancestry infoWithName: nameString message: messageString. ancestry := MCWorkingAncestry new addAncestor: info. self modified: true; modified: false. "*" depsMsgString := messageString, String cr, String cr, 'Required by ', nameString. "*" deps := self collectDependenciesWithMessage: depsMsgString in: aRepository. (self repositoryGroup includes: aRepository) ifFalse: [ self repositoryGroup addRepository: aRepository ]. ^ MCVersion package: package info: info snapshot: package snapshot dependencies: deps Your thoughts? Changed lines marked "*". cheers -ben

1 0

MetacelloLoaderPolicy
by Ben Coman Jan. 28, 2017

Jan. 28, 2017

Hi Dale, Just tracing through a Metacello load and I see MetacelloLoaderPolicy. This has no comment and its superclass is Object so its not clear what other loader policies there might be. Is there just that one? cheers -ben

1 0

MetacelloBaseConfiguration deprecation
by Ben Coman Jan. 28, 2017

Jan. 28, 2017

In ConfigurationOfDependencyAnalyser class >> DevelopmentSupport I read... "See the methods in the 'development support' category on the class-side of MetacelloBaseConfiguration. Decide what development support methods you would like to use and copy them the the class-side of your configuration." <apiDocumentation> but MetacelloBaseConfiguration is empty and in package Metacello-Core-Deprecated. So what is the replacement? cheers -ben

1 0

Debugger layout again
by Yuriy Tymchuk Jan. 27, 2017

Jan. 27, 2017

Hi again, there is still no way to change the debugger layout to have it the same way as Mooseâ¦ Why did we change layout in Pharo in the first place? It was so nice: you had the stack with the whole highlighting on the side, so you could see more. Why do we have to make things unusable in Pharo? Uko P.S. ok, ok, I guess I will just move to Moose.

3 4

[bloc] addressing the moz2d issue
by Tudor Girba Jan. 27, 2017

Jan. 27, 2017

Hi, Thank you for the intensive set of issues you raised during the Bloc presentation. I think it is worthwhile addressing them more thoroughly, so let me start with the issue that seemed to have caused the most worries: Sparta & Moz2D. Please keep in mind that while I am involved to some extent in Bloc, the real credits for the current state go to Glenn and Alex. Moz2D (https://github.com/mozilla/moz2d, https://wiki.mozilla.org/Platform/GFX/Moz2D) offers an advanced backend and using it puts us on par with the rendering speed of a web browser, which is a significant added value over what we have now. However, as it was noted, it does come with a cost due to the fact that it is not available as standalone with only the features we are interested in. The vector graphics part is actually buildable out of the box. However, the text support needs to be extracted out of Moz2D, and this is where the patching scripts are used. The patches are there only for compilation purposes and not for features and they are applied automatically. You can see it here: https://github.com/syrel/Moz2D Alex updated recently the Moz2D version and it worked without problems. Of course, future changes in Moz2D might imply changes in this script as well, and this implies that we will need to maintain that script. And we could imagine applying these patches on the trunk of Moz2D to see if they work, and we can also imagine engaging with the Moz2D owners to see if we can find a middle ground. Now, letâs put this into perspective. We are currently using Athens and the Cairo backend. While Cairo is provided as a standalone library it has not seen significant advances since Mozzila shifted its focus towards Moz2D. So, sticking with it might not be an ideal strategy either. Furthermore, just like Athens, Sparta is an abstraction that allows us to switch the underlying backend should we need to. Until now we did not find a cross-platform backend that is as advanced and complete as Moz2D, but there is no reason to think that none other will appear in the future. Skia is an alternative but it is only a vector graphic engine without text support, so using it would imply to have another library for the text support. Sparta also comes with a reasonable set of tests that is aimed at testing the basic Moz2D functionality to make sure that the assumptions on top of which Sparta is built are correct. All in all, I think that the current situation is not ideal, but there is already enough engineering in place to actually make it work. And I definitely think that the potential it opens is rather significant. And, if more people look at the scripts, we might find even better and cheaper ways to express it. Cheers, Doru -- www.tudorgirba.com www.feenk.com "We cannot reach the flow of things unless we let go."

10 23

memoized vs once
by Eliot Miranda Jan. 27, 2017

Jan. 27, 2017

Hi Ben, via FaceBook via twitter I hear you've coined BlockClosure>>#memoized. Allow me to beg you to rename it to BlockClosure>>#once. There's a preexisting implementation of this in VisualWorks by Travis Griggs called once. I hope you agree that it's good to eliminate gratuitous incompatibilities between dialects and that "once" is an elegant name. _,,,^..^,,,_ best, Eliot

14 42

Immutability support
by Norbert Hartl Jan. 27, 2017

Jan. 27, 2017

Does anyone know the state of immutability support in vm and image? The latest vm downloadable is compiled with IMMUTABILITY=1 (Esteban said that). When I open a pharo6 image with this VM and do: ASUser new setIsReadOnlyObject: true; name: 'foo' with ASUser>>#name: arg1 name := arg1 I don't get an exception. Is there something missing or am I not understanding? Norbert

9 47

HandMorph>>#processEvents
by phil＠highoctane.be Jan. 27, 2017

Jan. 27, 2017

Why do we have this twice in processEvents? self mouseOverHandler processMouseOver: lastMouseEvent ? Well, maybe that is not there that things happen but if you spy on mouseEnter and mouseLeave events, you will notice that they are both sent **twice** all the time. I think that this happens so often that we would benefit from getting rid of that twice enter/leave thing (and it is really annoying to have to deal with these two events when you expect only one. I am not "entering entering" a rectangle). processEvents "Process user input events from the local input devices." | evt evtBuf type hadAny | ActiveEvent ifNotNil: ["Meaning that we were invoked from within an event response. Make sure z-order is up to date" self mouseOverHandler processMouseOver: lastMouseEvent]. hadAny := false. [(evtBuf := Sensor nextEvent) isNil] whileFalse: [evt := nil. "for unknown event types" type := evtBuf first. type = EventTypeMouse ifTrue: [recentModifiers := evtBuf sixth. evt := self generateMouseEvent: evtBuf]. type = EventTypeKeyboard ifTrue: [recentModifiers := evtBuf fifth. evt := self generateKeyboardEvent: evtBuf]. type = EventTypeDragDropFiles ifTrue: [evt := self generateDropFilesEvent: evtBuf]. type = EventTypeWindow ifTrue:[evt := self generateWindowEvent: evtBuf]. "All other events are ignored" (type ~= EventTypeDragDropFiles and: [evt isNil]) ifTrue: [^self]. evt isNil ifFalse: ["Finally, handle it" self handleEvent: evt. hadAny := true. "For better user feedback, return immediately after a mouse event has been processed." (evt isMouse and: [evt isMouseWheel not]) ifTrue: [^self]]]. "note: if we come here we didn't have any mouse events" mouseClickState notNil ifTrue: ["No mouse events during this cycle. Make sure click states time out accordingly" mouseClickState handleEvent: lastMouseEvent asMouseMove from: self]. hadAny ifFalse: ["No pending events. Make sure z-order is up to date" self mouseOverHandler processMouseOver: lastMouseEvent] Clues? Phil

3 2

[pharo-project/pharo-core] d8e815: 60362
by GitHub Jan. 27, 2017

Jan. 27, 2017

Branch: refs/heads/6.0 Home: https://github.com/pharo-project/pharo-core Commit: d8e8156550b7a55bd615c4aba76c1d8241afb073 https://github.com/pharo-project/pharo-core/commit/d8e8156550b7a55bd615c4ab… Author: Jenkins Build Server <board(a)pharo-project.org> Date: 2017-01-27 (Fri, 27 Jan 2017) Changed paths: M FileSystem-Core.package/FileReference.class/instance/accessing/creationTime.st M FileSystem-Core.package/FileReference.class/instance/accessing/modificationTime.st M FileSystem-Core.package/FileReference.class/instance/accessing/size.st A FileSystem-Core.package/FileSystem.class/instance/public/creationTimeOf_.st M FileSystem-Core.package/FileSystem.class/instance/public/creationTime_.st A FileSystem-Core.package/FileSystem.class/instance/public/modificationTimeOf_.st M FileSystem-Core.package/FileSystem.class/instance/public/modificationTime_.st A FileSystem-Core.package/FileSystem.class/instance/public/sizeOf_.st M FileSystem-Core.package/FileSystem.class/instance/public/size_.st A FileSystem-Core.package/FileSystemStore.class/instance/abstract/basicCreationTimeOf_.st M FileSystem-Core.package/FileSystemStore.class/instance/abstract/basicCreationTime_.st A FileSystem-Core.package/FileSystemStore.class/instance/abstract/basicSizeOf_.st M FileSystem-Core.package/FileSystemStore.class/instance/abstract/basicSize_.st M FileSystem-Core.package/FileSystemStore.class/instance/private/entryFromNode_fileSystem_path_.st A FileSystem-Core.package/FileSystemStore.class/instance/public/creationTimeOf_.st M FileSystem-Core.package/FileSystemStore.class/instance/public/creationTime_.st A FileSystem-Core.package/FileSystemStore.class/instance/public/modificationTimeOf_.st M FileSystem-Core.package/FileSystemStore.class/instance/public/modificationTime_.st A FileSystem-Core.package/FileSystemStore.class/instance/public/sizeOf_.st M FileSystem-Core.package/FileSystemStore.class/instance/public/size_.st A FileSystem-Disk.package/DiskStore.class/instance/private/basicCreationTimeOf_.st M FileSystem-Disk.package/DiskStore.class/instance/private/basicCreationTime_.st A FileSystem-Disk.package/DiskStore.class/instance/private/basicModificationTimeOf_.st M FileSystem-Disk.package/DiskStore.class/instance/private/basicModificationTime_.st A FileSystem-Disk.package/DiskStore.class/instance/private/basicSizeOf_.st M FileSystem-Disk.package/DiskStore.class/instance/private/basicSize_.st M FileSystem-Disk.package/FileHandle.class/instance/public/size.st A FileSystem-Disk.package/FilePluginPrims.class/instance/file primitives/sizeOf_.st M FileSystem-Disk.package/FilePluginPrims.class/instance/file primitives/size_.st A FileSystem-Memory.package/MemoryStore.class/instance/private/basicCreationTimeOf_.st M FileSystem-Memory.package/MemoryStore.class/instance/private/basicCreationTime_.st A FileSystem-Memory.package/MemoryStore.class/instance/private/basicModificationTimeOf_.st M FileSystem-Memory.package/MemoryStore.class/instance/private/basicModificationTime_.st A FileSystem-Memory.package/MemoryStore.class/instance/private/basicSizeOf_.st M FileSystem-Memory.package/MemoryStore.class/instance/private/basicSize_.st A GeneralRules.package/GRUndeclaredVariableRule.class/README.md A GeneralRules.package/GRUndeclaredVariableRule.class/definition.st A GeneralRules.package/GRUndeclaredVariableRule.class/instance/accessing/group.st A GeneralRules.package/GRUndeclaredVariableRule.class/instance/accessing/name.st A GeneralRules.package/GRUndeclaredVariableRule.class/instance/accessing/severity.st A GeneralRules.package/GRUndeclaredVariableRule.class/instance/helpers/critiqueFor_.st A GeneralRules.package/GRUndeclaredVariableRule.class/instance/running/check_forCritiquesDo_.st A Graphics-Primitives.package/Bitmap.class/class/class initialization/initialize.st M Kernel-Tests.package/MethodContextTest.class/README.md A Kernel-Tests.package/MethodContextTest.class/instance/private/expectedFailures.st A Kernel-Tests.package/MethodContextTest.class/instance/tests/testJump.st A Kernel-Tests.package/MethodContextTest.class/instance/tests/verifyJumpWithSelector_.st A Kernel-Tests.package/SimulationMock.class/README.md A Kernel-Tests.package/SimulationMock.class/definition.st A Kernel-Tests.package/SimulationMock.class/instance/example/exampleClosure.st A Kernel-Tests.package/SimulationMock.class/instance/example/exampleSend.st A Kernel-Tests.package/SimulationMock.class/instance/example/exampleStore.st M Kernel.package/Context.class/instance/controlling/jump.st A Kernel.package/Context.class/instance/system simulation/stepUntilSomethingOnStack.st A Kernel.package/NoModificationError.class/README.md A Kernel.package/NoModificationError.class/definition.st A Kernel.package/NoModificationError.class/instance/accessing/index_.st A Kernel.package/NoModificationError.class/instance/accessing/object_.st A Kernel.package/NoModificationError.class/instance/accessing/selector_.st A Kernel.package/NoModificationError.class/instance/accessing/value_.st A Kernel.package/NoModificationError.class/instance/printing/messageText.st A Kernel.package/NoModificationError.class/instance/printing/printObject_on_.st A Kernel.package/NoModificationError.class/instance/printing/standardMessageText.st A Kernel.package/NoModificationError.class/instance/retrying/retryModification.st A Kernel.package/NoModificationError.class/instance/testing/isResumable.st M Kernel.package/Object.class/instance/write barrier/attemptToAssign_withIndex_.st A Refactoring-Critics.package/RBUndeclaredReferenceRule.class/class/instance creation/new.st R Refactoring-Critics.package/RBUndeclaredReferenceRule.class/class/testing/checksMethod.st A Refactoring-Critics.package/RBUndeclaredReferenceRule.class/class/testing/isVisible.st R Refactoring-Critics.package/RBUndeclaredReferenceRule.class/instance/accessing/group.st R Refactoring-Critics.package/RBUndeclaredReferenceRule.class/instance/accessing/name.st R Refactoring-Critics.package/RBUndeclaredReferenceRule.class/instance/accessing/rationale.st R Refactoring-Critics.package/RBUndeclaredReferenceRule.class/instance/accessing/severity.st R Refactoring-Critics.package/RBUndeclaredReferenceRule.class/instance/running/basicCheck_.st R Refactoring-Critics.package/RBUndeclaredReferenceRule.class/instance/running/checkMethod_.st A Refactoring-Critics.package/RBVariableNotDefinedRule.class/class/instance creation/new.st R Refactoring-Critics.package/RBVariableNotDefinedRule.class/class/testing/checksMethod.st A Refactoring-Critics.package/RBVariableNotDefinedRule.class/class/testing/isVisible.st R Refactoring-Critics.package/RBVariableNotDefinedRule.class/instance/accessing/group.st R Refactoring-Critics.package/RBVariableNotDefinedRule.class/instance/accessing/name.st R Refactoring-Critics.package/RBVariableNotDefinedRule.class/instance/accessing/severity.st R Refactoring-Critics.package/RBVariableNotDefinedRule.class/instance/running/basicCheck_.st R Refactoring-Critics.package/RBVariableNotDefinedRule.class/instance/running/checkMethod_.st R ScriptLoader60.package/ScriptLoader.class/instance/pharo - scripts/script60361.st A ScriptLoader60.package/ScriptLoader.class/instance/pharo - scripts/script60362.st R ScriptLoader60.package/ScriptLoader.class/instance/pharo - updates/update60361.st A ScriptLoader60.package/ScriptLoader.class/instance/pharo - updates/update60362.st M ScriptLoader60.package/ScriptLoader.class/instance/public/commentForCurrentUpdate.st M WriteBarrierTests.package/WriteBarrierTest.class/instance/tests - object/testMutateIVObject.st M WriteBarrierTests.package/WriteBarrierTest.class/instance/tests - object/testMutateVariableObject.st Log Message: ----------- 60362 17351 Bitmap should have a class initialization https://pharo.fogbugz.com/f/cases/17351 18157 ModificationTime creationTime size https://pharo.fogbugz.com/f/cases/18157 19613 Problem with read-only objects https://pharo.fogbugz.com/f/cases/19613 19537 "Variable used, but not defined anywhere" critique false positive with some slots https://pharo.fogbugz.com/f/cases/19537 http://files.pharo.org/image/60/60362.zip

1 0

[pharo-project/pharo-core]
by GitHub Jan. 27, 2017

Jan. 27, 2017

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

1 0