- Pharo-dev - lists.pharo.org

[pharo-project/pharo-core]
by GitHub Aug. 26, 2015

Aug. 26, 2015

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

1 0

Working Directory and FileSystem
by Guillermo Polito Aug. 26, 2015

Aug. 26, 2015

(I cc the VM dev as this topic touches a bit the VM) Hi, chasing a bug I found that FileSystem does not handle well the working directory of the system. The working directory of a process is the directory from where we started the process. That is, the directory from where we launched the VM with the image. However, FileSystem has two problems with this: 1) it believes that the workingDirectory is where the image *is* and not where it was open. (Check: FileSystem workingDirectory). 2) if you try to use a relative file name FileSystem will use the working directory he believes instead of letting the primitive do his job. (Just open your image from another directory and do 'asd.txt' asFileReference fullName). This problem becomes apparent when you use the launcher, or when you launch your image from other directory. Now, I was checking with Santi how to obtain the working directory of the process and we saw that there are two primitives """"included""" in the VM as part of the OSProcess plugins to do it (Included as they are in the PharoVM build). <primitive: 'primitiveGetCurrentWorkingDirectory' module: 'Win32OSProcessPlugin' > AND <primitive: 'primitiveGetCurrentWorkingDirectory' module: 'UnixOSProcessPlugin' > So I'd like to propose: - include those primitives in the default VM (as we already have the ones to get the image file path and the vm file path). - Add methods calling those primitives in OSPlatform and subclasses - Change FileSystem to have the working directory work as expected. What do you think? Guille

2 1

Strange Validation with issue 15820
by Guillermo Polito Aug. 26, 2015

Aug. 26, 2015

Hi, We have an issue that is platform dependent (file-related): https://pharo.fogbugz.com/f/cases/15820/Better-File-API Now, we tested with Nicolai in Windows, Mac and Linux and we had all tests green. However, the monkey does not seem to want to be green. - Would anyone have an idea of what should be disturbing the monkey? - Can people in different linuxes/unixes test the slice just to make sure we are on the right path? (that is, load the slice and run the tests in the File package). Thanks a lot, Guille

3 4

[pharo-project/pharo-core]
by GitHub Aug. 26, 2015

Aug. 26, 2015

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

1 0

[pharo-project/pharo-core] 3bb420: 50271
by GitHub Aug. 26, 2015

Aug. 26, 2015

