[GR-46590] Remove Truffle dependency on the jdk.vm.ci.services.Services#getSavedProperties.

tzezula · tzezula · commit b79291f523e5 · 2023-06-15T08:12:32.000Z
PullRequest: graal/14785
diff --git a/compiler/src/jdk.internal.vm.compiler/src/org/graalvm/compiler/truffle/runtime/GraalRuntimeSupport.java b/compiler/src/jdk.internal.vm.compiler/src/org/graalvm/compiler/truffle/runtime/GraalRuntimeSupport.java
@@ -49,8 +49,6 @@
 import com.oracle.truffle.api.nodes.Node;
 import com.oracle.truffle.api.nodes.RootNode;
 
-import jdk.vm.ci.services.Services;
-
 final class GraalRuntimeSupport extends RuntimeSupport {
 
     GraalRuntimeSupport(Object permission) {
@@ -214,11 +212,6 @@ public Assumption createAlwaysValidAssumption() {
         return OptimizedAssumption.createAlwaysValid();
     }
 
-    @Override
-    public String getSavedProperty(String key) {
-        return Services.getSavedProperties().get(key);
-    }
-
     @Override
     public void reportPolymorphicSpecialize(Node source) {
         final RootNode rootNode = source.getRootNode();
diff --git a/sdk/src/org.graalvm.polyglot/src/org/graalvm/polyglot/Context.java b/sdk/src/org.graalvm.polyglot/src/org/graalvm/polyglot/Context.java
@@ -1938,7 +1938,7 @@ public Context build() {
                             hostAccess, polyglotAccess, nativeAccess, createThread, hostClassLoading, innerContextOptions,
                             experimentalOptions, localHostLookupFilter, contextOptions, arguments == null ? Collections.emptyMap() : arguments,
                             permittedLanguages, useIOAccess, logHandler, createProcess, processHandler, useEnvironmentAccess, environment, zone, limits,
-                            localCurrentWorkingDirectory, hostClassLoader, allowValueSharing, useSystemExit);
+                            localCurrentWorkingDirectory, System.getProperty("java.io.tmpdir"), hostClassLoader, allowValueSharing, useSystemExit);
             return ctx;
         }
 
diff --git a/sdk/src/org.graalvm.polyglot/src/org/graalvm/polyglot/Engine.java b/sdk/src/org.graalvm.polyglot/src/org/graalvm/polyglot/Engine.java
@@ -1195,7 +1195,7 @@ public Value asValue(Object o) {
         }
 
         @Override
-        public FileSystem newDefaultFileSystem() {
+        public FileSystem newDefaultFileSystem(String hostTmpDir) {
             throw noPolyglotImplementationFound();
         }
 
diff --git a/sdk/src/org.graalvm.polyglot/src/org/graalvm/polyglot/impl/AbstractPolyglotImpl.java b/sdk/src/org.graalvm.polyglot/src/org/graalvm/polyglot/impl/AbstractPolyglotImpl.java
@@ -585,8 +585,8 @@ public abstract Context createContext(Object receiver, SandboxPolicy sandboxPoli
                         Predicate<String> classFilter,
                         Map<String, String> options,
                         Map<String, String[]> arguments, String[] onlyLanguages, IOAccess ioAccess, LogHandler logHandler, boolean allowCreateProcess, ProcessHandler processHandler,
-                        EnvironmentAccess environmentAccess, Map<String, String> environment, ZoneId zone, Object limitsImpl, String currentWorkingDirectory, ClassLoader hostClassLoader,
-                        boolean allowValueSharing, boolean useSystemExit);
+                        EnvironmentAccess environmentAccess, Map<String, String> environment, ZoneId zone, Object limitsImpl, String currentWorkingDirectory, String tmpDir,
+                        ClassLoader hostClassLoader, boolean allowValueSharing, boolean useSystemExit);
 
         public abstract String getImplementationName(Object receiver);
 
@@ -1150,8 +1150,8 @@ public Object buildLimits(long statementLimit, Predicate<Source> statementLimitS
         return getNext().buildLimits(statementLimit, statementLimitSourceFilter, onLimit);
     }
 
-    public FileSystem newDefaultFileSystem() {
-        return getNext().newDefaultFileSystem();
+    public FileSystem newDefaultFileSystem(String hostTmpDir) {
+        return getNext().newDefaultFileSystem(hostTmpDir);
     }
 
     public FileSystem allowLanguageHomeAccess(FileSystem fileSystem) {
diff --git a/sdk/src/org.graalvm.polyglot/src/org/graalvm/polyglot/io/FileSystem.java b/sdk/src/org.graalvm.polyglot/src/org/graalvm/polyglot/io/FileSystem.java
@@ -495,7 +495,7 @@ default boolean isSameFile(Path path1, Path path2, LinkOption... options) throws
      * @since 20.2.0
      */
     static FileSystem newDefaultFileSystem() {
-        return IOHelper.IMPL.newDefaultFileSystem();
+        return IOHelper.IMPL.newDefaultFileSystem(System.getProperty("java.io.tmpdir"));
     }
 
     /**
diff --git a/sdk/src/org.graalvm.polyglot/src/org/graalvm/polyglot/io/IOHelper.java b/sdk/src/org.graalvm.polyglot/src/org/graalvm/polyglot/io/IOHelper.java
@@ -1,5 +1,5 @@
 /*
- * Copyright (c) 2018, 2022, Oracle and/or its affiliates. All rights reserved.
+ * Copyright (c) 2018, 2023, Oracle and/or its affiliates. All rights reserved.
  * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
  *
  * The Universal Permissive License (UPL), Version 1.0
diff --git a/truffle/src/com.oracle.truffle.api.test/src/com/oracle/truffle/api/test/polyglot/FileSystemsTest.java b/truffle/src/com.oracle.truffle.api.test/src/com/oracle/truffle/api/test/polyglot/FileSystemsTest.java
@@ -2973,9 +2973,9 @@ protected Object execute(RootNode node, Env env, Object[] contextArguments, Obje
 
     private static FileSystem createPreInitializeContextFileSystem() throws ReflectiveOperationException {
         Class<? extends FileSystem> clazz = Class.forName("com.oracle.truffle.polyglot.FileSystems$PreInitializeContextFileSystem").asSubclass(FileSystem.class);
-        Constructor<? extends FileSystem> init = clazz.getDeclaredConstructor();
+        Constructor<? extends FileSystem> init = clazz.getDeclaredConstructor(String.class);
         ReflectionUtils.setAccessible(init, true);
-        return init.newInstance();
+        return init.newInstance(System.getProperty("java.io.tmpdir"));
     }
 
     private static void switchToImageExecutionTime(FileSystem fileSystem, Path cwd) throws ReflectiveOperationException {
diff --git a/truffle/src/com.oracle.truffle.api.test/src/com/oracle/truffle/api/test/wrapper/HostEngineDispatch.java b/truffle/src/com.oracle.truffle.api.test/src/com/oracle/truffle/api/test/wrapper/HostEngineDispatch.java
@@ -87,16 +87,16 @@ public Context createContext(Object receiver, SandboxPolicy sandboxPolicy, Outpu
                     boolean allowNativeAccess, boolean allowCreateThread, boolean allowHostClassLoading, boolean allowInnerContextOptions, boolean allowExperimentalOptions,
                     Predicate<String> classFilter, Map<String, String> options, Map<String, String[]> arguments, String[] onlyLanguages, IOAccess ioAccess, LogHandler logHandler,
                     boolean allowCreateProcess, ProcessHandler processHandler, EnvironmentAccess environmentAccess, Map<String, String> environment, ZoneId zone, Object limitsImpl,
-                    String currentWorkingDirectory, ClassLoader hostClassLoader, boolean allowValueSharing, boolean useSystemExit) {
+                    String currentWorkingDirectory, String tmpDir, ClassLoader hostClassLoader, boolean allowValueSharing, boolean useSystemExit) {
         HostEngine engine = (HostEngine) receiver;
         Engine localEngine = engine.localEngine;
         AbstractEngineDispatch dispatch = api.getDispatch(localEngine);
         Object engineReceiver = api.getReceiver(localEngine);
         Context localContext = dispatch.createContext(engineReceiver, sandboxPolicy, out, err, in, allowHostAccess, hostAccess, polyglotAccess, allowNativeAccess, allowCreateThread,
                         allowHostClassLoading,
                         allowInnerContextOptions, allowExperimentalOptions, classFilter, options, arguments, onlyLanguages, ioAccess, logHandler, allowCreateProcess, processHandler,
-                        environmentAccess, environment, zone, limitsImpl, currentWorkingDirectory, hostClassLoader, true, useSystemExit);
-        long guestContextId = hostToGuest.remoteCreateContext(engine.remoteEngine, sandboxPolicy);
+                        environmentAccess, environment, zone, limitsImpl, currentWorkingDirectory, tmpDir, hostClassLoader, true, useSystemExit);
+        long guestContextId = hostToGuest.remoteCreateContext(engine.remoteEngine, sandboxPolicy, tmpDir);
         HostContext context = new HostContext(engine, guestContextId, localContext);
         hostToGuest.registerHostContext(guestContextId, context);
         return polyglot.getAPIAccess().newContext(remoteContext, context, engine.api);
diff --git a/truffle/src/com.oracle.truffle.api.test/src/com/oracle/truffle/api/test/wrapper/HostEntryPoint.java b/truffle/src/com.oracle.truffle.api.test/src/com/oracle/truffle/api/test/wrapper/HostEntryPoint.java
@@ -117,15 +117,15 @@ <T> T unmarshall(Class<T> type, long id) {
         return type.cast(idToGuestObject.get(id));
     }
 
-    public long remoteCreateContext(long engineId, SandboxPolicy sandboxPolicy) {
+    public long remoteCreateContext(long engineId, SandboxPolicy sandboxPolicy, String tmpDir) {
         Engine engine = unmarshall(Engine.class, engineId);
         Object receiver = api.getReceiver(engine);
         AbstractEngineDispatch dispatch = api.getDispatch(engine);
         Context remoteContext = dispatch.createContext(receiver, sandboxPolicy, null, null, null, false, null, PolyglotAccess.NONE, false,
                         false, false, false, false, null, new HashMap<>(), new HashMap<>(),
                         new String[0], IOAccess.NONE, null,
                         false, null, EnvironmentAccess.NONE,
-                        null, null, null, null, null, true, false);
+                        null, null, null, null, tmpDir, null, true, false);
         return guestToHost(remoteContext);
     }
 
diff --git a/truffle/src/com.oracle.truffle.api.test/src/com/oracle/truffle/api/test/wrapper/RemoteSLTest.java b/truffle/src/com.oracle.truffle.api.test/src/com/oracle/truffle/api/test/wrapper/RemoteSLTest.java
@@ -88,6 +88,7 @@ public void setup() throws Throwable {
         HostPolyglotDispatch dispatch = new HostPolyglotDispatch();
         dispatch.setConstructors(previousPolyglot.getAPIAccess());
         dispatch.setNext(previousPolyglot);
+        dispatch.setIO(previousPolyglot.getIO());
         setPolyglotImpl(dispatch);
     }
 
diff --git a/truffle/src/com.oracle.truffle.api/src/com/oracle/truffle/api/impl/Accessor.java b/truffle/src/com.oracle.truffle.api/src/com/oracle/truffle/api/impl/Accessor.java
@@ -1178,8 +1178,6 @@ public ThreadLocalHandshake getThreadLocalHandshake() {
 
         public abstract Assumption createAlwaysValidAssumption();
 
-        public abstract String getSavedProperty(String key);
-
         public abstract void reportPolymorphicSpecialize(Node source);
 
         public abstract Object callInlined(Node callNode, CallTarget target, Object... arguments);
diff --git a/truffle/src/com.oracle.truffle.api/src/com/oracle/truffle/api/impl/DefaultRuntimeAccessor.java b/truffle/src/com.oracle.truffle.api/src/com/oracle/truffle/api/impl/DefaultRuntimeAccessor.java
@@ -172,11 +172,6 @@ public void onEngineClosed(Object runtimeData) {
 
         }
 
-        @Override
-        public String getSavedProperty(String key) {
-            return System.getProperty(key);
-        }
-
         @Override
         public Object callInlined(Node callNode, CallTarget target, Object... arguments) {
             return ((DefaultCallTarget) target).callDirectOrIndirect(callNode, arguments);
diff --git a/truffle/src/com.oracle.truffle.polyglot/src/com/oracle/truffle/polyglot/EngineAccessor.java b/truffle/src/com.oracle.truffle.polyglot/src/com/oracle/truffle/polyglot/EngineAccessor.java
@@ -985,8 +985,8 @@ public TruffleContext createInternalContext(Object sourcePolyglotLanguageContext
                 fileSystemConfig = creatorConfig.fileSystemConfig;
             } else {
                 FileSystem publicFileSystem = FileSystems.newNoIOFileSystem();
-                FileSystem internalFileSystem = PolyglotEngineImpl.ALLOW_IO ? FileSystems.newLanguageHomeFileSystem() : publicFileSystem;
-                fileSystemConfig = new FileSystemConfig(IOAccess.NONE, publicFileSystem, internalFileSystem);
+                FileSystem internalFileSystem = PolyglotEngineImpl.ALLOW_IO ? FileSystems.newLanguageHomeFileSystem(creatorConfig.fileSystemConfig.hostTmpDir) : publicFileSystem;
+                fileSystemConfig = new FileSystemConfig(IOAccess.NONE, publicFileSystem, internalFileSystem, creatorConfig.fileSystemConfig.hostTmpDir);
             }
 
             /*
diff --git a/truffle/src/com.oracle.truffle.polyglot/src/com/oracle/truffle/polyglot/FileSystems.java b/truffle/src/com.oracle.truffle.polyglot/src/com/oracle/truffle/polyglot/FileSystems.java
@@ -95,16 +95,16 @@ private FileSystems() {
         throw new IllegalStateException("No instance allowed");
     }
 
-    static FileSystem newDefaultFileSystem() {
-        return new NIOFileSystem(findDefaultFileSystem(), true);
+    static FileSystem newDefaultFileSystem(String hostTmpDirPath) {
+        return new NIOFileSystem(findDefaultFileSystem(), hostTmpDirPath, true);
     }
 
     static FileSystem newNIOFileSystem(java.nio.file.FileSystem fileSystem) {
-        return new NIOFileSystem(fileSystem, false);
+        return new NIOFileSystem(fileSystem, null, false);
     }
 
     static FileSystem allowLanguageHomeAccess(FileSystem fileSystem) {
-        return new LanguageHomeFileSystem(newDefaultFileSystem(), fileSystem);
+        return new LanguageHomeFileSystem(newDefaultFileSystem(null), fileSystem);
     }
 
     static FileSystem newReadOnlyFileSystem(FileSystem fileSystem) {
@@ -115,8 +115,8 @@ static FileSystem newNoIOFileSystem() {
         return new DeniedIOFileSystem();
     }
 
-    static FileSystem newLanguageHomeFileSystem() {
-        FileSystem defaultFS = newDefaultFileSystem();
+    static FileSystem newLanguageHomeFileSystem(String hostTmpDir) {
+        FileSystem defaultFS = newDefaultFileSystem(hostTmpDir);
         return new LanguageHomeFileSystem(new ReadOnlyFileSystem(defaultFS), new PathOperationsOnlyFileSystem(defaultFS));
     }
 
@@ -217,8 +217,8 @@ static final class PreInitializeContextFileSystem implements PolyglotFileSystem
         private FileSystem delegate; // effectively final after patch context
         private Function<Path, PreInitializePath> factory;
 
-        PreInitializeContextFileSystem() {
-            this.delegate = newDefaultFileSystem();
+        PreInitializeContextFileSystem(String tmpDir) {
+            this.delegate = newDefaultFileSystem(tmpDir);
             this.factory = new ImageBuildTimeFactory();
         }
 
@@ -245,15 +245,15 @@ String pathToString(Path path) {
                 return path.toString();
             }
             verifyImageState();
-            return ((PreInitializePath) path).resolve(newDefaultFileSystem()).toString();
+            return ((PreInitializePath) path).resolve(newDefaultFileSystem(null)).toString();
         }
 
         URI absolutePathtoURI(Path path) {
             if (delegate != INVALID_FILESYSTEM) {
                 return path.toUri();
             }
             verifyImageState();
-            Path resolved = ((PreInitializePath) path).resolve(newDefaultFileSystem());
+            Path resolved = ((PreInitializePath) path).resolve(newDefaultFileSystem(null));
             if (!resolved.isAbsolute()) {
                 throw new IllegalArgumentException("Path must be absolute.");
             }
@@ -720,14 +720,16 @@ private static final class NIOFileSystem implements PolyglotFileSystem {
         private final FileSystemProvider fileSystemProvider;
 
         private final boolean isDefault;
+        private final String hostTmpDirPath;
 
         private volatile Path userDir;
         private volatile Path tmpDir;
 
-        NIOFileSystem(java.nio.file.FileSystem fileSystem, boolean isDefault) {
+        NIOFileSystem(java.nio.file.FileSystem fileSystem, String hostTmpDirPath, boolean isDefault) {
             Objects.requireNonNull(fileSystem, "FileSystem must be non null.");
             this.fileSystem = fileSystem;
             this.fileSystemProvider = fileSystem.provider();
+            this.hostTmpDirPath = hostTmpDirPath;
             this.isDefault = isDefault;
         }
 
@@ -921,12 +923,8 @@ public Path toRealPath(Path path, LinkOption... linkOptions) throws IOException
         public Path getTempDirectory() {
             Path result = tmpDir;
             if (result == null) {
-                String tmpDirPath = EngineAccessor.RUNTIME.getSavedProperty("java.io.tmpdir");
-                if (tmpDirPath == null) {
-                    throw new IllegalStateException("The java.io.tmpdir is not set.");
-                }
-                result = parsePath(tmpDirPath);
-                if (isDefault || isDirectory(result)) {
+                if (hostTmpDirPath != null) {
+                    result = parsePath(hostTmpDirPath);
                     tmpDir = result;
                 } else {
                     throw new UnsupportedOperationException("Temporary directories not supported");
@@ -935,14 +933,6 @@ public Path getTempDirectory() {
             return result;
         }
 
-        private boolean isDirectory(Path path) {
-            try {
-                return (boolean) readAttributes(path, "isDirectory").get("isDirectory");
-            } catch (IOException ioe) {
-                return false;
-            }
-        }
-
         @Override
         public boolean isSameFile(Path path1, Path path2, LinkOption... options) throws IOException {
             Objects.requireNonNull(path1);
diff --git a/truffle/src/com.oracle.truffle.polyglot/src/com/oracle/truffle/polyglot/PolyglotContextConfig.java b/truffle/src/com.oracle.truffle.polyglot/src/com/oracle/truffle/polyglot/PolyglotContextConfig.java
@@ -121,19 +121,21 @@ static class FileSystemConfig {
         final IOAccess ioAccess;
         final FileSystem fileSystem;
         final FileSystem internalFileSystem;
+        final String hostTmpDir;
 
-        FileSystemConfig(IOAccess ioAccess, FileSystem publicFileSystem, FileSystem internalFileSystem) {
+        FileSystemConfig(IOAccess ioAccess, FileSystem publicFileSystem, FileSystem internalFileSystem, String hostTmpDir) {
             this.ioAccess = ioAccess;
             this.fileSystem = publicFileSystem;
             this.internalFileSystem = internalFileSystem;
+            this.hostTmpDir = hostTmpDir;
         }
 
         static FileSystemConfig createPatched(FileSystemConfig preInitialized, FileSystemConfig patch) {
             PreInitializeContextFileSystem preInitFs = (PreInitializeContextFileSystem) preInitialized.fileSystem;
             preInitFs.onLoadPreinitializedContext(patch.fileSystem);
             PreInitializeContextFileSystem preInitInternalFs = (PreInitializeContextFileSystem) preInitialized.internalFileSystem;
             preInitInternalFs.onLoadPreinitializedContext(patch.internalFileSystem);
-            return new FileSystemConfig(patch.ioAccess, preInitFs, preInitInternalFs);
+            return new FileSystemConfig(patch.ioAccess, preInitFs, preInitInternalFs, patch.hostTmpDir);
         }
     }
 
diff --git a/truffle/src/com.oracle.truffle.polyglot/src/com/oracle/truffle/polyglot/PolyglotContextImpl.java b/truffle/src/com.oracle.truffle.polyglot/src/com/oracle/truffle/polyglot/PolyglotContextImpl.java
@@ -3449,7 +3449,8 @@ private synchronized void checkSystemThreadsFinished() {
 
     static PolyglotContextImpl preinitialize(final PolyglotEngineImpl engine, final PreinitConfig preinitConfig, PolyglotSharingLayer sharableLayer, Set<PolyglotLanguage> languagesToPreinitialize,
                     boolean emitWarning) {
-        final FileSystemConfig fileSystemConfig = new FileSystemConfig(IOAccess.ALL, new PreInitializeContextFileSystem(), new PreInitializeContextFileSystem());
+        String tmpDir = System.getProperty("java.io.tmpdir");
+        final FileSystemConfig fileSystemConfig = new FileSystemConfig(IOAccess.ALL, new PreInitializeContextFileSystem(tmpDir), new PreInitializeContextFileSystem(tmpDir), tmpDir);
         final PolyglotContextConfig config = new PolyglotContextConfig(engine, fileSystemConfig, preinitConfig);
         final PolyglotContextImpl context = new PolyglotContextImpl(engine, config);
         synchronized (engine.lock) {
diff --git a/truffle/src/com.oracle.truffle.polyglot/src/com/oracle/truffle/polyglot/PolyglotEngineDispatch.java b/truffle/src/com.oracle.truffle.polyglot/src/com/oracle/truffle/polyglot/PolyglotEngineDispatch.java
@@ -159,14 +159,14 @@ public Context createContext(Object oreceiver, SandboxPolicy sandboxPolicy, Outp
                     boolean allowExperimentalOptions, Predicate<String> classFilter,
                     Map<String, String> options, Map<String, String[]> arguments, String[] onlyLanguages, IOAccess ioAccess, LogHandler logHandler, boolean allowCreateProcess,
                     ProcessHandler processHandler, EnvironmentAccess environmentAccess, Map<String, String> environment, ZoneId zone, Object limitsImpl, String currentWorkingDirectory,
-                    ClassLoader hostClassLoader, boolean allowValueSharing, boolean useSystemExit) {
+                    String tmpDir, ClassLoader hostClassLoader, boolean allowValueSharing, boolean useSystemExit) {
         PolyglotEngineImpl receiver = (PolyglotEngineImpl) oreceiver;
         PolyglotContextImpl context = receiver.createContext(sandboxPolicy, out, err, in, allowHostLookup, hostAccess, polyglotAccess,
                         allowNativeAccess, allowCreateThread, allowHostClassLoading,
                         allowInnerContextOptions,
                         allowExperimentalOptions,
                         classFilter, options, arguments, onlyLanguages, ioAccess, logHandler, allowCreateProcess, processHandler, environmentAccess, environment, zone, limitsImpl,
-                        currentWorkingDirectory, hostClassLoader, allowValueSharing, useSystemExit);
+                        currentWorkingDirectory, tmpDir, hostClassLoader, allowValueSharing, useSystemExit);
         return polyglot.getAPIAccess().newContext(polyglot.contextDispatch, context, context.engine.api);
     }
 
diff --git a/truffle/src/com.oracle.truffle.polyglot/src/com/oracle/truffle/polyglot/PolyglotEngineImpl.java b/truffle/src/com.oracle.truffle.polyglot/src/com/oracle/truffle/polyglot/PolyglotEngineImpl.java
diff --git a/truffle/src/com.oracle.truffle.polyglot/src/com/oracle/truffle/polyglot/PolyglotImpl.java b/truffle/src/com.oracle.truffle.polyglot/src/com/oracle/truffle/polyglot/PolyglotImpl.java

Original file line number	Diff line number	Diff line change
`@@ -1938,7 +1938,7 @@ public Context build() {`
`1938`	`1938`	`hostAccess, polyglotAccess, nativeAccess, createThread, hostClassLoading, innerContextOptions,`
`1939`	`1939`	`experimentalOptions, localHostLookupFilter, contextOptions, arguments == null ? Collections.emptyMap() : arguments,`
`1940`	`1940`	`permittedLanguages, useIOAccess, logHandler, createProcess, processHandler, useEnvironmentAccess, environment, zone, limits,`
`1941`		`- localCurrentWorkingDirectory, hostClassLoader, allowValueSharing, useSystemExit);`
	`1941`	`+ localCurrentWorkingDirectory, System.getProperty("java.io.tmpdir"), hostClassLoader, allowValueSharing, useSystemExit);`
`1942`	`1942`	`return ctx;`
`1943`	`1943`	`}`
`1944`	`1944`
Original file line number	Diff line number	Diff line change
`@@ -1195,7 +1195,7 @@ public Value asValue(Object o) {`
`1195`	`1195`	`}`
`1196`	`1196`
`1197`	`1197`	`@Override`
`1198`		`- public FileSystem newDefaultFileSystem() {`
	`1198`	`+ public FileSystem newDefaultFileSystem(String hostTmpDir) {`
`1199`	`1199`	`throw noPolyglotImplementationFound();`
`1200`	`1200`	`}`
`1201`	`1201`
Original file line number	Diff line number	Diff line change
`@@ -495,7 +495,7 @@ default boolean isSameFile(Path path1, Path path2, LinkOption... options) throws`
`495`	`495`	`* @since 20.2.0`
`496`	`496`	`*/`
`497`	`497`	`static FileSystem newDefaultFileSystem() {`
`498`		`- return IOHelper.IMPL.newDefaultFileSystem();`
	`498`	`+ return IOHelper.IMPL.newDefaultFileSystem(System.getProperty("java.io.tmpdir"));`
`499`	`499`	`}`
`500`	`500`
`501`	`501`	`/**`
Original file line number	Diff line number	Diff line change
`@@ -1,5 +1,5 @@`
`1`	`1`	`/*`
`2`		`- * Copyright (c) 2018, 2022, Oracle and/or its affiliates. All rights reserved.`
	`2`	`+ * Copyright (c) 2018, 2023, Oracle and/or its affiliates. All rights reserved.`
`3`	`3`	`* DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.`
`4`	`4`	`*`
`5`	`5`	`* The Universal Permissive License (UPL), Version 1.0`
Original file line number	Diff line number	Diff line change
`@@ -88,6 +88,7 @@ public void setup() throws Throwable {`
`88`	`88`	`HostPolyglotDispatch dispatch = new HostPolyglotDispatch();`
`89`	`89`	`dispatch.setConstructors(previousPolyglot.getAPIAccess());`
`90`	`90`	`dispatch.setNext(previousPolyglot);`
	`91`	`+ dispatch.setIO(previousPolyglot.getIO());`
`91`	`92`	`setPolyglotImpl(dispatch);`
`92`	`93`	`}`
`93`	`94`