[mdt-papyrus.dev] RE : Compilation errors in the Papyrus main-line proje

[LETAVERNIER Camille <Camille.LETAVERNIER@xxxxxx>]

Hi Christian,


The answer is actually quite simple: it doesn't work.

Almost all Papyrus builds are red on hudson.

Papyrus 0.10/Kepler is still supposed to work on Java 5, except for the extra-plugins which may require Java 6.


Regards,
Camille
________________________________________
De : mdt-papyrus.dev-bounces@xxxxxxxxxxx [mdt-papyrus.dev-bounces@xxxxxxxxxxx] de la part de Christian W. Damus [give.a.damus@xxxxxxxxx]
Date d'envoi : jeudi 10 octobre 2013 00:22
À : Papyrus Project list
Objet : Re: [mdt-papyrus.dev] Compilation errors in the Papyrus main-line       projects

Hi, Ed,

Camille has already announced that Papyrus will require 6.0 for Luna, which is fine (and possibly motivated by that cross-project discussion).

The immediate problem is not a pedantic complaint, but actual compilation errors in my workspace that break stuff at run-time.  I don't know how this code compiles for the rest of the team, which is why I am asking for help to understand how to make it work.

I can fix the problems myself by pushing a commit that either

  * removes @Override on interface method implementations or
  * updates the BREE of all plug-ins

but in the first case somebody's workbench will just continue to add invalid @Overrides automatically and in the second case I don't know that I'm the one that should be doing such a sweeping clean-up action.  Is the rest of the team ready for that?

Thanks,

Christian



On 2013-10-09, at 6:12 PM, Ed Willink <ed@xxxxxxxxxxxxx> wrote:

> Hi Christian
>
> There has been discussion on cross-project-dev.
>
> org.eclipse.osgi.util.NLS has moved to Java 6 making any realistic Java 5 usage impossible.
>
> The platform actually discontinued Java 5 support in Helios.
>
> As a minimum move all launches, or all test plugins to Java 6 and the problem goes away.
>
> But pedantically you are right to complain about Java 6 code in a Java 5 BREE plugin.
>
>    Regards
>
>        Ed Willink
>
>
> On 09/10/2013 22:53, Christian W. Damus wrote:
>> Hi, Team,
>>
>> I've been focusing on some UML2 and CDO work of late but had a need to rebase my Papyrus workspace today on the latest Git master.  I now have a bunch of compilation errors (see below) indicating that code has been committed that requires Java 6.0 source compliance level.  The plan is for 1.0 to require JavaSE-6.0 BREE, but currently all of the bundle manifests still stipulate a J2SE-1.5 BREE.  Consequently, Eclipse PDE compiles these projects with 1.5 source compatibility and generates these errors.
>>
>> These errors (in particular the one in the ResourceUpdateService) actually cause stuff to fail in JUnit tests and the run-time workbench.
>>
>> Am I the only one with this problem?  If so, why doesn't everybody have it?  I think it would be best if we update the BREEs now, at least in the plug-ins that now actually require 1.6 compatibility for compilation.
>>
>> In the mean-time, can anybody suggest how I can work around this?  Note that I also have CDO and other Eclipse projects in this workspace, so overriding the Java compiler settings workspace-wide is not really an option.
>>
>> Thanks,
>>
>> Christian
>>
>> -------- 8<  --------
>>
>> The method setStereotypePropertiesInBrace(String) of type LollipopInterfaceFigure must override a superclass method  LollipopInterfaceFigure.java
>> The method setStereotypePropertiesInCompartment(String) of type LollipopInterfaceFigure must override a superclass method    LollipopInterfaceFigure.java
>> The method getStereotypesLabel() of type LollipopInterfaceFigure must override a superclass method   LollipopInterfaceFigure.java
>> The method setStereotypeDisplay(String, Image) of type LollipopInterfaceFigure must override a superclass method     LollipopInterfaceFigure.java
>> The method run() of type new Runnable(){} must override a superclass method  ResourceUpdateService.java
>> The method getPrimaryShape() of type NodeEditPart must override a superclass method  NodeEditPart.java
>>
>> _______________________________________________
>> mdt-papyrus.dev mailing list
>> mdt-papyrus.dev@xxxxxxxxxxx
>> https://dev.eclipse.org/mailman/listinfo/mdt-papyrus.dev
>>
>>
>> -----
>> No virus found in this message.
>> Checked by AVG - www.avg.com
>> Version: 2013.0.3408 / Virus Database: 3222/6736 - Release Date: 10/09/13
>>
>>
>
> _______________________________________________
> mdt-papyrus.dev mailing list
> mdt-papyrus.dev@xxxxxxxxxxx
> https://dev.eclipse.org/mailman/listinfo/mdt-papyrus.dev

_______________________________________________
mdt-papyrus.dev mailing list
mdt-papyrus.dev@xxxxxxxxxxx
https://dev.eclipse.org/mailman/listinfo/mdt-papyrus.dev