Logo AND Algorithmique Numérique Distribuée

Public GIT Repository
Ugly hack to remove temporary files on Windows.
[simgrid.git] / src / bindings / java / org / simgrid / NativeLib.java
index 13d27ff..757b8ef 100644 (file)
@@ -12,6 +12,7 @@ import java.io.OutputStream;
 import java.io.File;
 import java.nio.file.Files;
 import java.nio.file.Path;
+import java.util.stream.Stream;
 
 /** Helper class loading the native functions of SimGrid that we use for downcalls
  *
@@ -69,9 +70,9 @@ public final class NativeLib {
                
                System.err.println("\nCannot load the bindings to the "+name+" library in path "+getPath()+" and no usable SimGrid installation found on disk.");
                if (cause != null) {
-                       if (cause.getMessage().matches(".*libcgraph.so.*"))
+                       if (cause.getMessage().contains("libcgraph.so"))
                                System.err.println("HINT: Try to install the libcgraph package (sudo apt-get install libcgraph).");
-                       else if (cause.getMessage().matches(".*libboost_context.so.*"))
+                       else if (cause.getMessage().contains("libboost_context.so"))
                                System.err.println("HINT: Try to install the boost-context package (sudo apt-get install libboost-context-dev).");
                        else
                                System.err.println("Try to install the missing dependencies, if any. Read carefully the following error message.");
@@ -90,6 +91,19 @@ public final class NativeLib {
                
                // We must write the lib onto the disk before loading it -- stupid operating systems
                if (tempDir == null) {
+
+                       if (System.getProperty("os.name").toLowerCase().startsWith("win")) {
+                               // The cleanup at exit fails on Windows where it is impossible to delete files which are still in
+                               // use.  Try to remove stale temporary files from previous executions, and limit disk usage.
+                               Path tmpdir = (new File(System.getProperty("java.io.tmpdir"))).toPath();
+                               Files.find(tmpdir, 1, (Path p, java.nio.file.attribute.BasicFileAttributes a) ->
+                                          a.isDirectory() && !p.equals(tmpdir) &&
+                                          p.getFileName().toString().startsWith("simgrid-java-"))
+                                    .map(Path::toFile)
+                                    .map(FileCleaner::new)
+                                    .forEach(FileCleaner::run);
+                       }
+
                        tempDir = Files.createTempDirectory("simgrid-java-");
                        // don't leak the files on disk, but remove it on JVM shutdown
                        Runtime.getRuntime().addShutdownHook(new Thread(new FileCleaner(tempDir.toFile())));
@@ -102,27 +116,20 @@ public final class NativeLib {
                     name+".dll", "lib"+name+".dll", /* windows (pure and mingw) */
                     "lib"+name+".dylib"             /* macOS */}) {
                                                
-                       File fileOut = new File(tempDir.toFile().getAbsolutePath() + File.separator + filename);
-                       boolean done = false;
+                       File fileOut = new File(tempDir.toFile(), filename);
                        try ( // Try-with-resources. These stream will be autoclosed when needed.
                                InputStream in = NativeLib.class.getClassLoader().getResourceAsStream(path+filename);
-                               OutputStream out = new FileOutputStream(fileOut);
                        ) {
                                if (in != null) {
                                        /* copy the library in position */
-                                       byte[] buffer = new byte[4096];
-                                       int bytesRead;
-                                       while ((bytesRead = in.read(buffer)) != -1)     // Read until EOF
-                                               out.write(buffer, 0, bytesRead);
-                                       done = true;
-                               }
-                       }
-                       if (done) {
-                               /* load that library */
-                               System.load(fileOut.getAbsolutePath());
+                                       Files.copy(in, fileOut.toPath());
+
+                                       /* load that library */
+                                       System.load(fileOut.getAbsolutePath());
 
-                               /* It loaded! we're good */
-                               return true;
+                                       /* It loaded! we're good */
+                                       return true;
+                               }
                        }
                }
                
@@ -161,16 +168,16 @@ public final class NativeLib {
                }
                @Override
                public void run() {
-                       try {
-                               for (File f : dir.listFiles())
-                                       if (! f.delete() && !f.getAbsolutePath().contains("appveyor")) // Be silent on AppVeyor to not break the tests. Ugly trick :)
-                                               System.out.println("Unable to clean temporary file "+f.getAbsolutePath()+" during shutdown.");
-                               if (! dir.delete() && !dir.getAbsolutePath().contains("appveyor") )
-                                       System.out.println("Unable to clean temporary file "+dir.getAbsolutePath()+" during shutdown.");                                
+                        try (Stream<Path> paths = Files.walk(dir.toPath())) {
+                                paths.sorted(java.util.Comparator.reverseOrder())
+                                     .map(java.nio.file.Path::toFile)
+                                     //.peek(System.err::println) // Prints what gets removed
+                                     .forEach(java.io.File::delete);
                        } catch(Exception e) {
-                               System.out.println("Error while cleaning temporary file "+dir.getAbsolutePath()+" during shutdown: "+e.getCause());
+                               System.err.println("Error while cleaning temporary file " + dir.getAbsolutePath() +
+                                                  ": " + e.getCause());
                                e.printStackTrace();
-                       }
+                        }
                }
        }
 }