[JENKINS-8446] pick more consistent default value if someone is insta…

…lling Jenkins fresh
jenkinsci · Jul 6, 2011 · 4f0ea9d · jglick · Mar 9, 2013 · 4f0ea9d
1 parent 68d4d22
commit 4f0ea9d
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 1 deletion.
diff --git a/core/src/main/java/hudson/model/Slave.java b/core/src/main/java/hudson/model/Slave.java
@@ -236,7 +236,7 @@ public Computer createComputer() {
     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
@@ -628,6 +628,11 @@ protected Jenkins(File root, ServletContext context, PluginManager pluginManager
                 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());