vaadin / maven-plugin

Vaadin Maven Plug-in
4 stars 18 forks source link

Project does not compile using Java 6 #1

Closed Artur- closed 7 years ago

Artur- commented 8 years ago
mvn clean install -Denforcer.skip=true 
[INFO] -------------------------------------------------------------
[ERROR] COMPILATION ERROR : 
[INFO] -------------------------------------------------------------
[ERROR] /opt/home/buildagent/tst/maven-plugin/src/main/java/com/vaadin/integration/maven/wscdn/CvalInfo.java:[6,18] cannot find symbol
symbol  : class Transient
location: package java.beans
[ERROR] /opt/home/buildagent/tst/maven-plugin/src/main/java/com/vaadin/integration/maven/wscdn/CvalInfo.java:[38,6] cannot find symbol
symbol  : class Transient
location: class com.vaadin.integration.maven.wscdn.CvalInfo
[ERROR] /opt/home/buildagent/tst/maven-plugin/src/main/java/com/vaadin/integration/maven/wscdn/CvalInfo.java:[6,18] cannot find symbol
symbol  : class Transient
location: package java.beans
[ERROR] /opt/home/buildagent/tst/maven-plugin/src/main/java/com/vaadin/integration/maven/wscdn/CvalInfo.java:[38,6] cannot find symbol
symbol  : class Transient
location: class com.vaadin.integration.maven.wscdn.CvalInfo
[INFO] 4 errors 
Artur- commented 8 years ago

There are also test failures of type

testFindServletWithPrependPath(org.codehaus.mojo.gwt.ServletAnnotationFinderTest)  Time elapsed: 0.56 sec  <<< ERROR!
java.lang.UnsupportedClassVersionError: com/google/gwt/user/client/rpc/RemoteServiceRelativePath : Unsupported major.minor version 51.0

This seems to come from an incorrect dependency on vaadin-client LATEST (which is 8.0.0.something)

hesara commented 8 years ago

In addition to project settings and Animal Sniffer and eliminating diamond operations, this requires also dependency updates and changing some API (JAX-RS etc.).

hesara commented 8 years ago

WSCDN client version that does not require Java 7 in https://dev.vaadin.com/review/#/c/14680/ .

hesara commented 8 years ago

Updated WSCDN version in both master and 7.7 branch to a Java 6 compatible one (0.9.2). Fixed some other issues. To be tested more extensively, but at least several of the goals do work in a Java 6 only environment.

Artur- commented 7 years ago

So this is fixed?

hesara commented 7 years ago

The issue that was identified then was fixed for Vaadin 7 versions of the plug-in, but this wasn't tested in a pure Java 6 only environment. Closing this as probably fixed, please comment to have this reopened if there are still some problems.