From 4f0ea9da03301e6c671523cee1b4cf9e40a64c38 Mon Sep 17 00:00:00 2001
From: Kohsuke Kawaguchi <kk@kohsuke.org>
Date: Tue, 5 Jul 2011 19:03:08 -0700
Subject: [PATCH] [JENKINS-8446] pick more consistent default value if someone
 is installing Jenkins fresh

---
 core/src/main/java/hudson/model/Slave.java    | 2 +-
 core/src/main/java/jenkins/model/Jenkins.java | 5 +++++
 2 files changed, 6 insertions(+), 1 deletion(-)

diff --git a/core/src/main/java/hudson/model/Slave.java b/core/src/main/java/hudson/model/Slave.java
index 77f4a30487..b5cd9cd41e 100644
--- a/core/src/main/java/hudson/model/Slave.java
+++ b/core/src/main/java/hudson/model/Slave.java
@@ -236,7 +236,7 @@ public abstract class Slave extends Node implements Serializable {
     public FilePath getWorkspaceFor(TopLevelItem item) {
         FilePath r = getWorkspaceRoot();
         if(r==null)     return null;    // offline
-        return r.child(item.getName());
+        return r.child(item.getFullName());
     }
 
     public FilePath getRootPath() {
diff --git a/core/src/main/java/jenkins/model/Jenkins.java b/core/src/main/java/jenkins/model/Jenkins.java
index f1612353ad..7d6d096a21 100644
--- a/core/src/main/java/jenkins/model/Jenkins.java
+++ b/core/src/main/java/jenkins/model/Jenkins.java
@@ -628,6 +628,11 @@ public class Jenkins extends AbstractCIBase implements ModifiableItemGroup<TopLe
                 throw new IllegalStateException("second instance");
             theInstance = this;
 
+            if (!new File(root,"jobs").exists()) {
+                // if this is a fresh install, use more modern default layout that's consistent with slaves
+                workspaceDir = "${JENKINS_HOME}/workspace/${ITEM_FULLNAME}";
+            }
+
             // doing this early allows InitStrategy to set environment upfront
             final InitStrategy is = InitStrategy.get(Thread.currentThread().getContextClassLoader());
 
-- 
GitLab