- Pharo-dev - lists.pharo.org

Re: [Pharo-dev] Latest changes broke Icons

by phil＠highoctane.be

Have not tires #50359 but my WFT rate per minute just had a dirac spike. In TWM that I am trying to port to 5.0 there are tons of icons... Geez. Phil On Fri, Oct 2, 2015 at 7:40 PM, Torsten Bergmann <astares(a)gmx.de> wrote: > The latest changes on ThemeIcons completely broke functionality and > icons of external and included packages. > > 1. Intelli J Icons are now used as default, but most icons of > external packages (Pomodoro, MongoTools, EventRecorder, ScriptManager, > ....) > align with the eclipse icon set. Can we please stay with Eclipse as > default > so that we do not have to introduce intelliJ icons on each external > project as well? > > 2. The mechanisms of ThemeIcons class do not work anymore. The icons in > Versionner are lost. Also other external package who extended ThemeIcons > (like DesktopManager) are now broken. > Just try to load it into a Pharo5.0 Latest update: #50359 now > > 3. Some of the icons are removed - for instance in catalog and moved to > be retrieved from Github. But we wanted to componentize the image > into configs. So the icons of catalog should belong to this project. > Right? > > 4. I can not open the settings in the latest image anymore because the > icons > are tried to retrieve from Github using a HTTPS connection. > Doesnt work either because I'm on Windows, because of HTTPs, because > ZnClient does not deal with a firewall correctly. > It is not good to rely on additional external downloads directly. > > I know we want to make progress ... but were such changes discussed on the > list > before? > > Thx > T. > > > > >

Oct. 2, 2015

Re: [Pharo-dev] If all Pharo projects were on GitHub...

by Dale Henrichs

Excellent! On 09/30/2015 11:36 PM, Nicolai Hess wrote: > > > 2015-10-01 6:33 GMT+02:00 Jan Vrany <jan.vrany(a)fit.cvut.cz > <mailto:jan.vrany@fit.cvut.cz>>: > > On Mon, 2015-09-28 at 16:48 -0700, Dale Henrichs wrote: > > The big issue for git and windows (in general) is that there is > a 255 > > character limit on the paths for files .... I recently pruned some > > file names for Metacello (on a win-hack branch) just to get > Metacello > > to work with filetree .. > > The limit is actually 260 characters. Not that it helps much :-) > However, there's a way (well, hack) to overcome this limit > using special UNC pathnames like > \\?\C:\very\long\path\here or \\?\UNC\server\share\very\long\path. > > > This hack is now implemented in the latest squeak and pharo vm. > > > The "only" issue is that not all tools in the chain are using > this hack... > > Jan > > >

Oct. 2, 2015

Re: [Pharo-dev] Latest changes broke Icons

by Nicolai Hess

2015-10-02 19:40 GMT+02:00 Torsten Bergmann <astares(a)gmx.de>: > > 4. I can not open the settings in the latest image anymore because the > icons > are tried to retrieve from Github using a HTTPS connection. > > That is really bad, I am working behind a proxy too.

Oct. 2, 2015

Re: [Pharo-dev] Querying the accessors of a variable

by Eliot Miranda

