don't delete export folders if they don't exist

benfry · benfry · commit 72b19485cda2 · 2017-01-31T17:11:29.000-05:00
diff --git a/app/src/processing/app/Mode.java b/app/src/processing/app/Mode.java
@@ -1017,10 +1017,13 @@ public void prepareExportFolder(File targetFolder) {
     if (targetFolder != null) {
       // Nuke the old applet/application folder because it can cause trouble
       if (Preferences.getBoolean("export.delete_target_folder")) {
-        try {
-          Platform.deleteFile(targetFolder);
-        } catch (IOException e) {
-          e.printStackTrace();
+        if (targetFolder.exists()) {
+          try {
+            Platform.deleteFile(targetFolder);
+          } catch (IOException e) {
+            // ignore errors/continue; likely to be ok
+            e.printStackTrace();
+          }
         }
       }
       // Create a fresh output folder (needed before preproc is run next)
diff --git a/todo.txt b/todo.txt
@@ -1,11 +1,14 @@
 0257 (3.2.5)
+X check for already-exported folders before trying to remove them
+X   was spewing 'file not found' errors into the console
 
 
 cleaning
 X Contribution Manager does not show all libraries until filter cleared
 X   https://github.com/processing/processing/issues/4840
 X   fixed in 3.2.4
 
+
 _ NVIDIA driver problems (and means to check)
 _   https://github.com/processing/processing/issues/4853
 