Branch: refs/heads/5.0 Home: https://github.com/pharo-project/pharo-core Commit: 3bb4203ad985bc9426b19cfe1a4a90ee7ef75064 https://github.com/pharo-project/pharo-core/commit/3bb4203ad985bc9426b19cfe… Author: Jenkins Build Server <board(a)pharo-project.org> Date: 2015-08-26 (Wed, 26 Aug 2015) Changed paths: M Komitter.package/KomitterUI.class/instance/initialization/initializeLatelyPresenter.st M Komitter.package/KomitterUI.class/instance/initialization/initializeWidgets.st A Komitter.package/KomitterUI.class/instance/private/refreshDiffForItem_.st M Morphic-Base.package/extension/GradientFillStyle/instance/changeColorOf_rampIndex_.st R Nautilus.package/AbstractNautilusUI.class/class/menu/debugMenu_.st M Nautilus.package/AbstractNautilusUI.class/instance/group announcements/aGroupHasBeenAdded_.st M Nautilus.package/AbstractNautilusUI.class/instance/group announcements/aGroupHasBeenRemoved_.st R Nautilus.package/Nautilus.class/class/accessing/historyMaxSize.st R Nautilus.package/Nautilus.class/class/accessing/historyMaxSize_.st R Nautilus.package/Nautilus.class/class/accessing/maxSize.st R Nautilus.package/Nautilus.class/class/accessing/maxSize_.st R Nautilus.package/Nautilus.class/class/accessing/openOnGroups.st R Nautilus.package/Nautilus.class/class/accessing/openOnGroups_.st M Nautilus.package/Nautilus.class/class/accessing/resetGroups.st R Nautilus.package/Nautilus.class/class/accessing/showAnnotationPane_.st M Nautilus.package/Nautilus.class/class/opening/openInEnvironment_.st M Nautilus.package/Nautilus.class/class/opening/openOnClass_inEnvironment_.st M Nautilus.package/Nautilus.class/class/opening/openOnClass_onProtocol_inEnvironment_.st M Nautilus.package/Nautilus.class/class/opening/openOnMethod_inEnvironment_.st R Nautilus.package/Nautilus.class/class/settings/commentPosition.st R Nautilus.package/Nautilus.class/class/settings/commentPosition_.st R Nautilus.package/Nautilus.class/class/settings/commentPositions.st R Nautilus.package/Nautilus.class/class/settings/emptyCommentWarning_.st M Nautilus.package/Nautilus.class/class/settings/nautilusSettingsOn_.st R Nautilus.package/Nautilus.class/class/settings/showDebugMenus.st R Nautilus.package/Nautilus.class/class/settings/showDebugMenus_.st R Nautilus.package/Nautilus.class/class/settings/showGroupsOnTop.st R Nautilus.package/Nautilus.class/class/settings/showGroupsOnTop_.st R Nautilus.package/Nautilus.class/class/settings/switchClassesAndPackages.st R Nautilus.package/Nautilus.class/class/settings/switchClassesAndPackages_.st M Nautilus.package/Nautilus.class/definition.st R Nautilus.package/Nautilus.class/instance/accessing/commentPosition.st R Nautilus.package/Nautilus.class/instance/accessing/switchClassesAndPackages.st R Nautilus.package/Nautilus.class/instance/browser compatibility/openEditString_.st M Nautilus.package/Nautilus.class/instance/browser compatibility/spawnHierarchy.st M Nautilus.package/Nautilus.class/instance/browser compatibility/spawnHierarchyForClass_selector_.st R Nautilus.package/Nautilus.class/instance/history/removeLastHistoryEntry.st R Nautilus.package/Nautilus.class/instance/private/packagesUsedByTheSelectedClass.st R Nautilus.package/Nautilus.class/instance/private/shouldOpenOnGroups.st M Nautilus.package/PackageTreeModel.class/instance/accessing/rootNodes.st R Nautilus.package/PackageTreeNautilusUI.class/instance/accessing/groupsAreVisible.st R NautilusCommon.package/NautilusHistoryEntryWithSourceCode.class/README.md R NautilusCommon.package/NautilusHistoryEntryWithSourceCode.class/class/instance creation/with_sourceCode_.st R NautilusCommon.package/NautilusHistoryEntryWithSourceCode.class/definition.st R NautilusCommon.package/NautilusHistoryEntryWithSourceCode.class/instance/accessing/sourceCode.st R NautilusCommon.package/NautilusHistoryEntryWithSourceCode.class/instance/accessing/sourceCode_.st R NautilusCommon.package/NautilusHistoryEntryWithSourceCode.class/instance/protocol/applyTo_.st R NautilusCommon.package/NautilusHistoryEntryWithSourceCode.class/instance/protocol/asHistoryString.st A Polymorph-Widgets.package/MorphicUIManager.class/instance/accessing/theme.st A Polymorph-Widgets.package/MorphicUIManager.class/instance/ui requests/chooseOrRequestFrom_title_.st A Polymorph-Widgets.package/MorphicUIManager.class/instance/ui requests/enterOrRequestFrom_title_.st R ScriptLoader50.package/ScriptLoader.class/instance/pharo - scripts/script50270.st A ScriptLoader50.package/ScriptLoader.class/instance/pharo - scripts/script50271.st R ScriptLoader50.package/ScriptLoader.class/instance/pharo - updates/update50270.st A ScriptLoader50.package/ScriptLoader.class/instance/pharo - updates/update50271.st M ScriptLoader50.package/ScriptLoader.class/instance/public/commentForCurrentUpdate.st M Shout.package/SHPreferences.class/class/settings/styleTableRow.st M Tools.package/SyntaxErrorDebugger.class/class/instance creation/buildMorphicViewOn_.st M Tools.package/SyntaxErrorDebugger.class/definition.st A Tools.package/SyntaxErrorDebugger.class/instance/accessing/announcer.st M Tools.package/SyntaxErrorDebugger.class/instance/accessing/codePaneMenu_shifted_.st A Tools.package/SyntaxErrorDebugger.class/instance/accessing/model.st R Tools.package/SyntaxErrorDebugger.class/instance/accessing/shiftedYellowButtonMenu.st R Tools.package/SyntaxErrorDebugger.class/instance/accessing/yellowButtonMenu.st M Tools.package/SyntaxErrorDebugger.class/instance/other/contents_notifying_.st M UIManager.package/ManifestUIManager.class/class/meta-data - dependency analyser/manuallyResolvedDependencies.st R UIManager.package/UIManager.class/instance/accessing/theme.st R UIManager.package/UIManager.class/instance/ui requests/chooseOrRequestFrom_title_.st R UIManager.package/UIManager.class/instance/ui requests/enterOrRequestFrom_title_.st Log Message: ----------- 50271 16385 Clean a lot of dead code and unused class variables in Nautilus https://pharo.fogbugz.com/f/cases/16385 16382 Remove PluggableTextMorph reference in SyntaxErrorDebugger https://pharo.fogbugz.com/f/cases/16382 16378 Highlighting code in Komitter https://pharo.fogbugz.com/f/cases/16378 16190 UIManager should not depends on Polymorph-Widgets https://pharo.fogbugz.com/f/cases/16190 http://files.pharo.org/image/50/50271.zip

