11 Oct
2010
11 Oct
'10
4:20 p.m.
On 11 oct. 2010, at 15:23, Alexandre Bergel wrote:
Hi Guillermo,
I checked version the 123 of Famix-Core. Why have you added the comment in the method?
FAMIXUnknownVariable>>container
self flag: #useless. "seems like this attribute is never set"
^ container
FAMIXUnknownVariable defines container:, which has many senders
Hi Alex
I think you took a look at a larger change, not the one from Guillermo. What you see is a
change which has been deleted for Moose 4.1. The comment is meant to say this attribute is
always nil and hence useless as the importer does not care about setting a container for
unknown variables. And since nobody ever complained about a container set to nil, I assume
nobody needs container on FamixUnknownVariable. We can always add it later.
Cheers,
Alexandre
On 11 Oct 2010, at 09:32, Simon Denier wrote:
--
Simon
Hi Guillermo
I saw your commit and especially your comment
"saving locally first, then remotelly, doesn't simply copy the changes like
Mercurial does..."
The way it works with Monticello, compared to Git/Mercurial (remember that Monticello was
conceived before/in parallel with such systems...)
- the save command in MC always creates a new version of your package in the repository
you selected
- saving a version on a remote repository also saves a local copy
- if you want to save a version first locally, then want to copy it later to another
repository, use the "copy" command from the MC browser
Alternatively, the Gofer API provides commands which are closes to Git/Mercurial
workflows: push/fetch etc.
See http://www.lukas-renggli.ch/blog/gofer
BTW, is the fix complete? Your package only provides a new method
FamixReference>>identityInstanceVariables
--
Simon
_______________________________________________
Moose-dev mailing list
Moose-dev(a)iam.unibe.ch
https://www.iam.unibe.ch/mailman/listinfo/moose-dev
--
_,.;:~^~:;._,.;:~^~:;._,.;:~^~:;._,.;:~^~:;._,.;:
Alexandre Bergel http://www.bergel.eu
^~:;._,.;:~^~:;._,.;:~^~:;._,.;:~^~:;._,.;:~^~:;.
_______________________________________________
Moose-dev mailing list
Moose-dev(a)iam.unibe.ch
https://www.iam.unibe.ch/mailman/listinfo/moose-dev