Eclipse Community Forums: Papyrus » Papyrus-0.9.0M7 has Dependency Cycle

Home » Modeling » Papyrus » Papyrus-0.9.0M7 has Dependency Cycle(org.eclipse.papyrus.infra.tools and org.eclipse.papyrus.infra.widgets has Dependency Cycle)

Show: Today's Messages :: Show Polls :: Message Navigator

Papyrus-0.9.0M7 has Dependency Cycle [message #871483]

Mon, 14 May 2012 06:48

MyungWoon Oh

Messages: 52
Registered: March 2012

Member

Thanks to the release of M7,
I could build nearly all of the Papyrus-0.9.0M7.

Just one file has all remained problems, that is org.eclipse.papyrus.infra.tools.databinding.MultipleObservableList.java.

That file has actually 3 errors.

#1. Dependency Cycle
MultipleObservableList.commit(AbstractEditor editor) needs AbstractEditor and ICommitListener of org.eclipse.papyrus.infra.widget plug-in, which leads to the Dependency Cycle between org.eclipse.papyrus.infra.tools and org.eclipse.papyrus.infra.widgets

#2. Import sentence is wrong
AggregatedObservable class is in org.eclipse.papyrus.infra.tools.databinding package,
not in org.eclipse.papyrus.infra.widgets.databinding as source says.

#3. Need to implement the inherited abstract method
Need to implement 2 abstract method aggregate(IObservable), hasDifferentValues()

Please fix it.. Smile

Attachment: Papyrus0.9.0M7-BuildError.png
(Size: 43.18KB, Downloaded 1422 times)

Report message to a moderator

Re: Papyrus-0.9.0M7 has Dependency Cycle [message #871991 is a reply to message #871483]

Tue, 15 May 2012 09:03

Camille Letavernier

Messages: 935
Registered: February 2011

Senior Member

This is surprising, as infra.tools doesn't depend on anything (Except the Papyrus log).

I cannot reproduce the error with my source installation.

By the way, MultipleObservableList should be excluded from the classpath (It is not included in the Papyrus build).

Camille Letavernier

Report message to a moderator

Re: Papyrus-0.9.0M7 has Dependency Cycle [message #872373 is a reply to message #871991]

Wed, 16 May 2012 01:45

MyungWoon Oh

Messages: 52
Registered: March 2012

Member

Camille Letavernier wrote on Tue, 15 May 2012 05:03

This is surprising, as infra.tools doesn't depend on anything (Except the Papyrus log).

I cannot reproduce the error with my source installation.

By the way, MultipleObservableList should be excluded from the classpath (It is not included in the Papyrus build).

Thank you for the quick reply.

The attached file shows what I got from the official Papyrus download site - 'www.eclipse.org/downloads/download.php?file=/modeling/mdt/papyrus/downloads/drops/0.9.0/S201205090534/Papyrus-Update-0.9.0_M7.zip'

index.php/fa/9739/0/

The top picture of Eclipse Editor shows the source of org.eclipse.papyrus.infra.tools.databinding.MultipleObservableList.java.
Line 20 to 22 shows it depends on infra.widgets.
(I just made single line comment on line 20 because 'AggregatedObservable' is not of org.eclipse.papyrus.infra.widgets.databinding as the source says,
but of org.eclipse.papyrus.infra.tools.databinding, actually.)

The middle picture of Windows Explorer shows the contents of infra.tools.source.jar.
You can see MultipleObservableList.java which is not shown in the bottom picture showing the contents of infra.tools class jar.

As you can see, the source jar doesn't seem to match well with class jar.

If your reply is right, then the released source jar may be wrong.

Actually I found other wrong sources during the building.

For example, org.eclipse.papyrus.infra.widgets.xtext.adapter.IXtextAdapter.java does not exist in source jar nor in class jar.
But files below still implements the IXtextAdapter.
- org.eclipse.papyrus.uml.properties.xtext.CollaborationUseAdapter.java
- org.eclipse.papyrus.uml.properties.xtext.ConnectionPointReferenceAdapter.java
- org.eclipse.papyrus.uml.properties.xtext.MessageAdapter.java
- org.eclipse.papyrus.uml.properties.xtext.ParameterAdapter.java
- org.eclipse.papyrus.uml.properties.xtext.PropertyAdapter.java
- org.eclipse.papyrus.uml.properties.xtext.StateAdapter.java
- org.eclipse.papyrus.uml.properties.xtext.TransitionAdapter.java

and in 48 Line of IXtextAdapter.java
'public Injector getInjector();' should be 'public Injector getInjector(EObject sourceModelObject);'

I fixed IXtextAdapter problem by finding that file from older version of Papyrus and added to the 0.9.0M7 with some modification above and package name.

As a result, I dare to guess that released source jar need to be modified to match well with class jar which runs well(And your source file may not be the same as the released source jar)
Moreover, the problems I wrote above do exist in 0.8.X Build, too.

So I appreciate it so much if you or Papyrus Team check the whole source jar released for the Papyrus user.

Best regards.

Attachment: MultipleObservableList.png
(Size: 112.47KB, Downloaded 1952 times)

Report message to a moderator

Re: Papyrus-0.9.0M7 has Dependency Cycle [message #872488 is a reply to message #872373]

Wed, 16 May 2012 08:07

Camille Letavernier

Messages: 935
Registered: February 2011

Senior Member

It seems the "import source" doesn't take into account the .classpath file which is actually used to build Papyrus. Not too surprising, as this file isn't included in the binary nor in the source build.

So, that's a bug in our source builds.

All the file you mention are actually excluded from our builds ; they aren't supposed to be compiled. You can simply ignore them (Build path -> Exclude)

Camille Letavernier

Report message to a moderator

Re: Papyrus-0.9.0M7 has Dependency Cycle [message #872945 is a reply to message #872488]

Thu, 17 May 2012 06:42

MyungWoon Oh

Messages: 52
Registered: March 2012

Member

Camille Letavernier wrote on Wed, 16 May 2012 04:07

It seems the "import source" doesn't take into account the .classpath file which is actually used to build Papyrus. Not too surprising, as this file isn't included in the binary nor in the source build.

So, that's a bug in our source builds.

All the file you mention are actually excluded from our builds ; they aren't supposed to be compiled. You can simply ignore them (Build path -> Exclude)

Thank you for the quick reply again. Smile

I just deleted the file 'org.eclipse.papyrus.infra.tools.databinding.MultipleObservableList.java' and fixed an import promblem in 'org.eclipse.papyrus.uml.alf.syntax.expressions.TemplateParameterSubstitution.java'.

Finally I got it worked. Laughing

But.. I think I find another runtime bug.
When I close the Eclipse after some model.di testing, Eclipse doesn't exit for a click.
I mean sometimes it exited after 2 clicks Exit(first click does nothing but producing error log), sometimes it doesn't exit at all producing error logs as attached below.
And I confirmed this bug happens on My Papyrus build as well as on the installed Papyrus.

Anyway, I appreciate your great help so much, Camille.
I just wait until the official Juno and Papyrus release.

Attachment: Eclipse-Papyrus-Exiting-Error.log
(Size: 668.66KB, Downloaded 277 times)

Report message to a moderator

Re: Papyrus-0.9.0M7 has Dependency Cycle [message #1718101 is a reply to message #872945]

Sat, 19 December 2015 16:21

wiem salah

Messages: 5
Registered: December 2015

Junior Member

For the purpose of an University project, i have to create an eclipse plugin that allow users to do certain operation in a Class Diagram. I try to get , to modify the source code of Papyrus plugin and customize it for particular objectifs. but i have a lot of problems:
* how to resolve the error when i try to get the source code with Git Repositories

* how run this source code

* how can i modify this source code

thanks you for your helps
looking forward to hearing from you

Attachment: papyruserror.PNG
(Size: 43.80KB, Downloaded 120 times)

Report message to a moderator

Re: Papyrus-0.9.0M7 has Dependency Cycle [message #1718112 is a reply to message #1718101]

Sat, 19 December 2015 20:12

Ed Willink

Messages: 6993
Registered: July 2009

Senior Member

Hi

Papyrus 0.9.0 was released in 2012, so you should NOT be using a
pre-release milestone build (M7).

If you want sources, you would be well advised to use the Mars release
or newer so that you get the benefits of GIT repos and OOMPH setups.

Papyrus has a very large number of plugins. If you are only interested
in one or two you may be better to
- install Mars
- install Papyrus
- import one or two selected plugins with source, optionally check these
projects out from GIT.

This can avoid struggling with the entire Papyrus environment.

Regards

Ed Willink

On 19/12/2015 16:21, wiem salah wrote:
> For the purpose of an University project, i have to create an eclipse plugin that allow users to do certain operation in a Class Diagram. I try to get , to modify the source code of Papyrus plugin and customize it for particular objectifs. but i have a lot of problems:
> * how to resolve the error when i try to get the source code with Git Repositories
>
> * how run this source code
>
> * how can i modify this source code
>
> thanks you for your helps
> looking forward to hearing from you
>

Report message to a moderator