From 0f5eb4b357e4ceb52cd84757feb0edfb186ff124 Mon Sep 17 00:00:00 2001
From: Tomas Zezula <tomas.zezula@oracle.com>
Date: Mon, 10 Jun 2024 12:49:45 +0200
Subject: [PATCH 1/2] Added support for polyglot isolates for specific
 platform.

---
 README.md |  1 +
 pom.xml   | 88 +++++++++++++++++++++++++++++++++++++++++++++++++++++--
 2 files changed, 87 insertions(+), 2 deletions(-)

diff --git a/README.md b/README.md
index 88effc1..e56750b 100644
--- a/README.md
+++ b/README.md
@@ -22,6 +22,7 @@ Download Maven or import as Maven project into your IDE.
 * `mvn -Pnative package` to build a native-image
 * `mvn -Passembly package` to build an uber JAR containing all dependencies using the Maven Assembly Plugin. The resulting JAR can be executed using `java -jar embedding-1.0-SNAPSHOT-jar-with-dependencies.jar`. We do recommend not using shading whenever possible.
 * `mvn -Pshade package` to build an uber JAR containing all dependencies using the Maven Shade Plugin. The resulting JAR can be executed using `java -jar embedding-1.0-SNAPSHOT.jar`. We do recommend not using shading whenever possible.
+* `mvn -Pisolated package` to install native isolate versions of languages for the current platform 
 
 Please see the [pom.xml](./pom.xml) file for further details on the configuration.
 
diff --git a/pom.xml b/pom.xml
index ae68eb9..6ddbe63 100644
--- a/pom.xml
+++ b/pom.xml
@@ -44,6 +44,8 @@
 
         Switch to community licenses by adding a `-community` suffix to the artefact id (e.g. `js-communtiy`).
         Switch to native isolate versions of languages by adding a `-isolate` suffix. (`js-isolate`).
+        Since Polyglot version 24.1, native isolate versions of languages for specific platforms are supported.
+        Refer to the `isolate` profiles for instructions on how to activate them.
 
         Any dependency in the org.graalvm.polyglot group is intended for use by polyglot embeddings.
         -->
@@ -401,8 +403,90 @@
                 </plugins>
             </build>
         </profile>
+        <!--
+        Profiles for using a native isolate versions of languages for a specific platform.
+        Native isolate versions of languages for a specific platforms are supported since
+        Polyglot version 24.1 for JavaScript (js) and Python (python).
+        These profiles may be removed if you are not using native isolate versions of languages.
+        -->
+        <!-- Linux AMD64 -->
+        <profile>
+            <id>isolate-linux-amd64</id>
+            <activation>
+                <os>
+                    <family>unix</family>
+                    <name>linux</name>
+                    <arch>amd64</arch>
+                </os>
+            </activation>
+            <properties>
+                <isolate.platform>linux-amd64</isolate.platform>
+            </properties>
+        </profile>
+        <!-- Linux AARCH64 -->
+        <profile>
+            <id>isolate-linux-aarch64</id>
+            <activation>
+                <os>
+                    <family>unix</family>
+                    <name>linux</name>
+                    <arch>aarch64</arch>
+                </os>
+            </activation>
+            <properties>
+                <isolate.platform>linux-aarch64</isolate.platform>
+            </properties>
+        </profile>
+        <!-- macOS AMD64 -->
+        <profile>
+            <id>isolate-darwin-amd64</id>
+            <activation>
+                <os>
+                    <family>mac</family>
+                    <arch>x86_64</arch>
+                </os>
+            </activation>
+            <properties>
+                <isolate.platform>darwin-amd64</isolate.platform>
+            </properties>
+        </profile>
+        <!-- macOS AARCH64 -->
+        <profile>
+            <id>isolate-darwin-aarch64</id>
+            <activation>
+                <os>
+                    <family>mac</family>
+                    <arch>aarch64</arch>
+                </os>
+            </activation>
+            <properties>
+                <isolate.platform>darwin-aarch64</isolate.platform>
+            </properties>
+        </profile>
+        <!-- Windows AMD64 -->
+        <profile>
+            <id>isolate-windows-amd64</id>
+            <activation>
+                <os>
+                    <family>windows</family>
+                    <arch>x86_64</arch>
+                </os>
+            </activation>
+            <properties>
+                <isolate.platform>windows-amd64</isolate.platform>
+            </properties>
+        </profile>
+        <profile>
+            <id>isolated</id>
+            <dependencies>
+                <dependency>
+                    <groupId>org.graalvm.polyglot</groupId>
+                    <artifactId>js-isolate-${isolate.platform}</artifactId>
+                    <version>${graalvm.version}</version>
+                    <type>pom</type>
+                </dependency>
+            </dependencies>
+        </profile>
     </profiles>
 
 </project>