On Fri, Oct 2, 2015 at 11:09 AM, Eliot Miranda <eliot.miranda(a)gmail.com> wrote: > Hi Henrik, > > On Thu, Oct 1, 2015 at 2:18 AM, Henrik Nergaard <henrin10(a)student.uia.no> > wrote: > >> Hi, >> >> Is there a simple way to query a variable to check how many methods that >> access it? (just want to know the size) >> >> This works, but it is quite slow. >> (Morph whichSelectorsAccess: #owner) size >> Morph withAllSubclasses inject: 0 into: [ :sum :class | sum + (class >> whichSelectorsAccess: #owner) size ] >> > > If it is slow then there's some mistake. Here are some measurements from > the relevant code in my Spur Squeak v 5.0 image: > > > | visited t | > visited := 0. > t := [self systemNavigation allMethodsSelect: [:m| visited := visited + 1. > (m readsField: 66) or: (m writesField: 66)] localToPackage: #VMMaker] > timeToRun. > {t. visited. t * 1000.0 / visited } #(89 13553 6.566811775990556) > > > | visited t | > visited := 0. > t := [self systemNavigation allMethodsSelect: [:m| visited := visited + 1. > (m readsField: 6) or: (m writesField: 6)] localToPackage: #VMMaker] > timeToRun. > {t. visited. t * 1000.0 / visited } #(85 13553 6.271674168080867) > > So 85 milliseconds to analyse 13k methods, for a cost of 6 1/4 secs per > method. I wouldn't call that slow. > > The measurements for Morph and subclasses are even better: > > > | visited t | > visited := 0. > t := [self systemNavigation allMethodsSelect: [:m| visited := visited + 1. > (m readsField: 66) or: (m writesField: 66)] localTo: Morph] timeToRun. > {t. visited. t * 1000.0 / visited } #(34 12434 2.7344378317516487) > | visited t | > visited := 0. > t := [self systemNavigation allMethodsSelect: [:m| visited := visited + 1. > (m readsField: 6) or: (m writesField: 6)] localTo: Morph] timeToRun. > {t. visited. t * 1000.0 / visited } #(34 12434 2.7344378317516487) > > 34 milliseconds, 2 3/4 usecs per method. > Oops. This should read | visited t | visited := 0. t := [self systemNavigation allMethodsSelect: [:m| visited := visited + 1. (m readsField: 66) or: (m writesField: 66)] localTo: Morph] timeToRun. {t. visited. t * 1000.0 / visited } #(33 12434 2.654013189641306) | visited t | visited := 0. t := [self systemNavigation allMethodsSelect: [:m| visited := visited + 1. (m readsField: 6) or: (m writesField: 6)] localTo: Morph] timeToRun. {t. visited. t * 1000.0 / visited } #(34 12434 2.7344378317516487) > > > This on a late 2012 Mac Mini 2.3GHz Core i7. > > >> <http://forum.world.st/file/n4852956/aMenuMorph%28885522432%29.png> >> >> Best regards, >> Henrik >> >> >> >> -- >> View this message in context: >> http://forum.world.st/Querying-the-accessors-of-a-variable-tp4852956.html >> Sent from the Pharo Smalltalk Developers mailing list archive at >> Nabble.com. >> >> > > > -- > _,,,^..^,,,_ > best, Eliot > -- _,,,^..^,,,_ best, Eliot

Oct. 2, 2015

Re: [Pharo-dev] Querying the accessors of a variable

by Eliot Miranda

Hi Henrik, On Thu, Oct 1, 2015 at 2:18 AM, Henrik Nergaard <henrin10(a)student.uia.no> wrote: > Hi, > > Is there a simple way to query a variable to check how many methods that > access it? (just want to know the size) > > This works, but it is quite slow. > (Morph whichSelectorsAccess: #owner) size > Morph withAllSubclasses inject: 0 into: [ :sum :class | sum + (class > whichSelectorsAccess: #owner) size ] > If it is slow then there's some mistake. Here are some measurements from the relevant code in my Spur Squeak v 5.0 image: | visited t | visited := 0. t := [self systemNavigation allMethodsSelect: [:m| visited := visited + 1. (m readsField: 66) or: (m writesField: 66)] localToPackage: #VMMaker] timeToRun. {t. visited. t * 1000.0 / visited } #(89 13553 6.566811775990556) | visited t | visited := 0. t := [self systemNavigation allMethodsSelect: [:m| visited := visited + 1. (m readsField: 6) or: (m writesField: 6)] localToPackage: #VMMaker] timeToRun. {t. visited. t * 1000.0 / visited } #(85 13553 6.271674168080867) So 85 milliseconds to analyse 13k methods, for a cost of 6 1/4 secs per method. I wouldn't call that slow. The measurements for Morph and subclasses are even better: | visited t | visited := 0. t := [self systemNavigation allMethodsSelect: [:m| visited := visited + 1. (m readsField: 66) or: (m writesField: 66)] localTo: Morph] timeToRun. {t. visited. t * 1000.0 / visited } #(34 12434 2.7344378317516487) | visited t | visited := 0. t := [self systemNavigation allMethodsSelect: [:m| visited := visited + 1. (m readsField: 6) or: (m writesField: 6)] localTo: Morph] timeToRun. {t. visited. t * 1000.0 / visited } #(34 12434 2.7344378317516487) 34 milliseconds, 2 3/4 usecs per method. This on a late 2012 Mac Mini 2.3GHz Core i7. > <http://forum.world.st/file/n4852956/aMenuMorph%28885522432%29.png> > > Best regards, > Henrik > > > > -- > View this message in context: > http://forum.world.st/Querying-the-accessors-of-a-variable-tp4852956.html > Sent from the Pharo Smalltalk Developers mailing list archive at > Nabble.com. > > -- _,,,^..^,,,_ best, Eliot

Oct. 2, 2015

Latest changes broke Icons

by Torsten Bergmann

The latest changes on ThemeIcons completely broke functionality and icons of external and included packages. 1. Intelli J Icons are now used as default, but most icons of external packages (Pomodoro, MongoTools, EventRecorder, ScriptManager, ....) align with the eclipse icon set. Can we please stay with Eclipse as default so that we do not have to introduce intelliJ icons on each external project as well? 2. The mechanisms of ThemeIcons class do not work anymore. The icons in Versionner are lost. Also other external package who extended ThemeIcons (like DesktopManager) are now broken. Just try to load it into a Pharo5.0 Latest update: #50359 now 3. Some of the icons are removed - for instance in catalog and moved to be retrieved from Github. But we wanted to componentize the image into configs. So the icons of catalog should belong to this project. Right? 4. I can not open the settings in the latest image anymore because the icons are tried to retrieve from Github using a HTTPS connection. Doesnt work either because I'm on Windows, because of HTTPs, because ZnClient does not deal with a firewall correctly. It is not good to rely on additional external downloads directly. I know we want to make progress ... but were such changes discussed on the list before? Thx T.

Oct. 2, 2015

[pharo-project/pharo-core]

by GitHub

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

Oct. 2, 2015

[pharo-project/pharo-core] 461f66: 50359

by GitHub

Branch: refs/heads/5.0 Home: https://github.com/pharo-project/pharo-core Commit: 461f66b36cc81a85ea80a6d6d14d87a60d691273 https://github.com/pharo-project/pharo-core/commit/461f66b36cc81a85ea80a6d6… Author: Jenkins Build Server <board(a)pharo-project.org> Date: 2015-10-02 (Fri, 02 Oct 2015) Changed paths: M NautilusCommon.package/NautilusPluginManager.class/instance/lists behavior/pluginClassesSelectedIndex_.st R Reflectivity-Examples.package/CoverageStyler.class/README.md R Reflectivity-Examples.package/CoverageStyler.class/definition.st R Reflectivity-Examples.package/CoverageStyler.class/instance/formatting/addStyle_attribute_forNode_.st R Reflectivity-Examples.package/CoverageStyler.class/instance/formatting/addStyle_forNode_.st A Reflectivity-Examples.package/ExecutionDisplayMorph.class/README.md A Reflectivity-Examples.package/ExecutionDisplayMorph.class/definition.st A Reflectivity-Examples.package/ExecutionDisplayMorph.class/instance/as yet unclassified/noteExecuted.st A Reflectivity-Examples.package/ExecutionDisplayMorph.class/instance/as yet unclassified/noteExecution.st A Reflectivity-Examples.package/ExecutionDisplayMorph.class/instance/as yet unclassified/step.st A Reflectivity-Examples.package/ExecutionDisplayMorph.class/instance/as yet unclassified/stepTime.st A Reflectivity-Examples.package/ExecutionDisplayMorph.class/instance/as yet unclassified/wantsSteps.st A Reflectivity-Examples.package/ExecutionDisplayPlugin.class/README.md A Reflectivity-Examples.package/ExecutionDisplayPlugin.class/class/display/pluginName.st A Reflectivity-Examples.package/ExecutionDisplayPlugin.class/class/information/description.st A Reflectivity-Examples.package/ExecutionDisplayPlugin.class/class/information/possiblePositions.st A Reflectivity-Examples.package/ExecutionDisplayPlugin.class/class/position/defaultPosition.st A Reflectivity-Examples.package/ExecutionDisplayPlugin.class/definition.st A Reflectivity-Examples.package/ExecutionDisplayPlugin.class/instance/display/display.st A Reflectivity-Examples.package/ExecutionDisplayPlugin.class/instance/execution Visualizer/noteExectuted_.st A Reflectivity-Examples.package/ExecutionDisplayPlugin.class/instance/initialization/initialize.st A Reflectivity-Examples.package/ExecutionDisplayPlugin.class/instance/registration/registerTo_.st A Reflectivity-Tests.package/MetaLinkTest.class/instance/tests - options/testLinkOptionMetaLevel.st A Reflectivity-Tests.package/ReflectivityControlTest.class/instance/tests - options/testLinkOptionDisabledLink.st M Reflectivity.package/HookGenerator.class/instance/private/hookFor_.st M Reflectivity.package/MetaLink.class/class/options/defaultOptions.st M Reflectivity.package/MetaLink.class/instance/accessing/level.st A Reflectivity.package/MetaLink.class/instance/options/optionDisabledLink.st A Reflectivity.package/MetaLink.class/instance/options/optionDisabledLink_.st A Reflectivity.package/MetaLink.class/instance/options/optionMetalevel.st A Reflectivity.package/MetaLink.class/instance/options/optionMetalevel_.st M Reflectivity.package/MetaLink.class/instance/private/doesNotUnderstand_.st R ScriptLoader50.package/ScriptLoader.class/instance/pharo - scripts/script50358.st A ScriptLoader50.package/ScriptLoader.class/instance/pharo - scripts/script50359.st R ScriptLoader50.package/ScriptLoader.class/instance/pharo - updates/update50358.st A ScriptLoader50.package/ScriptLoader.class/instance/pharo - updates/update50359.st M ScriptLoader50.package/ScriptLoader.class/instance/public/commentForCurrentUpdate.st M Slot.package/MorphSlot.class/README.md R Slot.package/MorphSlot.class/instance/as yet unclassified/write_to_.st A Slot.package/MorphSlot.class/instance/meta-object-protocol/write_to_.st Log Message: ----------- 50359 16696 Execution Demo with Reflectivity https://pharo.fogbugz.com/f/cases/16696 16694 optionMetalevel and some cleanups https://pharo.fogbugz.com/f/cases/16694 16697 NautliusPluginManager context menu broken https://pharo.fogbugz.com/f/cases/16697 http://files.pharo.org/image/50/50359.zip

Oct. 2, 2015

[pharo-project/pharo-core]

by GitHub

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

Oct. 2, 2015

[pharo-project/pharo-core] eb25da: 50358

by GitHub

Branch: refs/heads/5.0 Home: https://github.com/pharo-project/pharo-core Commit: eb25da0bdf9bbbf574666cbb8de8023fcbb1d032 https://github.com/pharo-project/pharo-core/commit/eb25da0bdf9bbbf574666cbb… Author: Jenkins Build Server <board(a)pharo-project.org> Date: 2015-10-02 (Fri, 02 Oct 2015) Changed paths: M ConfigurationOfCatalog.package/ConfigurationOfCatalog.class/instance/symbolic versions/stable_.st A ConfigurationOfCatalog.package/ConfigurationOfCatalog.class/instance/versions/version091_.st M ConfigurationOfGTInspectorCore.package/ConfigurationOfGTInspectorCore.class/instance/symbolic versions/development_.st M ConfigurationOfGTInspectorCore.package/ConfigurationOfGTInspectorCore.class/instance/symbolic versions/stable_.st A ConfigurationOfGTInspectorCore.package/ConfigurationOfGTInspectorCore.class/instance/versions/version2020_.st A ConfigurationOfGTInspectorCore.package/ConfigurationOfGTInspectorCore.class/instance/versions/version2021_.st M ConfigurationOfGTPlaygroundCore.package/ConfigurationOfGTPlaygroundCore.class/instance/symbolic versions/development_.st M ConfigurationOfGTPlaygroundCore.package/ConfigurationOfGTPlaygroundCore.class/instance/symbolic versions/stable_.st A ConfigurationOfGTPlaygroundCore.package/ConfigurationOfGTPlaygroundCore.class/instance/versions/version2110_.st M ConfigurationOfGTSpotter.package/ConfigurationOfGTSpotter.class/instance/symbolic versions/development_.st M ConfigurationOfGTSpotter.package/ConfigurationOfGTSpotter.class/instance/symbolic versions/stable_.st A ConfigurationOfGTSpotter.package/ConfigurationOfGTSpotter.class/instance/versions/version1213_.st M GT-InspectorExtensions-Core.package/extension/Bitmap/class/gtExampleIconBits.st M GT-InspectorExtensions-Core.package/extension/FileReference/class/gtExampleGif.st M GT-InspectorExtensions-Core.package/extension/FileReference/class/gtExamplePng.st R GT-InspectorExtensions-Core.package/extension/ThemeIcons/instance/gtInspectorIconsIn_.st M GT-Playground.package/GTPlayground.class/instance/building/codeIn_.st A GT-SpotterExtensions-Core.package/extension/ThemeIconPack/class/gtExampleThemeIcons.st R GT-SpotterExtensions-Core.package/extension/ThemeIcons/class/gtExampleThemeIcons.st R GT-SpotterExtensions-Core.package/extension/ThemeIcons/instance/spotterForIconsFor_.st R ScriptLoader50.package/ScriptLoader.class/instance/pharo - scripts/script50357.st A ScriptLoader50.package/ScriptLoader.class/instance/pharo - scripts/script50358.st R ScriptLoader50.package/ScriptLoader.class/instance/pharo - updates/update50357.st A ScriptLoader50.package/ScriptLoader.class/instance/pharo - updates/update50358.st M ScriptLoader50.package/ScriptLoader.class/instance/public/commentForCurrentUpdate.st R Tool-Catalog.package/extension/ThemeIcons/instance/catalogIcon.st R Tool-Catalog.package/extension/ThemeIcons/instance/catalogIconContents.st R Tool-Catalog.package/extension/ThemeIcons/instance/configIcon.st R Tool-Catalog.package/extension/ThemeIcons/instance/configIconContents.st R Tool-Catalog.package/extension/ThemeIcons/instance/configIconLoaded.st R Tool-Catalog.package/extension/ThemeIcons/instance/configIconLoadedContents.st Log Message: ----------- 50358 16685 remove ThemeIcons references in GTools https://pharo.fogbugz.com/f/cases/16685 16702 remove ThemeIcons references in GTInspector https://pharo.fogbugz.com/f/cases/16702 16701 remove ThemeIcons references in Catalog https://pharo.fogbugz.com/f/cases/16701 16700 remove ThemeIcons references in GTSpotter https://pharo.fogbugz.com/f/cases/16700 http://files.pharo.org/image/50/50358.zip

Oct. 2, 2015