[FIXED JENKINS-13515] FingerprintingCopyMethod.copyOne can fail on Wi…

…ndows due to timestamp.
jenkinsci · Sep 21, 2012 · d86690c · d86690c
1 parent c587a69
commit d86690c
Showing 1 changed file with 12 additions and 1 deletion.
diff --git a/src/main/java/hudson/plugins/copyartifact/FingerprintingCopyMethod.java b/src/main/java/hudson/plugins/copyartifact/FingerprintingCopyMethod.java
@@ -2,6 +2,7 @@
 
 import hudson.Extension;
 import hudson.FilePath;
+import hudson.Functions;
 import hudson.Util;
 import hudson.model.AbstractBuild;
 import hudson.model.Fingerprint;
@@ -17,6 +18,7 @@
 import java.security.NoSuchAlgorithmException;
 import java.util.HashMap;
 import java.util.Map;
+import java.util.logging.Logger;
 
 /**
  * Performs fingerprinting during the copy.
@@ -76,7 +78,16 @@ public void copyOne(FilePath s, FilePath d) throws IOException, InterruptedExcep
                 out.close();
             }
             d.chmod(s.mode());
-            d.touch(s.lastModified());
+            // FilePath.setLastModifiedIfPossible private; copyToWithPermission OK but would have to calc digest separately:
+            try {
+                d.touch(s.lastModified());
+            } catch (IOException x) {
+                if (Functions.isWindows()) {
+                    Logger.getLogger(FingerprintingCopyMethod.class.getName()).warning(x.getMessage());
+                } else {
+                    throw x;
+                }
+            }
             String digest = Util.toHexString(md5.digest());
 
             FingerprintMap map = Jenkins.getInstance().getFingerprintMap();