Merge pull request #8 from JaroslavTulach/CompileOnJDK8

Make the project compilable on vanilla JDK8

Author: boris-spas

.travis.yml

@@ -2,9 +2,12 @@ os:
     - linux
     - osx
 
+osx_image: xcode9.3
+
 language: java
 
 jdk:
+    - openjdk8
     - openjdk11
 
 env:

pom.xml

@@ -149,6 +149,88 @@
                 </plugins>
             </build>
         </profile>
+        <profile>
+            <id>jdk8</id>
+            <activation>
+                <jdk>1.8</jdk>
+                <file>
+                    <missing>${java.home}/lib/truffle/truffle-api.jar</missing>
+                </file>
+            </activation>
+            <dependencies>
+                <dependency>
+                    <groupId>org.graalvm.sdk</groupId>
+                    <artifactId>graal-sdk</artifactId>
+                    <version>${graalvm.version}</version>
+                    <scope>compile</scope>
+                </dependency>
+                <dependency>
+                    <groupId>org.graalvm.js</groupId>
+                    <artifactId>js</artifactId>
+                    <version>${graalvm.version}</version>
+                    <scope>runtime</scope>
+                </dependency>
+                <dependency>
+                    <groupId>org.graalvm.js</groupId>
+                    <artifactId>js-scriptengine</artifactId>
+                    <version>${graalvm.version}</version>
+                    <scope>runtime</scope>
+                </dependency>
+                <dependency>
+                    <groupId>org.graalvm.tools</groupId>
+                    <artifactId>profiler</artifactId>
+                    <version>${graalvm.version}</version>
+                    <scope>runtime</scope>
+                </dependency>
+                <dependency>
+                    <groupId>org.graalvm.tools</groupId>
+                    <artifactId>chromeinspector</artifactId>
+                    <version>${graalvm.version}</version>
+                    <scope>runtime</scope>
+                </dependency>
+            </dependencies>
+            <build>
+                <plugins>
+                    <plugin>
+                        <groupId>org.codehaus.mojo</groupId>
+                        <artifactId>exec-maven-plugin</artifactId>
+                        <version>1.6.0</version>
+                        <executions>
+                            <execution>
+                                <id>default-cli</id>
+                                <goals>
+                                    <goal>exec</goal>
+                                </goals>
+                                <configuration>
+                                    <arguments>
+                                        <argument>-classpath</argument>
+                                        <classpath/>
+                                        <argument>com.mycompany.app.App</argument>
+                                    </arguments>
+                                </configuration>
+                            </execution>
+                            <execution>
+                                <id>nograal</id>
+                                <goals>
+                                    <goal>exec</goal>
+                                </goals>
+                                <configuration>
+                                    <arguments>
+                                        <argument>-classpath</argument>
+                                        <!-- automatically creates the classpath using all project dependencies, also adding the project build directory -->
+                                        <classpath/>
+                                        <argument>com.mycompany.app.App</argument>
+                                    </arguments>
+                                </configuration>
+                            </execution>
+                        </executions>
+                        <configuration>
+                            <executable>${JAVA_HOME}/bin/java</executable>
+                        </configuration>
+                    </plugin>
+                </plugins>
+            </build>
+        </profile>
     </profiles>
     <build>
         <plugins>

src/test/java/com/mycompany/app/AppTest.java

@@ -1,6 +1,7 @@
 package com.mycompany.app;
 
 import static org.junit.Assert.fail;
+import org.junit.Assume;
 import org.junit.BeforeClass;
 import org.junit.Test;
 
@@ -14,6 +15,7 @@ public static void nashornBenchmark() throws Exception {
 
     @Test
     public void testGraalPolyglotSpeed() throws Exception {
+        assertGraalVMOrJDK11();
         long graalJS = App.benchGraalPolyglotContext();
         if (nashorn < graalJS) {
             fail(String.format("Graal.js (%d ms) should be faster than Nashorn (%d ms).", graalJS, nashorn));
@@ -22,9 +24,21 @@ public void testGraalPolyglotSpeed() throws Exception {
 
     @Test
     public void testGraalScriptEngineSpeed() throws Exception {
+        assertGraalVMOrJDK11();
         long graalJS = App.benchGraalScriptEngine();
         if (nashorn < graalJS) {
             fail(String.format("Graal.js (%d ms) should be faster than Nashorn (%d ms).", graalJS, nashorn));
         }
     }
+
+    private void assertGraalVMOrJDK11() {
+        if (System.getProperty("java.vm.name").contains("GraalVM")) {
+            return;
+        }
+        try {
+            Class.forName("java.lang.Module");
+        } catch (ClassNotFoundException ex) {
+            Assume.assumeNoException("Skipping the test on regular JDK8", ex);
+        }
+    }
 }