1 0

Re: [Pharo-dev] [Moose-dev] Re: Object new foo
by Vincent BLONDEAU Aug. 25, 2015

Aug. 25, 2015

Hi, On last Moose image, I debugged the VM and executed âException signalâ. I paused the VM and saw that 2 methods are looping together in gcc3x-cointerp.c: static sqInt findMethodWithPrimitiveFromContextUpToContext(sqInt primitive, sqInt senderContext, sqInt homeContext); static sqInt findMethodWithPrimitiveFromFPUpToContext(sqInt primitive, char *startFP, sqInt homeContext); The image is looping only when the debugger is launch on Error (not by launching it by the script below). Maybe someone on the Pharo mailing list knows what is happening? Thanks in advance, Cheers, Vincent De : moose-dev-bounces(a)iam.unibe.ch [mailto:moose-dev-bounces@iam.unibe.ch] De la part de Andrei Chis EnvoyÃ© : mardi 18 aoÃ»t 2015 10:53 Ã : Moose-related development Objet : [Moose-dev] Re: Object new foo Hi Vincent, You example will also kill the Pharo debugger :) The issue is that you pass 'Processor activeProcess' which gets terminated when you close the debugger. When you open the debugger you need to make sure the process on which it is opened is suspended and can get terminated with no problem. A different way to redo the example would be: context := Context sender: nil receiver: {1 . 2 . 2} method: (OrderedCollection>>#add:) arguments: #(10). process := Process forContext: context priority: Processor userInterruptPriority. GTGenericStackDebugger openOn: (DebugSession process: process context: context). There is a basic test for opening the debugger in GTGenericStackDebuggerSmokeTest but more could help. Cheers, Andrei On Mon, Aug 17, 2015 at 11:53 PM, Vincent BLONDEAU <vincent.blondeau(a)polytech-lille.net <mailto:vincent.blondeau@polytech-lille.net> > wrote: Indeed. Maybe some tests has to be added.. But for now, it seems that the debugger is crashing the imageâ¦ Not just by launching it: GTGenericStackDebugger openOn: (DebugSession process: Processor activeProcess context: (Context sender: nil receiver: {1 . 2 . 2} method: (OrderedCollection>>#add: ) arguments: #(10))) But when an exception is thrown, its seems that we fall into an infinite loop around the primitive 199. Maybe the context is not well set? Cheers, Vincent De : moose-dev-bounces(a)iam.unibe.ch <mailto:moose-dev-bounces@iam.unibe.ch> [mailto:moose-dev-bounces@iam.unibe.ch <mailto:moose-dev-bounces@iam.unibe.ch> ] De la part de Alexandre Bergel EnvoyÃ© : lundi 17 aoÃ»t 2015 13:26 Ã : Moose-related development Objet : [Moose-dev] Re: Object new foo This is weird that such bug can be introduced and no alarm is raised Alexandre Le 17 aoÃ»t 2015 Ã 08:03, Jan KurÅ¡ <kurs(a)iam.unibe.ch <mailto:kurs@iam.unibe.ch> > a Ã©crit : Yep I had this problem (falled back to previous version), I suspected debugger being the reason. Cheers Jan On Sun, 16 Aug 2015 03:05 Alexandre Bergel <alexandre.bergel(a)me.com <mailto:alexandre.bergel@me.com> > wrote: in Moose 6.0 simply freezes. Anyone else sees this? Alexandre -- _,.;:~^~:;._,.;:~^~:;._,.;:~^~:;._,.;:~^~:;._,.;: Alexandre Bergel http://www.bergel.eu ^~:;._,.;:~^~:;._,.;:~^~:;._,.;:~^~:;._,.;:~^~:;. _______________________________________________ Moose-dev mailing list Moose-dev(a)iam.unibe.ch <mailto:Moose-dev@iam.unibe.ch> https://www.iam.unibe.ch/mailman/listinfo/moose-dev _______________________________________________ Moose-dev mailing list Moose-dev(a)iam.unibe.ch <mailto:Moose-dev@iam.unibe.ch> https://www.iam.unibe.ch/mailman/listinfo/moose-dev _______________________________________________ Moose-dev mailing list Moose-dev(a)iam.unibe.ch <mailto:Moose-dev@iam.unibe.ch> https://www.iam.unibe.ch/mailman/listinfo/moose-dev

3 4

[pharo-project/pharo-core] 2748ef: 50270
by GitHub Aug. 25, 2015

Aug. 25, 2015

Branch: refs/heads/5.0 Home: https://github.com/pharo-project/pharo-core Commit: 2748ef632838d482014c7a033d3bbb622ef3c967 https://github.com/pharo-project/pharo-core/commit/2748ef632838d482014c7a03… Author: Jenkins Build Server <board(a)pharo-project.org> Date: 2015-08-25 (Tue, 25 Aug 2015) Changed paths: A CodeExport.package/extension/FileStream/class/writeSourceCodeFrom_baseName_isSt_.st R Files.package/FileStream.class/class/file reader services/writeSourceCodeFrom_baseName_isSt_.st M Files.package/FileStream.class/class/initialize-release/initialize.st M Files.package/ManifestFiles.class/class/meta-data - dependency analyser/manuallyResolvedDependencies.st R Files.package/extension/MacOSPlatform/instance/potentialLocationsOfSourcesFile.st R Files.package/extension/OSPlatform/instance/potentialLocationsOfSourcesFile.st R Files.package/extension/String/instance/asFileName.st R Files.package/extension/UnixPlatform/instance/potentialLocationsOfSourcesFile.st R Files.package/extension/Win32Platform/instance/potentialLocationsOfSourcesFile.st R Kernel.package/Character.class/class/accessing untypeable characters/euro.st R Kernel.package/Character.class/instance/accessing/digitValue2.st M Kernel.package/CompiledMethod.class/instance/literals/hasLiteralThorough_.st R Kernel.package/Object.class/instance/dependencies/release.st A Multilingual-Encodings.package/extension/Character/class/euro.st A Reflectivity-Tests.package/ReflectivityControlTest.class/instance/tagging/return3.st A Reflectivity-Tests.package/ReflectivityControlTest.class/instance/tests - instead/testInsteadAssign.st A Reflectivity-Tests.package/ReflectivityControlTest.class/instance/tests - instead/testInsteadBlock.st A Reflectivity-Tests.package/ReflectivityControlTest.class/instance/tests - instead/testInsteadLiteral.st A Reflectivity-Tests.package/ReflectivityControlTest.class/instance/tests - instead/testInsteadLiteralArray.st A Reflectivity-Tests.package/ReflectivityControlTest.class/instance/tests - instead/testInsteadReturn.st A Reflectivity-Tests.package/ReflectivityControlTest.class/instance/tests - instead/testInsteadSend.st A Reflectivity-Tests.package/ReflectivityControlTest.class/instance/tests - instead/testInsteadVariableReadGlobal.st A Reflectivity-Tests.package/ReflectivityControlTest.class/instance/tests - instead/testInsteadVariableReadIvar.st A Reflectivity-Tests.package/ReflectivityControlTest.class/instance/tests - instead/testInsteadVariableReadTemp.st A Reflectivity-Tests.package/ReflectivityControlTest.class/instance/tests - instead/testInsteadVariableWrite.st R Reflectivity-Tests.package/ReflectivityControlTest.class/instance/tests - instead/testReplaceAssign.st R Reflectivity-Tests.package/ReflectivityControlTest.class/instance/tests - instead/testReplaceReturn.st R Reflectivity-Tests.package/ReflectivityControlTest.class/instance/tests - instead/testReplaceSend.st R Reflectivity-Tests.package/ReflectivityControlTest.class/instance/tests - instead/testReplaceVariableReadGlobal.st R Reflectivity-Tests.package/ReflectivityControlTest.class/instance/tests - instead/testReplaceVariableReadIvar.st R Reflectivity-Tests.package/ReflectivityControlTest.class/instance/tests - instead/testReplaceVariableReadTemp.st R Reflectivity-Tests.package/ReflectivityControlTest.class/instance/tests - instead/testReplaceVariableWrite.st M Reflectivity.package/RFASTTranslator.class/instance/visitor-double dispatching/visitLiteralArrayNode_.st M Reflectivity.package/RFASTTranslator.class/instance/visitor-double dispatching/visitLiteralNode_.st R ScriptLoader50.package/ScriptLoader.class/instance/pharo - scripts/script50269.st A ScriptLoader50.package/ScriptLoader.class/instance/pharo - scripts/script50270.st R ScriptLoader50.package/ScriptLoader.class/instance/pharo - updates/update50269.st A ScriptLoader50.package/ScriptLoader.class/instance/pharo - updates/update50270.st M ScriptLoader50.package/ScriptLoader.class/instance/public/commentForCurrentUpdate.st A System-Object Events.package/extension/Object/instance/release.st A System-Sources.package/ChangesLog.class/README.md A System-Sources.package/ChangesLog.class/class/accessing/default.st A System-Sources.package/ChangesLog.class/class/accessing/reset.st A System-Sources.package/ChangesLog.class/definition.st A System-Sources.package/ChangesLog.class/instance/accessing/recordStartupStamp.st A System-Sources.package/ChangesLog.class/instance/event-listening/logClassRemoved_.st A System-Sources.package/ChangesLog.class/instance/event-listening/logClassRenamed_.st A System-Sources.package/ChangesLog.class/instance/event-listening/logExpressionEvaluated_.st A System-Sources.package/ChangesLog.class/instance/event-listening/logMethodRemoved_.st A System-Sources.package/ChangesLog.class/instance/event-listening/registerToAnnouncements.st A System-Sources.package/ChangesLog.class/instance/logging/logChange_.st A System-Sources.package/ChangesLog.class/instance/logging/logSnapshot_andQuit_.st A System-Sources.package/ChangesLog.class/instance/private/assureStartupStampLogged.st A System-Sources.package/extension/MacOSPlatform/instance/potentialLocationsOfSourcesFile.st A System-Sources.package/extension/OSPlatform/instance/potentialLocationsOfSourcesFile.st A System-Sources.package/extension/UnixPlatform/instance/potentialLocationsOfSourcesFile.st A System-Sources.package/extension/Win32Platform/instance/potentialLocationsOfSourcesFile.st R System-Support.package/ChangesLog.class/README.md R System-Support.package/ChangesLog.class/class/accessing/default.st R System-Support.package/ChangesLog.class/class/accessing/reset.st R System-Support.package/ChangesLog.class/definition.st R System-Support.package/ChangesLog.class/instance/accessing/recordStartupStamp.st R System-Support.package/ChangesLog.class/instance/event-listening/logClassRemoved_.st R System-Support.package/ChangesLog.class/instance/event-listening/logClassRenamed_.st R System-Support.package/ChangesLog.class/instance/event-listening/logExpressionEvaluated_.st R System-Support.package/ChangesLog.class/instance/event-listening/logMethodRemoved_.st R System-Support.package/ChangesLog.class/instance/event-listening/registerToAnnouncements.st R System-Support.package/ChangesLog.class/instance/logging/logChange_.st R System-Support.package/ChangesLog.class/instance/logging/logSnapshot_andQuit_.st R System-Support.package/ChangesLog.class/instance/private/assureStartupStampLogged.st A Tool-FileList.package/extension/String/instance/asFileName.st R Traits.package/TClass.class/class/fileIn%2FOut/fileOutPool_.st Log Message: ----------- 50270 16367 #hasLiteralThorough: should not check for names of Binding https://pharo.fogbugz.com/f/cases/16367 16189 System-Sources has low-level dependents https://pharo.fogbugz.com/f/cases/16189 16375 More tests for instead links https://pharo.fogbugz.com/f/cases/16375 http://files.pharo.org/image/50/50270.zip

1 0

[pharo-project/pharo-core]
by GitHub Aug. 25, 2015

Aug. 25, 2015

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

1 0

Cleaning up Locale, LanguageEnvironment and EncodedCharSet
by Guillermo Polito Aug. 25, 2015

Aug. 25, 2015

Hi guys, I was having a deep look today at Locale, LanguageEnvironment and EncodedCharset with the objective of understanding it and see how we can organize it better. As I saw, in almost all cases, it happened that the locale and language environment are only used to get the current system's encoding. Even, it is not that the system's encoding is obtained from the system's configuration but guessed. Also, I saw that the usage of leadingCharacter is very limited in the image and I'd say that most of the time we would not be using it: we will be using unicode. I proposed in the issue tracker a change in three steps that cleans up this: - Cleaning Locale's API ( https://pharo.fogbugz.com/f/cases/16379/Cleanup-Locale-API) - Fix users to make use of this new API ( https://pharo.fogbugz.com/f/cases/16380/Make-use-of-new-Locale-API) - Remove old unused code ( https://pharo.fogbugz.com/f/cases/16381/Removed-unused-Locale-code) I know that with this change we lose temporarily the ability to use other language environments (like greek or japanese), and thus, change to other system encodings that are not UTF8/16/32. However, I believe that we should not 'guess' the system converter from the system language but to ask the system the encoding it is using. And this should maybe be added as a primitive (as the others already existing in the Locale class for example). If somebody could review these issues or has some comment on something that I should not be aware of, I'd be grateful :). Guille

2 4

[pharo-project/pharo-core]
by GitHub Aug. 25, 2015

Aug. 25, 2015

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

1 0