Skip to content

Commit

Permalink
Add profile for neo4j-server with java.xml.bind
Browse files Browse the repository at this point in the history
JDK 9+ does not contain JAXB. It can be added with
"--add-modules java.xml.bind" for JDK 9 and 10. However, JAXB is
completely removed from JDK 11. This commit adds a new profile to the
neo4j-server module  with additional dependencies on JAXB to make
`ServerConfigIT` pass.
  • Loading branch information
lutovich committed Jul 4, 2018
1 parent e0f4915 commit 7395da2
Show file tree
Hide file tree
Showing 2 changed files with 56 additions and 0 deletions.
29 changes: 29 additions & 0 deletions community/server/pom.xml
Expand Up @@ -77,6 +77,35 @@
</license>
</licenses>

<profiles>
<!-- Profile for JDK 9+ that adds JAXB dependencies absent in the JDK -->
<profile>
<id>java9AndLater</id>
<activation>
<activeByDefault>false</activeByDefault>
<jdk>[9, )</jdk>
</activation>
<dependencies>
<dependency>
<groupId>javax.xml.bind</groupId>
<artifactId>jaxb-api</artifactId>
</dependency>
<dependency>
<groupId>com.sun.xml.bind</groupId>
<artifactId>jaxb-impl</artifactId>
</dependency>
<dependency>
<groupId>com.sun.xml.bind</groupId>
<artifactId>jaxb-core</artifactId>
</dependency>
<dependency>
<groupId>javax.activation</groupId>
<artifactId>activation</artifactId>
</dependency>
</dependencies>
</profile>
</profiles>

<build>
<plugins>
<plugin>
Expand Down
27 changes: 27 additions & 0 deletions pom.xml
Expand Up @@ -1405,6 +1405,33 @@
<version>0.15</version>
<scope>test</scope>
</dependency>

<!-- The following 4 dependencies are needed to run neo4j-server test on JDK 9+ where java.xml.bind is missing -->
<dependency>
<groupId>javax.xml.bind</groupId>
<artifactId>jaxb-api</artifactId>
<version>2.3.0</version>
<scope>test</scope>
</dependency>
<dependency>
<groupId>com.sun.xml.bind</groupId>
<artifactId>jaxb-impl</artifactId>
<version>2.3.0</version>
<scope>test</scope>
</dependency>
<dependency>
<groupId>com.sun.xml.bind</groupId>
<artifactId>jaxb-core</artifactId>
<version>2.3.0</version>
<scope>test</scope>
</dependency>
<dependency>
<groupId>javax.activation</groupId>
<artifactId>activation</artifactId>
<version>1.1.1</version>
<scope>test</scope>
</dependency>

</dependencies>
</dependencyManagement>
</project>

0 comments on commit 7395da2

Please sign in to comment.