|Re: Papyrus-0.9.0M7 has Dependency Cycle [message #872373 is a reply to message #871991]
||Wed, 16 May 2012 01:45
| MyungWoon Oh
Registered: March 2012
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'
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.
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.
|Re: Papyrus-0.9.0M7 has Dependency Cycle [message #872488 is a reply to message #872373]
||Wed, 16 May 2012 08:07
| Camille Letavernier
Registered: February 2011
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)
Powered by FUDForum
. Page generated in 0.02364 seconds