November 2015 - Pharo-dev - lists.pharo.org

Re: [Pharo-dev] A Question about Metacello's #repository:

by Thierry Goubier

2015-11-17 16:21 GMT+01:00 Sven Van Caekenberghe <sven(a)stfx.eu>: > > > On 17 Nov 2015, at 15:44, Thierry Goubier <thierry.goubier(a)gmail.com> > wrote: > > > > > > > > 2015-11-17 15:37 GMT+01:00 Sven Van Caekenberghe <sven(a)stfx.eu>: > > > > > On 17 Nov 2015, at 15:30, Thierry Goubier <thierry.goubier(a)gmail.com> > wrote: > > > > > > > > > > > > 2015-11-17 15:12 GMT+01:00 Sven Van Caekenberghe <sven(a)stfx.eu>: > > > Hi, > > > > > > I have a question about the way repositories are specified in > Metacello (i.e. by means on #repository:). > > > > > > Say I use FileTree where the files come from a classic VCS or just > from some archive. Using ZeroConf I can get an image+vm, etc. I can use the > config command line handler to load the configuration from a local file > directory (I haven't tried this yet, but I assume no scheme results in a > MCDirectoryRepository). So far so good. > > > > > > The question is, what do I put in as argument to #repository: in my > baseline ? > > > > > > baseline1: spec > > > <version: '1-baseline'> > > > > > > spec for: #common do: [ > > > spec > > > blessing: #baseline; > > > repository: 'http://mc.stfx.eu/Neo'; > > > package: 'Neo-Console-Core'; > > > group: 'default' with: #('Neo-Console-Core'); > > > group: 'Core' with: #('Neo-Console-Core') ] > > > > > > Obviously not a remote http repository. But also not an absolute path, > since I do not know where the user placed the files. Ideally, it should be > possible to leave out repository and let it resolve to wherever the > configuration was located. Can that be done ? Or is there another way to do > this ? > > > > > > I'm not sure it can be done that way. > > > > > > Loading from a local repository can be done on the command line, by > using Metacello directly with a local path. This is what I do for projects > where the repository is cloned before the start of Pharo. > > > > How do you do that ? > > > > "Loading from a local repository can be done on the command line, by > using Metacello directly with a local path." > > > > Suppose I have a repository that contains both my Configuration and my > code (except for the dependencies), let's say at $WORKDIR/repository. > > > > $ pharo Pharo.image config $WORKDIR/repository/ ConfigurationOfMyProject > --install > > > > $ pharo Pharo.image eval --save Metacello new configuration: > \'MyProject\'\; repository: \'$WORKDIR/repository\'\; load > > > > (Not sure about the way to designate just a file repository for > Metacello) > > Yes, but that is basically the same as what I was doing. It loads the > Metacello configuration from the directory and invokes load. The question > is, how can I specify in the configuration that it has to load its own code > from the same repo (which I cannot know absolutely when writing the > configuration). > > I think I asked Dale about that once, and I now consider that it can't be done. Thierry

Nov. 17, 2015

Re: [Pharo-dev] A Question about Metacello's #repository:

by Sven Van Caekenberghe

> On 17 Nov 2015, at 15:44, Thierry Goubier <thierry.goubier(a)gmail.com> wrote: > > > > 2015-11-17 15:37 GMT+01:00 Sven Van Caekenberghe <sven(a)stfx.eu>: > > > On 17 Nov 2015, at 15:30, Thierry Goubier <thierry.goubier(a)gmail.com> wrote: > > > > > > > > 2015-11-17 15:12 GMT+01:00 Sven Van Caekenberghe <sven(a)stfx.eu>: > > Hi, > > > > I have a question about the way repositories are specified in Metacello (i.e. by means on #repository:). > > > > Say I use FileTree where the files come from a classic VCS or just from some archive. Using ZeroConf I can get an image+vm, etc. I can use the config command line handler to load the configuration from a local file directory (I haven't tried this yet, but I assume no scheme results in a MCDirectoryRepository). So far so good. > > > > The question is, what do I put in as argument to #repository: in my baseline ? > > > > baseline1: spec > > <version: '1-baseline'> > > > > spec for: #common do: [ > > spec > > blessing: #baseline; > > repository: 'http://mc.stfx.eu/Neo'; > > package: 'Neo-Console-Core'; > > group: 'default' with: #('Neo-Console-Core'); > > group: 'Core' with: #('Neo-Console-Core') ] > > > > Obviously not a remote http repository. But also not an absolute path, since I do not know where the user placed the files. Ideally, it should be possible to leave out repository and let it resolve to wherever the configuration was located. Can that be done ? Or is there another way to do this ? > > > > I'm not sure it can be done that way. > > > > Loading from a local repository can be done on the command line, by using Metacello directly with a local path. This is what I do for projects where the repository is cloned before the start of Pharo. > > How do you do that ? > > "Loading from a local repository can be done on the command line, by using Metacello directly with a local path." > > Suppose I have a repository that contains both my Configuration and my code (except for the dependencies), let's say at $WORKDIR/repository. > > $ pharo Pharo.image config $WORKDIR/repository/ ConfigurationOfMyProject --install > > $ pharo Pharo.image eval --save Metacello new configuration: \'MyProject\'\; repository: \'$WORKDIR/repository\'\; load > > (Not sure about the way to designate just a file repository for Metacello) Yes, but that is basically the same as what I was doing. It loads the Metacello configuration from the directory and invokes load. The question is, how can I specify in the configuration that it has to load its own code from the same repo (which I cannot know absolutely when writing the configuration). > How do I tell ConfigurationOfMyProject to load its code from $WORKDIR/repository ? Maybe this is really simple and I just don't see it ? > > Maybe there is a simpler way than the one I use. > > Thierry > > > > Thierry > > > > > > > > > > Thanks, > > > > Sven > > > > > > > > > > >

Nov. 17, 2015

EyeInspector with subclasses of ProtoObject

by Esteban A. Maringolo

I'm dealing with the MessageArchiver of GLORP which is a subclass of ProtoObject and handles the "transparent" creation of expressions vÃa a DNU resolution. The problem I'm facing is that the EyeInspector expects the value in its value holder to respond to #basicSize, and because MessageArchiver inherits from ProtoObject it doesn't understand it. The problem is here: EyeInspector>>#variableFieldsToShow "Answers the indexes of the variable fields of the object to show. Shorten the list for very long collection (cf limit1 and limit2)" |bSize| bSize := self objectVariableSize. ^ bSize <= (self limit1 + self limit2) ifTrue: [1 to: bSize] ifFalse: [(1 to: self limit1) , (bSize - self limit2 + 1 to: bSize)]. If I implement #basicSize in MessageArchiver things go bananas. If I doesn't, and because #objectVariableSize is sent to MessageAchiver "proxy", bSize ends up with an instance of the MessageArchiver instead of an integer. I implemented #inspectorClass in MessageArchiver class to return EyeBasicInspector, but it is the same. Question: Is it documented what I should define/override in my classes to support EyeInspector? Regards, Esteban A. Maringolo pd: Stack trace =========== UndefinedObject(Object)>>error: MessageArchiver>>mustBeBoolean EyeBasicInspector(EyeInspector)>>variableFieldsToShow EyeBasicInspector(EyeInspector)>>addVariableFields: EyeBasicInspector(EyeInspector)>>generateElements EyeBasicInspector(EyeInspector)>>updateList EyeBasicInspector(EyeInspector)>>objectChanged [ self objectChanged ] in EyeBasicInspector(EyeAbstractInspector)>>initializePresenter in Block: [ self objectChanged ] BlockClosure>>cull: BlockClosure>>cull:cull: BlockClosure>>cull:cull:cull: BlockClosure>>cull:cull:cull:cull:

Nov. 17, 2015

Re: [Pharo-dev] A Question about Metacello's #repository:

by Thierry Goubier

2015-11-17 15:37 GMT+01:00 Sven Van Caekenberghe <sven(a)stfx.eu>: > > > On 17 Nov 2015, at 15:30, Thierry Goubier <thierry.goubier(a)gmail.com> > wrote: > > > > > > > > 2015-11-17 15:12 GMT+01:00 Sven Van Caekenberghe <sven(a)stfx.eu>: > > Hi, > > > > I have a question about the way repositories are specified in Metacello > (i.e. by means on #repository:). > > > > Say I use FileTree where the files come from a classic VCS or just from > some archive. Using ZeroConf I can get an image+vm, etc. I can use the > config command line handler to load the configuration from a local file > directory (I haven't tried this yet, but I assume no scheme results in a > MCDirectoryRepository). So far so good. > > > > The question is, what do I put in as argument to #repository: in my > baseline ? > > > > baseline1: spec > > <version: '1-baseline'> > > > > spec for: #common do: [ > > spec > > blessing: #baseline; > > repository: 'http://mc.stfx.eu/Neo'; > > package: 'Neo-Console-Core'; > > group: 'default' with: #('Neo-Console-Core'); > > group: 'Core' with: #('Neo-Console-Core') ] > > > > Obviously not a remote http repository. But also not an absolute path, > since I do not know where the user placed the files. Ideally, it should be > possible to leave out repository and let it resolve to wherever the > configuration was located. Can that be done ? Or is there another way to do > this ? > > > > I'm not sure it can be done that way. > > > > Loading from a local repository can be done on the command line, by > using Metacello directly with a local path. This is what I do for projects > where the repository is cloned before the start of Pharo. > > How do you do that ? > > "Loading from a local repository can be done on the command line, by using > Metacello directly with a local path." > > Suppose I have a repository that contains both my Configuration and my > code (except for the dependencies), let's say at $WORKDIR/repository. > > $ pharo Pharo.image config $WORKDIR/repository/ ConfigurationOfMyProject > --install > $ pharo Pharo.image eval --save Metacello new configuration: \'MyProject\'\; repository: \'$WORKDIR/repository\'\; load (Not sure about the way to designate just a file repository for Metacello) > > How do I tell ConfigurationOfMyProject to load its code from > $WORKDIR/repository ? Maybe this is really simple and I just don't see it ? > Maybe there is a simpler way than the one I use. Thierry > > > Thierry > > > > > > > > > > Thanks, > > > > Sven > > > > > > > > > > >

Nov. 17, 2015

Re: [Pharo-dev] A Question about Metacello's #repository:

by Sven Van Caekenberghe

> On 17 Nov 2015, at 15:30, Thierry Goubier <thierry.goubier(a)gmail.com> wrote: > > > > 2015-11-17 15:12 GMT+01:00 Sven Van Caekenberghe <sven(a)stfx.eu>: > Hi, > > I have a question about the way repositories are specified in Metacello (i.e. by means on #repository:). > > Say I use FileTree where the files come from a classic VCS or just from some archive. Using ZeroConf I can get an image+vm, etc. I can use the config command line handler to load the configuration from a local file directory (I haven't tried this yet, but I assume no scheme results in a MCDirectoryRepository). So far so good. > > The question is, what do I put in as argument to #repository: in my baseline ? > > baseline1: spec > <version: '1-baseline'> > > spec for: #common do: [ > spec > blessing: #baseline; > repository: 'http://mc.stfx.eu/Neo'; > package: 'Neo-Console-Core'; > group: 'default' with: #('Neo-Console-Core'); > group: 'Core' with: #('Neo-Console-Core') ] > > Obviously not a remote http repository. But also not an absolute path, since I do not know where the user placed the files. Ideally, it should be possible to leave out repository and let it resolve to wherever the configuration was located. Can that be done ? Or is there another way to do this ? > > I'm not sure it can be done that way. > > Loading from a local repository can be done on the command line, by using Metacello directly with a local path. This is what I do for projects where the repository is cloned before the start of Pharo. How do you do that ? "Loading from a local repository can be done on the command line, by using Metacello directly with a local path." Suppose I have a repository that contains both my Configuration and my code (except for the dependencies), let's say at $WORKDIR/repository. $ pharo Pharo.image config $WORKDIR/repository/ ConfigurationOfMyProject --install How do I tell ConfigurationOfMyProject to load its code from $WORKDIR/repository ? Maybe this is really simple and I just don't see it ? > Thierry > > > > > Thanks, > > Sven > > > >

Nov. 17, 2015

Re: [Pharo-dev] A Question about Metacello's #repository:

by Thierry Goubier

2015-11-17 15:12 GMT+01:00 Sven Van Caekenberghe <sven(a)stfx.eu>: > Hi, > > I have a question about the way repositories are specified in Metacello > (i.e. by means on #repository:). > > Say I use FileTree where the files come from a classic VCS or just from > some archive. Using ZeroConf I can get an image+vm, etc. I can use the > config command line handler to load the configuration from a local file > directory (I haven't tried this yet, but I assume no scheme results in a > MCDirectoryRepository). So far so good. > > The question is, what do I put in as argument to #repository: in my > baseline ? > > baseline1: spec > <version: '1-baseline'> > > spec for: #common do: [ > spec > blessing: #baseline; > repository: 'http://mc.stfx.eu/Neo'; > package: 'Neo-Console-Core'; > group: 'default' with: #('Neo-Console-Core'); > group: 'Core' with: #('Neo-Console-Core') ] > > Obviously not a remote http repository. But also not an absolute path, > since I do not know where the user placed the files. Ideally, it should be > possible to leave out repository and let it resolve to wherever the > configuration was located. Can that be done ? Or is there another way to do > this ? > I'm not sure it can be done that way. Loading from a local repository can be done on the command line, by using Metacello directly with a local path. This is what I do for projects where the repository is cloned before the start of Pharo. Thierry > > Thanks, > > Sven > > > >

Nov. 17, 2015

A Question about Metacello's #repository:

by Sven Van Caekenberghe

Hi, I have a question about the way repositories are specified in Metacello (i.e. by means on #repository:). Say I use FileTree where the files come from a classic VCS or just from some archive. Using ZeroConf I can get an image+vm, etc. I can use the config command line handler to load the configuration from a local file directory (I haven't tried this yet, but I assume no scheme results in a MCDirectoryRepository). So far so good. The question is, what do I put in as argument to #repository: in my baseline ? baseline1: spec <version: '1-baseline'> spec for: #common do: [ spec blessing: #baseline; repository: 'http://mc.stfx.eu/Neo'; package: 'Neo-Console-Core'; group: 'default' with: #('Neo-Console-Core'); group: 'Core' with: #('Neo-Console-Core') ] Obviously not a remote http repository. But also not an absolute path, since I do not know where the user placed the files. Ideally, it should be possible to leave out repository and let it resolve to wherever the configuration was located. Can that be done ? Or is there another way to do this ? Thanks, Sven

Nov. 17, 2015

[pharo-project/pharo-core]

by GitHub

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

Nov. 17, 2015

[pharo-project/pharo-core] f3add1: 50448

by GitHub

Branch: refs/heads/5.0 Home: https://github.com/pharo-project/pharo-core Commit: f3add10a5a59da648e8e7147be3b37787c686890 https://github.com/pharo-project/pharo-core/commit/f3add10a5a59da648e8e7147… Author: Jenkins Build Server <board(a)pharo-project.org> Date: 2015-11-17 (Tue, 17 Nov 2015) Changed paths: M ConfigurationOfFastTable.package/ConfigurationOfFastTable.class/instance/symbolic versions/stable_.st A ConfigurationOfFastTable.package/ConfigurationOfFastTable.class/instance/versions/version080_.st A Morphic-Widgets-FastTable.package/FTAllItemsStrategy.class/README.md A Morphic-Widgets-FastTable.package/FTAllItemsStrategy.class/definition.st A Morphic-Widgets-FastTable.package/FTAllItemsStrategy.class/instance/action/filter.st A Morphic-Widgets-FastTable.package/FTAllItemsStrategy.class/instance/action/initializeNewSearch.st A Morphic-Widgets-FastTable.package/FTAllItemsStrategy.class/instance/action/matchingFilter_.st A Morphic-Widgets-FastTable.package/FTAllItemsStrategy.class/instance/action/newStaticItemFrom_.st A Morphic-Widgets-FastTable.package/FTAllItemsStrategy.class/instance/action/optionalOperations.st A Morphic-Widgets-FastTable.package/FTAllItemsStrategy.class/instance/action/realSearch.st A Morphic-Widgets-FastTable.package/FTAllItemsStrategy.class/instance/action/searchIn_.st A Morphic-Widgets-FastTable.package/FTAllItemsStrategy.class/instance/action/unexpandAllChildOf_.st A Morphic-Widgets-FastTable.package/FTBasicItem.class/README.md A Morphic-Widgets-FastTable.package/FTBasicItem.class/class/expanding-collapsing/expandedForm.st A Morphic-Widgets-FastTable.package/FTBasicItem.class/class/expanding-collapsing/unexpandedForm.st A Morphic-Widgets-FastTable.package/FTBasicItem.class/definition.st A Morphic-Widgets-FastTable.package/FTBasicItem.class/instance/accessing/calculateChildren.st A Morphic-Widgets-FastTable.package/FTBasicItem.class/instance/accessing/children.st A Morphic-Widgets-FastTable.package/FTBasicItem.class/instance/accessing/data.st A Morphic-Widgets-FastTable.package/FTBasicItem.class/instance/accessing/data_.st A Morphic-Widgets-FastTable.package/FTBasicItem.class/instance/accessing/depth.st A Morphic-Widgets-FastTable.package/FTBasicItem.class/instance/accessing/depth_.st A Morphic-Widgets-FastTable.package/FTBasicItem.class/instance/accessing/expand.st A Morphic-Widgets-FastTable.package/FTBasicItem.class/instance/accessing/expandAndRefresh.st A Morphic-Widgets-FastTable.package/FTBasicItem.class/instance/accessing/isExpanded.st A Morphic-Widgets-FastTable.package/FTBasicItem.class/instance/accessing/isRootItem.st A Morphic-Widgets-FastTable.package/FTBasicItem.class/instance/accessing/unexpand.st A Morphic-Widgets-FastTable.package/FTBasicItem.class/instance/accessing/unexpandAndRefresh.st A Morphic-Widgets-FastTable.package/FTBasicItem.class/instance/button/expandedButton.st A Morphic-Widgets-FastTable.package/FTBasicItem.class/instance/button/generateButton.st A Morphic-Widgets-FastTable.package/FTBasicItem.class/instance/button/unexpandedButton.st A Morphic-Widgets-FastTable.package/FTBasicItem.class/instance/converting/asString.st A Morphic-Widgets-FastTable.package/FTBasicItem.class/instance/expanding-collapsing/expandAll.st A Morphic-Widgets-FastTable.package/FTBasicItem.class/instance/expanding-collapsing/expandAllTo_.st A Morphic-Widgets-FastTable.package/FTBasicItem.class/instance/initialization/initialize.st A Morphic-Widgets-FastTable.package/FTBasicItem.class/instance/printing/printOn_.st A Morphic-Widgets-FastTable.package/FTBasicItem.class/instance/updating/refreshTable.st A Morphic-Widgets-FastTable.package/FTBasicItem.class/instance/updating/updateData.st A Morphic-Widgets-FastTable.package/FTDataSource.class/instance/accessing/realElementAt_.st A Morphic-Widgets-FastTable.package/FTDataSource.class/instance/private/readyToBeDisplayed.st R Morphic-Widgets-FastTable.package/FTExamples.class/class/examples/exampleListWithExplicitFilter.st R Morphic-Widgets-FastTable.package/FTExamples.class/class/examples/exampleListWithExplicitFilterAndActionButton.st R Morphic-Widgets-FastTable.package/FTExamples.class/class/examples/exampleListWithExplicitFilterAndActionButton2.st R Morphic-Widgets-FastTable.package/FTExamples.class/class/examples/exampleListWithExplicitSearch.st A Morphic-Widgets-FastTable.package/FTExamples.class/class/function examples/exampleListWithExplicitFilter.st A Morphic-Widgets-FastTable.package/FTExamples.class/class/function examples/exampleListWithExplicitFilterAndActionButton.st A Morphic-Widgets-FastTable.package/FTExamples.class/class/function examples/exampleListWithExplicitFilterAndActionButton2.st A Morphic-Widgets-FastTable.package/FTExamples.class/class/function examples/exampleListWithExplicitSearch.st A Morphic-Widgets-FastTable.package/FTExamples.class/class/tree examples/exampleTree1.st A Morphic-Widgets-FastTable.package/FTExamples.class/class/tree examples/exampleTree2.st A Morphic-Widgets-FastTable.package/FTExamples.class/class/tree examples/exampleTreeWithAllItemsFilter.st A Morphic-Widgets-FastTable.package/FTExamples.class/class/tree examples/exampleTreeWithAllItemsSearch.st A Morphic-Widgets-FastTable.package/FTExamples.class/class/tree examples/exampleTreeWithRootFilter.st A Morphic-Widgets-FastTable.package/FTExamples.class/class/tree examples/exampleTreeWithRootSearch.st A Morphic-Widgets-FastTable.package/FTExamples.class/class/tree examples/exampleTreeWithVisibleItemsFilter.st A Morphic-Widgets-FastTable.package/FTExamples.class/class/tree examples/exampleTreeWithVisibleItemsSearch.st A Morphic-Widgets-FastTable.package/FTExamples.class/class/tree examples/exampleexampleOneSubLevelTree.st M Morphic-Widgets-FastTable.package/FTFilterFunction.class/instance/updating/filter.st M Morphic-Widgets-FastTable.package/FTFilterFunctionWithAction.class/instance/execute/execute.st A Morphic-Widgets-FastTable.package/FTRootItem.class/README.md A Morphic-Widgets-FastTable.package/FTRootItem.class/definition.st A Morphic-Widgets-FastTable.package/FTRootItem.class/instance/accessing/children.st A Morphic-Widgets-FastTable.package/FTRootItem.class/instance/accessing/children_.st A Morphic-Widgets-FastTable.package/FTRootItem.class/instance/accessing/dataSource_.st A Morphic-Widgets-FastTable.package/FTRootItem.class/instance/accessing/data_.st A Morphic-Widgets-FastTable.package/FTRootItem.class/instance/accessing/isExpanded.st A Morphic-Widgets-FastTable.package/FTRootItem.class/instance/expanding-collapsing/expandAll.st A Morphic-Widgets-FastTable.package/FTRootItem.class/instance/expanding-collapsing/expandAllTo_.st A Morphic-Widgets-FastTable.package/FTRootItem.class/instance/updating/updateData.st A Morphic-Widgets-FastTable.package/FTRootItemsStrategy.class/README.md A Morphic-Widgets-FastTable.package/FTRootItemsStrategy.class/definition.st A Morphic-Widgets-FastTable.package/FTRootItemsStrategy.class/instance/action/filter.st A Morphic-Widgets-FastTable.package/FTRootItemsStrategy.class/instance/action/realSearch.st A Morphic-Widgets-FastTable.package/FTSearchFunction.class/instance/accessing/realSearch.st R Morphic-Widgets-FastTable.package/FTSearchFunction.class/instance/accessing/search.st M Morphic-Widgets-FastTable.package/FTSearchFunction.class/instance/accessing/searchFor_.st A Morphic-Widgets-FastTable.package/FTStaticBasicItem.class/README.md A Morphic-Widgets-FastTable.package/FTStaticBasicItem.class/definition.st A Morphic-Widgets-FastTable.package/FTStaticBasicItem.class/instance/accessing/calculateChildren.st A Morphic-Widgets-FastTable.package/FTStaticBasicItem.class/instance/accessing/children.st A Morphic-Widgets-FastTable.package/FTStaticBasicItem.class/instance/accessing/children_.st A Morphic-Widgets-FastTable.package/FTStaticBasicItem.class/instance/updating/dataSource_.st A Morphic-Widgets-FastTable.package/FTStaticBasicItem.class/instance/updating/updateData.st M Morphic-Widgets-FastTable.package/FTTableContainerMorph.class/instance/drawing/drawOn_.st M Morphic-Widgets-FastTable.package/FTTableMorph.class/instance/event handling/keyStrokeArrowDown_.st M Morphic-Widgets-FastTable.package/FTTableMorph.class/instance/event handling/keyStrokeArrowUp_.st M Morphic-Widgets-FastTable.package/FTTableMorph.class/instance/menu/showMenuForRowIndex_columnIndex_.st M Morphic-Widgets-FastTable.package/FTTableMorph.class/instance/private/isVerticalScrollBarVisible.st M Morphic-Widgets-FastTable.package/FTTableMorph.class/instance/private/recalculateVerticalScrollBarVisibilityIfHidden_.st M Morphic-Widgets-FastTable.package/FTTableMorph.class/instance/private/resizeVerticalScrollBar.st M Morphic-Widgets-FastTable.package/FTTableMorph.class/instance/private/rowIndexToVerticalScrollBarValue_.st M Morphic-Widgets-FastTable.package/FTTableMorph.class/instance/private/selectRowIndex_event_.st M Morphic-Widgets-FastTable.package/FTTableMorph.class/instance/private/verticalScrollBarValue_.st A Morphic-Widgets-FastTable.package/FTTableMorph.class/instance/updating/themeChanged.st M Morphic-Widgets-FastTable.package/FTTreeDataSource.class/README.md A Morphic-Widgets-FastTable.package/FTTreeDataSource.class/class/accessing/defaultSearchStrategies.st A Morphic-Widgets-FastTable.package/FTTreeDataSource.class/class/accessing/rootItemFor_.st A Morphic-Widgets-FastTable.package/FTTreeDataSource.class/class/accessing/searchStrategies.st A Morphic-Widgets-FastTable.package/FTTreeDataSource.class/class/accessing/searchStrategies_.st A Morphic-Widgets-FastTable.package/FTTreeDataSource.class/class/class initialization/initialize.st A Morphic-Widgets-FastTable.package/FTTreeDataSource.class/class/instance creation/root_children_.st M Morphic-Widgets-FastTable.package/FTTreeDataSource.class/class/instance creation/roots_children_.st M Morphic-Widgets-FastTable.package/FTTreeDataSource.class/definition.st A Morphic-Widgets-FastTable.package/FTTreeDataSource.class/instance/accessing/buttonFor_.st R Morphic-Widgets-FastTable.package/FTTreeDataSource.class/instance/accessing/canDisplayChildrenOf_.st M Morphic-Widgets-FastTable.package/FTTreeDataSource.class/instance/accessing/cellColumn_row_.st M Morphic-Widgets-FastTable.package/FTTreeDataSource.class/instance/accessing/elementAt_.st R Morphic-Widgets-FastTable.package/FTTreeDataSource.class/instance/accessing/items.st R Morphic-Widgets-FastTable.package/FTTreeDataSource.class/instance/accessing/items_.st A Morphic-Widgets-FastTable.package/FTTreeDataSource.class/instance/accessing/maxDepth.st M Morphic-Widgets-FastTable.package/FTTreeDataSource.class/instance/accessing/newDataSourceMatching_.st M Morphic-Widgets-FastTable.package/FTTreeDataSource.class/instance/accessing/numberOfRows.st A Morphic-Widgets-FastTable.package/FTTreeDataSource.class/instance/accessing/realElementAt_.st A Morphic-Widgets-FastTable.package/FTTreeDataSource.class/instance/accessing/rootItem.st A Morphic-Widgets-FastTable.package/FTTreeDataSource.class/instance/accessing/rootItem_.st A Morphic-Widgets-FastTable.package/FTTreeDataSource.class/instance/accessing/rootsItems.st A Morphic-Widgets-FastTable.package/FTTreeDataSource.class/instance/accessing/searchStrategy.st A Morphic-Widgets-FastTable.package/FTTreeDataSource.class/instance/accessing/searchStrategy_.st M Morphic-Widgets-FastTable.package/FTTreeDataSource.class/instance/accessing/searchText_.st A Morphic-Widgets-FastTable.package/FTTreeDataSource.class/instance/expanding-collapsing/expandAll.st A Morphic-Widgets-FastTable.package/FTTreeDataSource.class/instance/expanding-collapsing/expandAllTo_.st A Morphic-Widgets-FastTable.package/FTTreeDataSource.class/instance/expanding-collapsing/expandRoots.st A Morphic-Widgets-FastTable.package/FTTreeDataSource.class/instance/testing/canDisplayChildrenOf_.st A Morphic-Widgets-FastTable.package/FTTreeDataSource.class/instance/updating/addSourceToRootItem.st A Morphic-Widgets-FastTable.package/FTTreeDataSource.class/instance/updating/updateData.st A Morphic-Widgets-FastTable.package/FTTreeFunctionStrategy.class/README.md A Morphic-Widgets-FastTable.package/FTTreeFunctionStrategy.class/class/instance-creation/filterWith_dataSource_.st A Morphic-Widgets-FastTable.package/FTTreeFunctionStrategy.class/class/instance-creation/searchWith_dataSource_.st A Morphic-Widgets-FastTable.package/FTTreeFunctionStrategy.class/definition.st A Morphic-Widgets-FastTable.package/FTTreeFunctionStrategy.class/instance/accessing/dataSource_.st A Morphic-Widgets-FastTable.package/FTTreeFunctionStrategy.class/instance/accessing/isMatching_.st A Morphic-Widgets-FastTable.package/FTTreeFunctionStrategy.class/instance/accessing/pattern_.st A Morphic-Widgets-FastTable.package/FTTreeFunctionStrategy.class/instance/action/filter.st A Morphic-Widgets-FastTable.package/FTTreeFunctionStrategy.class/instance/action/optionalOperations.st A Morphic-Widgets-FastTable.package/FTTreeFunctionStrategy.class/instance/action/realSearch.st A Morphic-Widgets-FastTable.package/FTTreeFunctionStrategy.class/instance/action/search.st M Morphic-Widgets-FastTable.package/FTTreeItem.class/README.md R Morphic-Widgets-FastTable.package/FTTreeItem.class/class/expanding-collapsing/expandedForm.st R Morphic-Widgets-FastTable.package/FTTreeItem.class/class/expanding-collapsing/unexpandedForm.st R Morphic-Widgets-FastTable.package/FTTreeItem.class/class/instance creation/data_from_.st A Morphic-Widgets-FastTable.package/FTTreeItem.class/class/instance-creation/data_from_.st M Morphic-Widgets-FastTable.package/FTTreeItem.class/definition.st R Morphic-Widgets-FastTable.package/FTTreeItem.class/instance/accessing/calculateChildren.st M Morphic-Widgets-FastTable.package/FTTreeItem.class/instance/accessing/children.st R Morphic-Widgets-FastTable.package/FTTreeItem.class/instance/accessing/data.st M Morphic-Widgets-FastTable.package/FTTreeItem.class/instance/accessing/data_.st R Morphic-Widgets-FastTable.package/FTTreeItem.class/instance/accessing/depth.st R Morphic-Widgets-FastTable.package/FTTreeItem.class/instance/accessing/depth_.st R Morphic-Widgets-FastTable.package/FTTreeItem.class/instance/accessing/expand.st R Morphic-Widgets-FastTable.package/FTTreeItem.class/instance/accessing/isExpanded.st A Morphic-Widgets-FastTable.package/FTTreeItem.class/instance/accessing/numberOfChildren.st M Morphic-Widgets-FastTable.package/FTTreeItem.class/instance/accessing/numberOfVisibleChildren.st R Morphic-Widgets-FastTable.package/FTTreeItem.class/instance/accessing/unexpand.st R Morphic-Widgets-FastTable.package/FTTreeItem.class/instance/button/expandedButton.st R Morphic-Widgets-FastTable.package/FTTreeItem.class/instance/button/generateButton.st R Morphic-Widgets-FastTable.package/FTTreeItem.class/instance/button/unexpandedButton.st R Morphic-Widgets-FastTable.package/FTTreeItem.class/instance/converting/asString.st A Morphic-Widgets-FastTable.package/FTTreeItem.class/instance/expanding-collapsing/expandAll.st A Morphic-Widgets-FastTable.package/FTTreeItem.class/instance/expanding-collapsing/expandAllTo_.st R Morphic-Widgets-FastTable.package/FTTreeItem.class/instance/initialization/initialize.st R Morphic-Widgets-FastTable.package/FTTreeItem.class/instance/printing/printOn_.st A Morphic-Widgets-FastTable.package/FTTreeItem.class/instance/testing/hasChildren.st A Morphic-Widgets-FastTable.package/FTTreeItem.class/instance/updating/isExpanded.st R Morphic-Widgets-FastTable.package/FTTreeItem.class/instance/updating/refreshTable.st A Morphic-Widgets-FastTable.package/FTTreeItem.class/instance/updating/updateData.st A Morphic-Widgets-FastTable.package/FTVisibleItemsStrategy.class/README.md A Morphic-Widgets-FastTable.package/FTVisibleItemsStrategy.class/definition.st A Morphic-Widgets-FastTable.package/FTVisibleItemsStrategy.class/instance/action/filter.st A Morphic-Widgets-FastTable.package/FTVisibleItemsStrategy.class/instance/action/matchingFilter_.st A Morphic-Widgets-FastTable.package/FTVisibleItemsStrategy.class/instance/action/newStaticItemFrom_.st A Morphic-Widgets-FastTable.package/FTVisibleItemsStrategy.class/instance/action/realSearch.st M Refactoring-Critics.package/RBVariableNotDefinedRule.class/instance/running/basicCheck_.st M Refactoring-Critics.package/RBVariableNotDefinedRule.class/instance/running/checkMethod_.st R ScriptLoader50.package/ScriptLoader.class/instance/pharo - scripts/script50447.st A ScriptLoader50.package/ScriptLoader.class/instance/pharo - scripts/script50448.st R ScriptLoader50.package/ScriptLoader.class/instance/pharo - updates/update50447.st A ScriptLoader50.package/ScriptLoader.class/instance/pharo - updates/update50448.st M ScriptLoader50.package/ScriptLoader.class/instance/public/commentForCurrentUpdate.st Log Message: ----------- 50448 17029 New FastTable configuration (see description) https://pharo.fogbugz.com/f/cases/17029 17026 Code critique should ignore meta link globals https://pharo.fogbugz.com/f/cases/17026 http://files.pharo.org/image/50/50448.zip

Nov. 17, 2015

Re: [Pharo-dev] Opal problem causing crash but old compiler works [WAS] Re: [Vm-dev] Debugging VM crash, INVALID RECEIVER / a(n) bad class ??

by Mariano Martinez Peck

Hi Nicolai, Thanks for sharing an updated cs. I can confirm that this version DOES work and fixes my VM crash as well. I have updated https://pharo.fogbugz.com/f/cases/13854 Thanks Nicolai!! On Tue, Nov 17, 2015 at 6:36 AM, Nicolai Hess <nicolaihess(a)gmail.com> wrote: > > > 2015-11-17 1:35 GMT+01:00 Mariano Martinez Peck <marianopeck(a)gmail.com>: > >> >> >> On Mon, Nov 16, 2015 at 8:41 PM, Nicolai Hess <nicolaihess(a)gmail.com> >> wrote: >> >>> >>> >>> 2015-11-16 22:16 GMT+01:00 Mariano Martinez Peck <marianopeck(a)gmail.com> >>> : >>> >>>> >>>> >>>> On Mon, Nov 16, 2015 at 5:37 PM, Mariano Martinez Peck < >>>> marianopeck(a)gmail.com> wrote: >>>> >>>>> Hi guys, >>>>> >>>>> So I found out the exact method that is causing the VM crash. If I >>>>> compile the method with Opal, it crashes the VM when I execute the code >>>>> that use that method. If I compile it with old compiler, the code does work >>>>> correctly. >>>>> >>>>> I tried comparing both compiled methods compiled from both Compilers >>>>> and I cannot see real differences. They both seem to have similar (same?) >>>>> bytecodes, literals, decompiled string, etc. The only difference I see is >>>>> in #frameSize (old compiler one is 16 while Opal one is 56). >>>>> Any idea what else can I check/compare? >>>>> >>>>> I also tried in Pharo 5.0 but same results. >>>>> >>>>> Thanks in advance, >>>>> >>>>> >>>> OK, it seems my issue may be related to: >>>> https://pharo.fogbugz.com/f/cases/13854/frameSize-calculated-wrongly-for-li… >>>> But the attached cs in there does NOT fixes mine. >>>> >>> >>> You need to: >>> switch to old compiler in settings >>> >> >> I was doing that via >> >> Smalltalk compilerClass: XXX. >> >> (I didn't know there was setting). >> >> >>> load change set fix_closure_stack_frame_size_computation.1.cs (not >>> fix_closure_stack_frame_size_computation.2.cs!) >>> switch back to opal >>> recompile all >>> >> >> Yes, I did that. >> >> >>> But the change set is old, and does not work for recent images (not even >>> for pharo 4.0 release), because there are many >>> changes for the bytecodegenerator. >>> >> >> Uhhhh. Yes, I saw some. For example, i changed from >> IRAbstractBytecodeGenerator to IRBytecodeGenerator >> >> Do you think we can re-create your fix with latest 4.0 or 5.0 so that I >> can see if the issue was the same? >> > > > you can try this one, (I just changed all the code that makes this run for > pharo 4, but I need more time to check if all of > this code is necessary - and correct) : > fix_closure_stack_size_4.0.cs (tested with Pharo 4.0 40624). > > >> >> >> >>> >>> >>> >>>> I can confirm it's a problem with the number of temp vars defined. As >>>> soon as I remove any of the temp vars, it works again. >>>> >>>> The method is this (a bit ugly, yes) pasted below. >>>> But the key point is: *As soon as I remove ANY tempVar the code >>>> starts to work again.* >>>> >>>> Any clues? >>>> >>>> createZacksAccountingRulesFromTable: t1 >>>> | t2 t3 t4 | >>>> t2 := FaUserContextInformation current dbAccessor >>>> db: 'zacks' >>>> getTableAccessorOn: t1 >>>> indexOn: nil. >>>> t3 := OrderedCollection new. >>>> t4 := FaFileTranscript named: 'create-zacks-rules.txt'. >>>> FaApplicationDB session >>>> inUnitOfWorkDo: [:t5 | >>>> | t6 | >>>> t6 := FaUserContextInformation current userDisplayName. >>>> t2 >>>> doWithRowDictionaries: [:t7 | (t7 at: 'label') >>>> == FaNullDatum instance >>>> ifFalse: [{{'Annual'. 'ZACKS_A_'}. {'Quarterly'. 'ZACKS_Q_'}} >>>> do: [:t8 | >>>> *| t9 t10 t11 t12 t13 t14 t15 t16 t17 t18 |* >>>> t4 crLog: t8 first. >>>> t9 := t8 at: 1. >>>> t16 := t7 at: 'label'. >>>> t18 := t7 at: 'TTM'. >>>> t10 := t7 at: 'quuveKeyEquivalent'. >>>> t12 := (t7 at: 'zacksKey') asUppercase. >>>> t13 := 'ZACKS_' , t12. >>>> t14 := 'ZACKS_Q_' , t12. >>>> t15 := 'ZACKS_A_' , t12. >>>> t11 := (t8 at: 2) >>>> , t12. >>>> t17 := FaAccountingRule new label: t16; >>>> selector: t13 asSymbol; >>>> context: 'Zacks' , t9 , 'Override'; >>>> argumentTypesSpecification: '{FaProcessorProxy}'; >>>> returnTypeSpecification: 'FaDatedFnSeries'; >>>> isReturnValueConstant: true; >>>> action: (self >>>> getZacksDataAtScriptForCache: t11 >>>> forZacksKey: t11 >>>> fromSet: t9); >>>> comment: (nil >>>> ifNil: [t16]); >>>> isSensitive: true; >>>> definedBy: t6; >>>> ownedBy: t6; >>>> lastEditBy: t6; >>>> yourself. >>>> t5 register: t17. >>>> t3 add: 'Successfully added ' , t17 selector , '/' , t17 context. >>>> t4 crLog: t3 last]]]]. >>>> ^ t3 >>>> >>>> >>>> >>>> >>>> >>>> >>>> >>>> >>>> >>>> >>>> >>>>> >>>>> On Mon, Nov 16, 2015 at 2:48 PM, Mariano Martinez Peck < >>>>> marianopeck(a)gmail.com> wrote: >>>>> >>>>>> >>>>>> >>>>>> On Mon, Nov 16, 2015 at 2:48 PM, Mariano Martinez Peck < >>>>>> marianopeck(a)gmail.com> wrote: >>>>>> >>>>>>> >>>>>>> >>>>>>> On Mon, Nov 16, 2015 at 1:07 PM, Nicolai Hess <nicolaihess(a)gmail.com >>>>>>> > wrote: >>>>>>> >>>>>>>> >>>>>>>> Hi Mariano, if you know the method that may cause this crash can >>>>>>>> you check >>>>>>>> if it works if you recompile this method with the old compiler, >>>>>>>> maybe this issue is responsible - wrong stack frame size : >>>>>>>> 13854 >>>>>>>> <https://pharo.fogbugz.com/f/cases/13854/frameSize-calculated-wrongly-for-li…> >>>>>>>> frameSize calculated wrongly for #lineSegmentsDo: >>>>>>>> >>>>>>>> >>>>>>> Hi Nicolai, >>>>>>> >>>>>>> Thanks for the pointer. I tried with your fix and I still get the >>>>>>> same results (even after recompiling). >>>>>>> However...if I swap back to old Compiler rather than OpalCompiler >>>>>>> and I recompile everything, then I do not have anymore the crash. >>>>>>> So it's definitively something related to Opal compilation, and >>>>>>> probably, related to closures compilation. >>>>>>> I will see if I find other opal issues opened in 4.0. >>>>>>> >>>>>>> Thanks! >>>>>>> >>>>>>> >>>>>>>> >>>>>>>> >>>>>>>> 2015-11-16 17:00 GMT+01:00 Mariano Martinez Peck < >>>>>>>> marianopeck(a)gmail.com>: >>>>>>>> >>>>>>>>> >>>>>>>>> Hi guys, >>>>>>>>> >>>>>>>>> I am debugging a Pharo VM crash I am having and I cannot figure >>>>>>>>> out what it is exactly. I suspect it might be related to block closures >>>>>>>>> compilation but I am not sure. I have a reproducible crash test under Pharo >>>>>>>>> 4.0 and OSX. >>>>>>>>> >>>>>>>>> I built a VM in debug mode and I run it via gdb. This is the kind >>>>>>>>> of info I am able to see: >>>>>>>>> >>>>>>>>> *Program received signal SIGSEGV, Segmentation fault.* >>>>>>>>> *0x000ab66b in updatePointersInRangeFromto (memStart=924623948, >>>>>>>>> memEnd=928201420) at >>>>>>>>> /Users/mariano/Pharo/git/pharo-vm/src/vm/gcc3x-cointerp.c:40444* >>>>>>>>> *40444 && (((longAt(fieldOop)) & MarkBit) != 0)) {* >>>>>>>>> *(gdb) call printAllStacks()* >>>>>>>>> *Process 0x30e228c4 priority 40* >>>>>>>>> *0xbffb2e10 M FaAction class>block: 0x20ebb104: a(n) FaAction >>>>>>>>> class* >>>>>>>>> *0xbffb2e30 M BlockClosure(FaMemoryStoreSession)>register: >>>>>>>>> 0x371d0968: a(n) BlockClosure* >>>>>>>>> *0xbffb2e4c M INVALID RECEIVER>register: 0x371cfd54: a(n) bad >>>>>>>>> class* >>>>>>>>> >>>>>>>>> *(callerContextOrNil == (nilObject())) || >>>>>>>>> (isContext(callerContextOrNil)) 48626* >>>>>>>>> *0x3721e1f0 is not a context* >>>>>>>>> *0x371c34b4 is not a context* >>>>>>>>> >>>>>>>>> And another crash: >>>>>>>>> >>>>>>>>> *Program received signal SIGSEGV, Segmentation fault.* >>>>>>>>> *0x000ab66b in updatePointersInRangeFromto (memStart=924677864, >>>>>>>>> memEnd=928262300) at >>>>>>>>> /Users/mariano/Pharo/git/pharo-vm/src/vm/gcc3x-cointerp.c:40444* >>>>>>>>> *40444 && (((longAt(fieldOop)) & MarkBit) != 0)) {* >>>>>>>>> *(gdb) call printAllStacks()* >>>>>>>>> *Process 0x30e228c4 priority 40* >>>>>>>>> *0xbffb2de0 M INVALID RECEIVER>initialize 0x3722b9a0: a(n) bad >>>>>>>>> class* >>>>>>>>> *0xbffb2df8 M FaAction class(Behavior)>new 0x20ebb104: a(n) >>>>>>>>> FaAction class* >>>>>>>>> *0xbffb2e10 M FaAction class>block: 0x20ebb104: a(n) FaAction >>>>>>>>> class* >>>>>>>>> *0xbffb2e30 M INVALID RECEIVER>register: 0x371ddee0: a(n) bad >>>>>>>>> class* >>>>>>>>> *0xbffb2e4c M INVALID RECEIVER>register: 0x371dd328 is in old >>>>>>>>> space* >>>>>>>>> >>>>>>>>> *(callerContextOrNil == (nilObject())) || >>>>>>>>> (isContext(callerContextOrNil)) 48626* >>>>>>>>> *0x3722b750 is not a context* >>>>>>>>> *[New Thread 0x1b43 of process 5886]* >>>>>>>>> *[New Thread 0x1d2f of process 5886]* >>>>>>>>> *[New Thread 0x1f07 of process 5886]* >>>>>>>>> *[New Thread 0x1e07 of process 5886]* >>>>>>>>> >>>>>>>>> From what I can see in *#printActivationNameFor: aMethod >>>>>>>>> receiver: anObject isBlock: isBlock firstTemporary: maybeMessage* >>>>>>>>> It looks like if the memory address is not an OOP, nor forwarding >>>>>>>>> pointer nor... >>>>>>>>> >>>>>>>>> It also seems like the above stack I can get is not the real cause >>>>>>>>> but a side effect that makes GC to crash (#updatePointersInRangeFromto) >>>>>>>>> >>>>>>>>> As said, I have a way to reproduce the crash, and I already have >>>>>>>>> the VM compiled in debug and gdb running. I can also attach the full output >>>>>>>>> of the gdb stack. >>>>>>>>> >>>>>>>>> BTW.... in the output of the gdb I see lots of printings like: >>>>>>>>> >>>>>>>>> *(numStack + ReceiverIndex) < (lengthOf(theContext)) 45421* >>>>>>>>> *(ReceiverIndex + contextSize) < (lengthOfbaseHeaderformat(oop, >>>>>>>>> header2, fmt)) 40416* >>>>>>>>> >>>>>>>>> >>>>>>>>> Any pointer is appreciated. >>>>>>>>> >>>>>>>>> Thanks! >>>>>>>>> >>>>>>>>> -- >>>>>>>>> Mariano >>>>>>>>> http://marianopeck.wordpress.com >>>>>>>>> >>>>>>>>> >>>>>>>> >>>>>>>> >>>>>>> >>>>>>> >>>>>>> -- >>>>>>> Mariano >>>>>>> http://marianopeck.wordpress.com >>>>>>> >>>>>> >>>>>> >>>>>> >>>>>> -- >>>>>> Mariano >>>>>> http://marianopeck.wordpress.com >>>>>> >>>>> >>>>> >>>>> >>>>> -- >>>>> Mariano >>>>> http://marianopeck.wordpress.com >>>>> >>>> >>>> >>>> >>>> -- >>>> Mariano >>>> http://marianopeck.wordpress.com >>>> >>> >>> >> >> >> -- >> Mariano >> http://marianopeck.wordpress.com >> > > -- Mariano http://marianopeck.wordpress.com

Nov. 17, 2015