Monticello remove methods

Previous Topic Next Topic
 
classic Classic list List threaded Threaded
4 messages Options
Reply | Threaded
Open this post in threaded view
|

Monticello remove methods

Stan Shepherd
Reply | Threaded
Open this post in threaded view
|

Re: Monticello remove methods

Stan Shepherd
Sorry, finger trouble there.
I have a refactoring as follows:

OR2CmdLintCommand subclass: #OR2CmdRefresh
        instanceVariableNames: ''
        classVariableNames: ''
        poolDictionaries: ''
        category: 'OB-Refactory-Lint'

methods:
#execute
#isEnabled etc

becomes:

OR2CmdLintCommand subclass: #OR2CmdRefresh
        uses: TGBCmdRefresh
        instanceVariableNames: ''
        classVariableNames: ''
        poolDictionaries: ''
        category: 'OBO2Merge'

methods: none

TGBCmdRefresh (trait)

methods:
#execute
#isEnabled etc

When I save to a Monticello repo, and import to a new image, TGBCmdRefresh still has the original methods. They are not in the class in the repository, when browsing that repository. Is this expected behaviour? Is it because of the change of package (and perhaps Issue 1760: Monticello enhancement: avoid removing a method when it does not belong to package anymore)?

How do I get the new version to import without the old methods?

Thanks,   ...Stan
Reply | Threaded
Open this post in threaded view
|

Re: [Pharo-project] Monticello remove methods

Alexandre Bergel
A merge should help no?

Alexandre


On 21 Jan 2010, at 11:12, Stan Shepherd wrote:

> How do I get the new version to import without the old methods?

--
_,.;:~^~:;._,.;:~^~:;._,.;:~^~:;._,.;:~^~:;._,.;:
Alexandre Bergel  http://www.bergel.eu
^~:;._,.;:~^~:;._,.;:~^~:;._,.;:~^~:;._,.;:~^~:;.






_______________________________________________
Pharo-project mailing list
[hidden email]
http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project
Reply | Threaded
Open this post in threaded view
|

Re: [Pharo-project] Monticello remove methods

Stan Shepherd
Merge appears to have the same result.

It's at http://www.squeaksource.com/OBO2Merge

...Stan