Ok, I see the problem now, although is not reproducible in the case of
ConfigurationOfMoose simply because of the load order I think. But, the bug
was clearly present.
I looked at your code and it fixed the problem. However, I now took a step
back and I realize that we are working on the wrong model :). All the
information we need is in the working copies, so I removed all intermediary
string-based collections and I am only using a collection of working
copies. It seems to work fine. I pushed it as stable. Could you have a look?
Cheers,
Doru
On Sat, Aug 10, 2013 at 6:14 PM, Stéphane Ducasse <stephane.ducasse(a)inria.fr
wrote:
Gofer new
smalltalkhubUser: 'girba' project: 'Snapshotcello';
package: 'ConfigurationOfSnapshotcello';
load.
ConfigurationOfSnapshotcello loadStable.
Snapshotcello new
configurationClass: ConfigurationOfMoose;
sourceVersion: #development;
repositoryFor: 'ConfigurationOfMooseAlgos'
==> 'http://www.smalltalkhub.com/mc/Moose/MooseAlgos/main/'
doru
I just checked because you put a doubt in my mind but I was right
repositoryFor: is for versioned package not package names
so the real case (in fact I put a transcript while snapshotting Moose) is
Snapshotcello new
configurationClass: ConfigurationOfMoose;
sourceVersion: #development;
repositoryFor: 'ConfigurationOfFame-TudorGirba.19'
With Snapshotcello 20
Snapshotcello new
configurationClass: ConfigurationOfFourDBrowser;
sourceVersion: #development;
repositoryFor: 'ConfigurationOfMooseAlgos-TudorGirba.34'
>>
'http://www.smalltalkhub.com/mc/Moose/Moose/main/'
With Snapshotcello 23 (my favorite solution but I need your look)
Snapshotcello new
configurationClass: ConfigurationOfFourDBrowser;
sourceVersion: #development;
repositoryFor: 'ConfigurationOfMooseAlgos-TudorGirba.34'
>>
'http://www.smalltalkhub.com/mc/Moose/MooseAlgos/main/'
With Snapshotcello21 (improving the match to cut at '-')
Snapshotcello new
configurationClass: ConfigurationOfFourDBrowser;
sourceVersion: #development;
repositoryFor: 'ConfigurationOfMooseAlgos-TudorGirba.34'
'http://www.smalltalkhub.com/mc/Moose/MooseAlgos/main/'
I was afraid that my changes was just fixing the consequences of a problem
in the method computePackagesAndRepositoriesFromConfiguration
but it is correct. What metacello gives us back is a correct list of pairs.
Stef
_______________________________________________
Moose-dev mailing list
Moose-dev(a)iam.unibe.ch
https://www.iam.unibe.ch/mailman/listinfo/moose-dev
--
www.tudorgirba.com
"Every thing has its own flow"