Skip to content

Commit e37dc7d

Browse files
committed
/issues/63 Fixed compilation config for supporting Java 9 modules.
1 parent 5f8a92a commit e37dc7d

File tree

2 files changed

+40
-35
lines changed

2 files changed

+40
-35
lines changed

pom.xml

Lines changed: 37 additions & 28 deletions
Original file line numberDiff line numberDiff line change
@@ -523,34 +523,6 @@
523523
<groupId>org.apache.maven.plugins</groupId>
524524
<artifactId>maven-compiler-plugin</artifactId>
525525
<version>3.7.0</version>
526-
<configuration>
527-
<debug>true</debug>
528-
<optimize>true</optimize>
529-
<verbose>false</verbose>
530-
<source>${maven.compile.sourceLevel}</source>
531-
<target>${maven.compile.targetLevel}</target>
532-
<compilerArgument>-g</compilerArgument>
533-
<encoding>${project.build.sourceEncoding}</encoding>
534-
</configuration>
535-
<executions>
536-
<execution>
537-
<id>default</id>
538-
<configuration>
539-
<excludes>
540-
<exclude>module-info.java</exclude>
541-
</excludes>
542-
</configuration>
543-
</execution>
544-
<execution>
545-
<id>moduledesc</id>
546-
<configuration>
547-
<includes>
548-
<include>module-info.java</include>
549-
</includes>
550-
<release>1.9</release>
551-
</configuration>
552-
</execution>
553-
</executions>
554526
</plugin>
555527
<plugin>
556528
<groupId>org.apache.maven.plugins</groupId>
@@ -810,6 +782,43 @@
810782
</execution>
811783
</executions>
812784
</plugin>
785+
<plugin>
786+
<groupId>org.apache.maven.plugins</groupId>
787+
<artifactId>maven-compiler-plugin</artifactId>
788+
<version>3.7.0</version>
789+
<executions>
790+
<execution>
791+
<id>default-compile</id>
792+
<configuration>
793+
<!-- compile everything to ensure module-info contains right entries -->
794+
<!-- required when JAVA_HOME is JDK 8 or below -->
795+
<jdkToolchain>
796+
<version>9</version>
797+
</jdkToolchain>
798+
<release>9</release>
799+
</configuration>
800+
</execution>
801+
<execution>
802+
<id>base-compile</id>
803+
<goals>
804+
<goal>compile</goal>
805+
</goals>
806+
<!-- recompile everything for target VM except the module-info.java -->
807+
<configuration>
808+
<excludes>
809+
<exclude>module-info.java</exclude>
810+
</excludes>
811+
</configuration>
812+
</execution>
813+
</executions>
814+
<configuration>
815+
<source>${maven.compile.sourceLevel}</source>
816+
<target>${maven.compile.targetLevel}</target>
817+
<excludes>
818+
<exclude>module-info.java</exclude>
819+
</excludes>
820+
</configuration>
821+
</plugin>
813822
</plugins>
814823
</build>
815824

src/main/java/module-info.java

Lines changed: 3 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -6,17 +6,13 @@
66
* BOTTOM OF THIS PAGE. Specification: JSR-354 Money and Currency API ("Specification") Copyright
77
* (c) 2012-2013, Credit Suisse All rights reserved.
88
*/
9-
module info {
9+
module javax.money {
1010
exports javax.money;
1111
exports javax.money.convert;
1212
exports javax.money.format;
1313
exports javax.money.spi;
14-
requires java.util;
15-
requires java.util.stream;
16-
requires java.util.logging;
17-
requires java.util.concurrent;
18-
requires java.io;
19-
requires java.math;
14+
requires java.base;
15+
requires java.logging;
2016
uses javax.money.spi.CurrencyProviderSpi;
2117
uses javax.money.spi.MonetaryAmountFactoryProviderSpi;
2218
uses javax.money.spi.MonetaryAmountFormatProviderSpi;

0 commit comments

Comments
 (0)