-
-

From ce1ce6888fd88b6da8a76c0e8d16375e2dc14480 Mon Sep 17 00:00:00 2001
From: Tomas Zezula <tomas.zezula@oracle.com>
Date: Thu, 13 Jun 2024 11:06:59 +0200
Subject: [PATCH 2/2] Resolved review comments.

---
 README.md |   5 +-
 pom.xml   | 138 ++++++++++++++++++++++++++++++++++++++++++++++--------
 2 files changed, 121 insertions(+), 22 deletions(-)

diff --git a/README.md b/README.md
index e56750b..94c5861 100644
--- a/README.md
+++ b/README.md
@@ -16,13 +16,12 @@ https://www.graalvm.org/latest/reference-manual/embed-languages/
 
 Download Maven or import as Maven project into your IDE.
 
-* `mvn package` build using javac
+* `mvn package` build using `javac`. Starting from GraalVM Polyglot API version 24.1.0, you can use `mvn -Pisolated package` to build with the native isolate version of a guest language. By default, only the native isolate library for the current platform is installed. To install native isolate libraries for all supported platforms, use `mvn -Pisolated -Disolated.all.platforms package`.
 * `mvn test` to run the tests
-* `mvn exec:exec` to run the Main application
+* `mvn exec:exec` to run the application. Starting from GraalVM Polyglot API version 24.1.0, you can use `mvn -Pisolated exec:exec` to embed the native isolate version of a guest language.
 * `mvn -Pnative package` to build a native-image
 * `mvn -Passembly package` to build an uber JAR containing all dependencies using the Maven Assembly Plugin. The resulting JAR can be executed using `java -jar embedding-1.0-SNAPSHOT-jar-with-dependencies.jar`. We do recommend not using shading whenever possible.
 * `mvn -Pshade package` to build an uber JAR containing all dependencies using the Maven Shade Plugin. The resulting JAR can be executed using `java -jar embedding-1.0-SNAPSHOT.jar`. We do recommend not using shading whenever possible.
-* `mvn -Pisolated package` to install native isolate versions of languages for the current platform 
 
 Please see the [pom.xml](./pom.xml) file for further details on the configuration.
 
diff --git a/pom.xml b/pom.xml
index 6ddbe63..8a39545 100644
--- a/pom.xml
+++ b/pom.xml
@@ -44,7 +44,7 @@
 
         Switch to community licenses by adding a `-community` suffix to the artefact id (e.g. `js-communtiy`).
         Switch to native isolate versions of languages by adding a `-isolate` suffix. (`js-isolate`).
-        Since Polyglot version 24.1, native isolate versions of languages for specific platforms are supported.
+        Starting from GraalVM Polyglot API version 24.1.0, native isolate versions of languages for specific platforms are supported.
         Refer to the `isolate` profiles for instructions on how to activate them.
 
         Any dependency in the org.graalvm.polyglot group is intended for use by polyglot embeddings.
@@ -108,8 +108,10 @@
                         <configuration>
                             <executable>${java.home}/bin/java</executable>
                             <arguments>
