[FIXED JENKINS-11073] handle failure to set timestamp on Windows plat…

…forms more gracefully
jenkinsci · Nov 12, 2011 · a7ce870 · a7ce870
1 parent 8f41302
commit a7ce870
Showing 1 changed file with 11 additions and 3 deletions.
diff --git a/core/src/main/java/hudson/FilePath.java b/core/src/main/java/hudson/FilePath.java
@@ -61,8 +61,6 @@
 import org.apache.commons.io.input.CountingInputStream;
 import org.apache.commons.fileupload.FileItem;
 import org.kohsuke.stapler.Stapler;
-import org.jvnet.robust_http_client.RetryableHttpStream;
-
 import java.io.BufferedOutputStream;
 import java.io.File;
 import java.io.FileFilter;
@@ -1426,7 +1424,17 @@ public void copyToWithPermission(FilePath target) throws IOException, Interrupte
         copyTo(target);
         // copy file permission
         target.chmod(mode());
-        target.touch(lastModified());
+
+        try {
+            target.touch(lastModified());
+        } catch (IOException e) {
+            // On Windows this seems to fail often. See JENKINS-11073
+            if (!isUnix()) {
+                LOGGER.warning("Failed to set timestamp on " + target.getRemote());
+            } else { // rethrow
+                throw new IOException2(e);
+            }
+        }
     }
 
     /**