JENKINS-24265: detect when environment variable isn't set. Point to i…

…ssue and workaround
jenkinsci · May 26, 2015 · 6859427 · 6859427
1 parent da01def
commit 6859427
Showing 1 changed file with 5 additions and 1 deletion.
diff --git a/src/main/java/org/jenkinsci/plugins/unity3d/Unity3dInstallation.java b/src/main/java/org/jenkinsci/plugins/unity3d/Unity3dInstallation.java
@@ -118,7 +118,11 @@ private File getEditorLogFile(String customLogFile) {
         if (customLogFile != null) return new File(customLogFile);
 
         if (Functions.isWindows()) {
-            File applocaldata = new File(EnvVars.masterEnvVars.get("LOCALAPPDATA"));
+            String localAppData = EnvVars.masterEnvVars.get("LOCALAPPDATA");
+            if (localAppData == null) {
+                throw new RuntimeException("Empty LOCALAPPDATA environment variable. Use -logFile command line argument as workaround. Unable to find Editor.log location (see JENKINS-24265).");
+            }
+            File applocaldata = new File(localAppData);
             return new File(applocaldata, "Unity/Editor/Editor.log");
         } else { // mac assumed
             File userhome = new File(EnvVars.masterEnvVars.get("HOME"));