-                                <!-- We recommend running from the module-path by default.
-                                     But you can also switch to the class-path here.-->
+                                <!--
+                                We recommend running from the module path by default.
+                                You can also switch to the classpath here.
+                                -->
                                 <argument>--module-path</argument>
                                 <modulepath/>
                                 <argument>-m</argument>
@@ -121,8 +123,10 @@
                 <configuration>
                     <executable>${java.home}/bin/java</executable>
                     <arguments>
-                        <!-- We recommend running from the module-path by default.
-                        But you can also switch to the class-path here.-->
+                        <!--
+                        We recommend running from the module path by default.
+                        You can also switch to the classpath here.
+                        -->
                         <argument>--module-path</argument>
                         <modulepath/>
                         <argument>-m</argument>
@@ -225,7 +229,7 @@
                         <executions>
                             <!--
                             Copies compiler dependencies to the target/compiler folder. In order to run with an
-                            optimizing runtime on other JDKs than GraalVM we need to put the compiler on the upgrade-module-path.
+                            optimizing runtime on other JDKs than GraalVM we need to put the compiler on the upgrade module path.
                             -->
                             <execution>
                                 <id>copy-dependencies</id>
@@ -404,14 +408,14 @@
             </build>
         </profile>
         <!--
-        Profiles for using a native isolate versions of languages for a specific platform.
+        Profiles for using a native isolate language version for a specific platform.
         Native isolate versions of languages for a specific platforms are supported since
         Polyglot version 24.1 for JavaScript (js) and Python (python).
         These profiles may be removed if you are not using native isolate versions of languages.
         -->
         <!-- Linux AMD64 -->
         <profile>
-            <id>isolate-linux-amd64</id>
+            <id>isolated-linux-amd64</id>
             <activation>
                 <os>
                     <family>unix</family>
@@ -420,12 +424,12 @@
                 </os>
             </activation>
             <properties>
-                <isolate.platform>linux-amd64</isolate.platform>
+                <isolate.platform.suffix>-linux-amd64</isolate.platform.suffix>
             </properties>
         </profile>
         <!-- Linux AARCH64 -->
         <profile>
-            <id>isolate-linux-aarch64</id>
+            <id>isolated-linux-aarch64</id>
             <activation>
                 <os>
                     <family>unix</family>
@@ -434,12 +438,12 @@
                 </os>
             </activation>
             <properties>
-                <isolate.platform>linux-aarch64</isolate.platform>
+                <isolate.platform.suffix>-linux-aarch64</isolate.platform.suffix>
             </properties>
         </profile>
         <!-- macOS AMD64 -->
         <profile>
-            <id>isolate-darwin-amd64</id>
+            <id>isolated-darwin-amd64</id>
             <activation>
                 <os>
                     <family>mac</family>
@@ -447,12 +451,12 @@
                 </os>
             </activation>
             <properties>
-                <isolate.platform>darwin-amd64</isolate.platform>
+                <isolate.platform.suffix>-darwin-amd64</isolate.platform.suffix>
             </properties>
         </profile>
         <!-- macOS AARCH64 -->
         <profile>
-            <id>isolate-darwin-aarch64</id>
+            <id>isolated-darwin-aarch64</id>
             <activation>
                 <os>
                     <family>mac</family>
@@ -460,12 +464,12 @@
                 </os>
             </activation>
             <properties>
-                <isolate.platform>darwin-aarch64</isolate.platform>
+                <isolate.platform.suffix>-darwin-aarch64</isolate.platform.suffix>
             </properties>
         </profile>
         <!-- Windows AMD64 -->
         <profile>
-            <id>isolate-windows-amd64</id>
+            <id>isolated-windows-amd64</id>
             <activation>
                 <os>
                     <family>windows</family>
@@ -473,20 +477,116 @@
                 </os>
             </activation>
             <properties>
-                <isolate.platform>windows-amd64</isolate.platform>
+                <isolate.platform.suffix>-windows-amd64</isolate.platform.suffix>
             </properties>
         </profile>
