Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Moose crashes when importing Citezen (OrderedCollection(Object)>>doesNotUnderstand: #unsafeAdd:) #736

Closed
seandenigris opened this issue Aug 3, 2015 · 13 comments

Comments

@seandenigris
Copy link
Contributor

Originally reported on Google Code with ID 736

Describe the problem: what do you get? 
Message-not-understood: OrderedCollection(Object)>>doesNotUnderstand: #unsafeAdd:
what do you expect?
a moose model

How to reproduce the problem: step by step if necessary
- download moose 4.6-dev
- import citezen (www.squeaksource.com/citezen)
- import citezen packages in moose

Additional information: platform, context which may impact the problem
- Mac OS x 10.6.6

Please fill in the labels with the following information:
* Type-Defect

Reported by usman.bhatti on 2011-11-02 12:18:47

@seandenigris
Copy link
Contributor Author

Worked a bit to dig the problem...
appears when parsing package: Citezen-Stef-Scripts, class: CZLocalisation, method (class
side): initializeTypes
The problem occurs when opposite link is added...
this expression: "(new perform: selector)" in method "on: element update: selector
from: old to: new" in class "FMMultivalueLink" returns an OrderedCollection when it
should return a FMMultivalueLink
not knowing the exact structure of the importer, i stopped digging at this point ..


Reported by usman.bhatti on 2011-11-02 13:43:11

@seandenigris
Copy link
Contributor Author

What parts of Citezen should I load? Could you provide a Gofer script, please?

Reported by tudor.girba on 2011-11-16 08:33:05

@seandenigris
Copy link
Contributor Author

Reported by tudor.girba on 2011-11-16 08:39:53

@seandenigris
Copy link
Contributor Author

the package StefScript.
I have the impression that it may be linked with the shared pool used.

Check in the configurationOfCitezen for noweb.


Reported by stephane.ducasse on 2011-11-17 18:46:03

@seandenigris
Copy link
Contributor Author

Please provide a full script that can be easily reproduced.

Reported by tudor.girba on 2011-12-12 20:43:09

@seandenigris
Copy link
Contributor Author

I hope to get some times next week to have a look.
Else this is the citizen-StefScripts that cannot be loaded.

Reported by stephane.ducasse on 2011-12-12 22:53:22

@seandenigris
Copy link
Contributor Author

I'm checking it now and I think that this is related to a bug in pharo 1.3 (related
to Pool variable look up). 
I will check if I load the fix if it solves the problem.

Reported by stephane.ducasse on 2011-12-13 12:28:50

@seandenigris
Copy link
Contributor Author

Thanks!

Reported by tudor@tudorgirba.com on 2011-12-13 12:35:07

@seandenigris
Copy link
Contributor Author

I checked and loading 4899 does not help but the problem is really linked with the use
of Pool Variable.
I will have a look.


Reported by stephane.ducasse on 2011-12-13 12:50:00

@seandenigris
Copy link
Contributor Author

Stef, some tests are red and the errors are related to parsing pool variables:
http://ci.moosetechnology.org/job/moose-latest-dev/lastCompletedBuild/testReport/

Is it possible that the Pharo patch is still missing? Should I update the base Pharo
image?

Reported by tudor.girba on 2011-12-15 23:55:01

  • Labels added: Component-SmalltalkImporter, Milestone-4.6

@seandenigris
Copy link
Contributor Author

Reported by tudor.girba on 2012-02-11 08:19:11

  • Labels removed: Milestone-4.6

@seandenigris
Copy link
Contributor Author

The problem seems to be resolved now. The import of Citezen (all packages) now works
in Moose.

Reported by usman.bhatti on 2012-02-13 15:24:38

@seandenigris
Copy link
Contributor Author

Reported by tudor@tudorgirba.com on 2013-06-23 05:20:26

  • Status changed: Fixed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant