From 056b446480d9f24619c2c30ebdf9df6122a9b653 Mon Sep 17 00:00:00 2001
From: Jesse Glick <jglick@cloudbees.com>
Date: Wed, 21 Jan 2015 13:13:57 -0500
Subject: [PATCH] [FIXED JENKINS-26519] Accept Windows text file quasi-symlinks
 for build numbers.

---
 changelog.html                                     |  3 +++
 .../src/main/java/jenkins/model/RunIdMigrator.java |  4 +++-
 .../test/java/jenkins/model/RunIdMigratorTest.java | 14 +++++++++++++-
 3 files changed, 19 insertions(+), 2 deletions(-)

diff --git a/changelog.html b/changelog.html
index 9f2c29fd59..0768c55a43 100644
--- a/changelog.html
+++ b/changelog.html
@@ -63,6 +63,9 @@ Upcoming changes</a>
 <div id="rc" style="display:none;"><!--=BEGIN=-->
 <h3><a name=v1.598>What's new in 1.598</a> <!--=DATE=--></h3>
 <ul class=image>
+  <li class='major bug'>
+    Build format change migrator in 1.597 did not work on some Windows systems.
+    (<a href="https://issues.jenkins-ci.org/browse/JENKINS-26519">issue 26519</a>)
   <li class=bug>
     FutureImpl does not cancel its start future.
     (<a href="https://issues.jenkins-ci.org/browse/JENKINS-25514">issue 25514</a>)
diff --git a/core/src/main/java/jenkins/model/RunIdMigrator.java b/core/src/main/java/jenkins/model/RunIdMigrator.java
index d03c85479b..f425a039e5 100644
--- a/core/src/main/java/jenkins/model/RunIdMigrator.java
+++ b/core/src/main/java/jenkins/model/RunIdMigrator.java
@@ -199,7 +199,9 @@ public final class RunIdMigrator {
             String name = kid.getName();
             try {
                 String link = Util.resolveSymlink(kid);
-                if (link == null) {
+                if (link == null && name.matches("\\d+") && kid.isFile()) { // legacy Windows format
+                    link = FileUtils.readFileToString(kid);
+                } else if (link == null) {
                     continue;
                 }
                 try {
diff --git a/core/src/test/java/jenkins/model/RunIdMigratorTest.java b/core/src/test/java/jenkins/model/RunIdMigratorTest.java
index ff4824ac93..b082e988e5 100644
--- a/core/src/test/java/jenkins/model/RunIdMigratorTest.java
+++ b/core/src/test/java/jenkins/model/RunIdMigratorTest.java
@@ -28,7 +28,6 @@ import hudson.Util;
 import hudson.util.StreamTaskListener;
 import java.io.File;
 import java.nio.charset.Charset;
-import java.util.Date;
 import java.util.Map;
 import java.util.TimeZone;
 import java.util.TreeMap;
@@ -42,6 +41,7 @@ import org.junit.Before;
 import org.junit.BeforeClass;
 import org.junit.Rule;
 import org.junit.rules.TemporaryFolder;
+import org.jvnet.hudson.test.Issue;
 
 public class RunIdMigratorTest {
 
@@ -117,6 +117,18 @@ public class RunIdMigratorTest {
         assertEquals("{1=→2014-01-02_03-04-05, 2014-01-02_03-04-05={build.xml='<?xml version='1.0' encoding='UTF-8'?>\n<run>\n  <stuff>ok</stuff>\n  <number>1</number>\n  <otherstuff>ok</otherstuff>\n</run>'}}", summarize());
     }
 
+    @Issue("JENKINS-26519")
+    @Test public void windowsQuasiLinks() throws Exception {
+        write("2014-01-02_03-04-05/build.xml", "<?xml version='1.0' encoding='UTF-8'?>\n<run>\n  <stuff>ok</stuff>\n  <number>99</number>\n  <otherstuff>ok</otherstuff>\n</run>");
+        write("99", "2014-01-02_03-04-05");
+        write("lastFailedBuild", "-1");
+        write("lastSuccessfulBuild", "99");
+        assertEquals("{2014-01-02_03-04-05={build.xml='<?xml version='1.0' encoding='UTF-8'?>\n<run>\n  <stuff>ok</stuff>\n  <number>99</number>\n  <otherstuff>ok</otherstuff>\n</run>'}, 99='2014-01-02_03-04-05', lastFailedBuild='-1', lastSuccessfulBuild='99'}", summarize());
+        assertTrue(migrator.migrate(dir, null));
+        assertEquals("{99={build.xml='<?xml version='1.0' encoding='UTF-8'?>\n<run>\n  <stuff>ok</stuff>\n  <id>2014-01-02_03-04-05</id>\n  <timestamp>1388649845000</timestamp>\n  <otherstuff>ok</otherstuff>\n</run>'}, lastFailedBuild='-1', lastSuccessfulBuild='99', legacyIds='2014-01-02_03-04-05 99\n'}", summarize());
+        assertEquals(99, migrator.findNumber("2014-01-02_03-04-05"));
+    }
+
     // TODO test sane recovery from various error conditions
 
     private void write(String file, String text) throws Exception {
-- 
GitLab