+        <!--
+        Profile to package polyglot isolate libraries for all supported platforms.
+        The profile is activated using `mvn -Pisolated -Disolated.all.platforms`
+        -->
+        <profile>
+            <id>isolated-all-platforms</id>
+            <activation>
+                <property>
+                    <name>isolated.all.platforms</name>
+                </property>
+            </activation>
+            <properties>
+                <isolate.platform.suffix></isolate.platform.suffix>
+            </properties>
+        </profile>
+        <!--
+        Profile for using isolated version of a guest language.
+        The profile is activated by `mvn -Pisolated`.
+        -->
         <profile>
             <id>isolated</id>
+            <properties>
+                <isolated.language.id>js</isolated.language.id>
+            </properties>
             <dependencies>
                 <dependency>
                     <groupId>org.graalvm.polyglot</groupId>
-                    <artifactId>js-isolate-${isolate.platform}</artifactId>
+                    <artifactId>js-isolate${isolate.platform.suffix}</artifactId>
+                    <version>${graalvm.version}</version>
+                    <type>pom</type>
+                </dependency>
+                <!--
+                Including the non-isolated language as a provided dependency ensures that the non-isolated language
+                included in the project dependencies is excluded from the Java module path.
+                -->
+                <dependency>
+                    <groupId>org.graalvm.polyglot</groupId>
+                    <artifactId>js</artifactId>
                     <version>${graalvm.version}</version>
                     <type>pom</type>
+                    <scope>provided</scope>
                 </dependency>
             </dependencies>
+            <build>
+                <plugins>
+                    <plugin>
+                        <groupId>org.apache.maven.plugins</groupId>
+                        <artifactId>maven-surefire-plugin</artifactId>
+                        <version>3.1.2</version>
+                        <configuration>
+                            <systemPropertyVariables>
+                                <!--
+                                The Maven Surefire plugin sets the `polyglot.engine.SpawnIsolate=<language>`
+                                system property to utilize the isolated version of the language during unit tests.
+                                -->
+                                <polyglot.engine.SpawnIsolate>${isolated.language.id}</polyglot.engine.SpawnIsolate>
+                            </systemPropertyVariables>
+                        </configuration>
+                    </plugin>
+                    <plugin>
+                        <groupId>org.codehaus.mojo</groupId>
+                        <artifactId>exec-maven-plugin</artifactId>
+                        <version>3.1.0</version>
+                        <executions>
+                            <execution>
+                                <goals>
+                                    <goal>exec</goal>
+                                </goals>
+                            </execution>
+                            <execution>
+                                <id>no-runtime-compilation</id>
+                                <goals>
+                                    <goal>exec</goal>
+                                </goals>
+                                <configuration>
+                                    <executable>${java.home}/bin/java</executable>
+                                    <arguments>
+                                        <argument>-Dpolyglot.engine.SpawnIsolate=${isolated.language.id}</argument>
+                                        <!--
+                                        We recommend running from the module path by default.
+                                        You can also switch to the classpath here.
+                                        -->
+                                        <argument>--module-path</argument>
+                                        <modulepath/>
+                                        <argument>-m</argument>
+                                        <argument>embedding/org.example.embedding.Main</argument>
+                                    </arguments>
+                                </configuration>
+                            </execution>
+                        </executions>
+                        <configuration>
+                            <executable>${java.home}/bin/java</executable>
+                            <arguments>
+                                <argument>-Dpolyglot.engine.SpawnIsolate=${isolated.language.id}</argument>
+                                <!--
+                                We recommend running from the module path by default.
+                                You can also switch to the classpath here.
+                                -->
+                                <argument>--module-path</argument>
+                                <modulepath/>
+                                <argument>-m</argument>
+                                <argument>embedding/org.example.embedding.Main</argument>
+                            </arguments>
+                        </configuration>
+                    </plugin>
+                </plugins>
+            </build>
         </profile>
     </profiles>
-
 </project>