Fix migration issue with setEnv

ballerina/Ballerina.toml

@@ -40,4 +40,4 @@ path = "./lib/core-12.62.7.jar"
 groupId = "io.github.toolfactory"
 artifactId = "jvm-driver"
 version = "9.4.3"
-path = "./lib/jvm-driver-9.4.3.jar"
+path = "./lib/jvm-driver-9.4.3.jar"

ballerina/build.gradle

@@ -63,6 +63,12 @@ dependencies {
     externalJars(group: 'io.ballerina.stdlib', name: 'io-native', version: "${stdlibIoVersion}") {
         transitive = false
     }
+    externalJars(group: 'org.burningwave', name: 'core', version: "${burningwaveCoreVersion}") {
+        transitive = false
+    }
+    externalJars(group: 'io.github.toolfactory', name: 'jvm-driver', version: "${toolfactoryJvmDriverVersion}") {
+        transitive = false
+    }
 }
 
 apply plugin: 'io.ballerina.plugin'
@@ -83,6 +89,8 @@ task updateTomlFiles {
         newConfig = newConfig.replace("@toml.version@", tomlVersion)
         newConfig = newConfig.replace("@io.native.version@", stdlibNativeIoVersion)
         newConfig = newConfig.replace("@io.version@", stdlibIoVersion)
+        newConfig = newConfig.replace("@burningwave.core.version@", burningwaveCoreVersion)
+        newConfig = newConfig.replace("@toolfactory.jvmdriver.version@", toolfactoryJvmDriverVersion)
         ballerinaTomlFile.text = newConfig
         setExecPath(configTOMLFile,distributionBinPath)
     }

ballerina/tests/Config.toml

@@ -1,2 +1,2 @@
 [os]
-bal_exec_path = "@exec.path@"
+bal_exec_path = "/Users/waruna/Dev/jdk17/module-ballerina-system/target/ballerina-runtime/bin/bal"

ballerina/tests/os_test.bal

@@ -20,9 +20,7 @@ import ballerina/io;
 
 configurable string bal_exec_path = ?;
 
-@test:Config {
-    enable:false
-}
+@test:Config {}
 function testGetEnv() {
     string expectedValue = getExpectedValidEnv();
     test:assertEquals(getEnv("JAVA_HOME"), expectedValue);
@@ -53,7 +51,6 @@ function setEnvDataProvider() returns (string[][]) {
 }
 
 @test:Config {
-    enable: false,
     dataProvider: setEnvDataProvider
 }
 function testSetEnv(string key, string value) {
@@ -94,7 +91,6 @@ function unsetEnvDataProvider() returns (string[][]) {
 }
 
 @test:Config {
-    enable: false,
     dataProvider: unsetEnvDataProvider
 }
 function testUnsetEnv(string key, string value) {

build-config/resources/Ballerina.toml

@@ -29,3 +29,15 @@ groupId = "io.ballerina.stdlib"
 artifactId = "os-test-utils"
 version = "@toml.version@"
 path = "../test-utils/build/libs/os-test-utils-@project.version@.jar"
+
+[[platform.java17.dependency]]
+groupId = "org.burningwave"
+artifactId = "core"
+version = "@burningwave.core.version@"
+path = "./lib/core-@burningwave.core.version@.jar"
+
+[[platform.java17.dependency]]
+groupId = "io.github.toolfactory"
+artifactId = "jvm-driver"
+version = "@toolfactory.jvmdriver.version@"
+path = "./lib/jvm-driver-@toolfactory.jvmdriver.version@.jar"

gradle.properties

@@ -8,7 +8,9 @@ shadowJarVersion=7.1.2
 undercouchDownloadVersion=5.4.0
 researchgateReleaseVersion=2.8.0
 puppycrawlCheckstyleVersion=10.12.0
+burningwaveCoreVersion=12.62.7
+toolfactoryJvmDriverVersion=9.4.3
 
 
-ballerinaLangVersion=2201.8.0-20230726-132400-5c2e50ed
+ballerinaLangVersion=2201.8.0-20230726-145300-b2bdf796
 stdlibIoVersion=1.5.0

native/build.gradle

@@ -25,6 +25,7 @@ dependencies {
     implementation group: 'org.ballerinalang', name: 'ballerina-runtime', version: "${ballerinaLangVersion}"
     implementation group: 'org.slf4j', name: 'slf4j-api', version: "${slf4jVersion}"
     implementation group: 'io.ballerina.stdlib', name: 'io-native', version: "${stdlibIoVersion}"
+    implementation group: 'org.burningwave', name: 'core', version: "${burningwaveCoreVersion}"
     checkstyle project(":checkstyle")
     checkstyle "com.puppycrawl.tools:checkstyle:${puppycrawlCheckstyleVersion}"
 }

native/src/main/java/io/ballerina/stdlib/os/nativeimpl/SetEnv.java

@@ -18,7 +18,7 @@
 package io.ballerina.stdlib.os.nativeimpl;
 
 import io.ballerina.runtime.api.values.BString;
-import sun.misc.Unsafe;
+import org.burningwave.core.assembler.StaticComponentContainer;
 
 import java.lang.reflect.Field;
 import java.security.AccessController;
@@ -39,6 +39,10 @@ private SetEnv() {
 
     }
 
+    static {
+        StaticComponentContainer.Modules.exportPackageToAllUnnamed("java.base", "java.util");
+    }
+
     @SuppressWarnings("unchecked")
     public static Object setEnv(BString key, Object value) {
         Map<String, String> env = null;
@@ -58,11 +62,6 @@ public static Object setEnv(BString key, Object value) {
                 return ErrorGenerator.createError("Error while accessing the environment variable map", e);
             }
         }
-        try {
-            disableAccessWarning();
-        } catch (Exception e) {
-            return ErrorGenerator.createError("Error disabling illegal access warnings", e);
-        }
         Field finalField = field;
         AccessController.doPrivileged((PrivilegedAction) () -> {
             finalField.setAccessible(true);
@@ -80,18 +79,4 @@ public static Object setEnv(BString key, Object value) {
         }
         return null;
     }
-
-    @SuppressWarnings("unchecked")
-    private static void disableAccessWarning() throws Exception {
-        Field theUnsafe = Unsafe.class.getDeclaredField("theUnsafe");
-        AccessController.doPrivileged((PrivilegedAction) () -> {
-            theUnsafe.setAccessible(true);
-            return null;
-        });
-        Unsafe u = (Unsafe) theUnsafe.get(null);
-
-        Class<?> cls = Class.forName("jdk.internal.module.IllegalAccessLogger");
-        Field logger = cls.getDeclaredField("logger");
-        u.putObjectVolatile(cls, u.staticFieldOffset(logger), null);
-    }
 }

native/src/main/java/module-info.java

@@ -21,6 +21,7 @@
     requires org.slf4j;
     requires io.ballerina.stdlib.io;
     requires jdk.unsupported;
+    requires org.burningwave.core;
     exports io.ballerina.stdlib.os.nativeimpl;
     exports io.ballerina.stdlib.os.utils;
 }

native/src/main/resources/burningwave.static.properties

@@ -0,0 +1,3 @@
+managed-logger.repository.enabled=false
+background-executor.all-tasks-monitoring.logger.enabled=false
+banner.hide=true