anuragraghavan / franca

Automatically exported from code.google.com/p/franca
0 stars 0 forks source link

Remove mandatory JDT dependency from Franca #91

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
It should be possible to install Franca without JDT. This can be tested by 
building the RCP product and checking if JDT is there. Currently it is required.

These are the two known mandatory JDT dependencies:

1. Xtext runtime => JDT. See 
https://bugs.eclipse.org/bugs/show_bug.cgi?id=404322#c5 for instructions how to 
make this optional. It has to be checked what drawbacks this solution will 
induce.

2. org.franca.core.ui.addons. The wizard in this plug-in uses JDT specific 
stuff. The wizard contains a widget for selecting a package from a java 
project. This should be available only if JDT is installed, otherwise the 
widget should be a plain textbox. Reflection can be used to implement this.

Original issue reported on code.google.com by klaus.birken@gmail.com on 14 Feb 2014 at 3:36

GoogleCodeExporter commented 9 years ago
We are using approach 1. in our own CDT based product without any problems 
(Xtext 2.4.3). There is an issue introduced with Xtext 2.5, see 
https://bugs.eclipse.org/bugs/show_bug.cgi?id=424455

Original comment by ChristophKulla@gmail.com on 20 Feb 2014 at 7:04

GoogleCodeExporter commented 9 years ago
Merged on develop, final commit is 9e62203.

Original comment by klaus.birken@gmail.com on 1 Apr 2014 at 2:08

GoogleCodeExporter commented 9 years ago

Original comment by klaus.birken@gmail.com on 7 May 2014 at 4:11

GoogleCodeExporter commented 9 years ago
Reopen: classpath-imports didn't work anymore for Franca IDL and Deployment, 
even if JDT is available.

We prepared a fix for this, which is currently being tested. With this fix, 
Franca will detect if JDT is available in the IDE. If yes, classpath-imports 
will work as expected. If no, classpath-imports will be tagged with an error 
marker "Imported resource could not be found.".

Original comment by klaus.birken@gmail.com on 12 May 2014 at 8:42

GoogleCodeExporter commented 9 years ago
Implement with commit 7e5ebaf on develop.

Original comment by klaus.birken@gmail.com on 12 May 2014 at 9:12

GoogleCodeExporter commented 9 years ago
Pushed to master, will update 0.9.0.

Original comment by klaus.birken@gmail.com on 12 May 2014 at 9:21

GoogleCodeExporter commented 9 years ago

Original comment by klaus.birken@gmail.com on 25 Jan 2015 at 12:30