Skip to content

Commit

Permalink
Browse files Browse the repository at this point in the history
JENKINS-24265: document the unnecessary obsolete fallback code
  • Loading branch information
lacostej committed May 26, 2015
1 parent a7e66b6 commit 132fa33
Showing 1 changed file with 3 additions and 0 deletions.
Expand Up @@ -133,6 +133,9 @@ private File getEditorLogFile(String customLogFile) {
log.fine("Found %LOCALAPPDATA% under " + localAppData);
} catch (RuntimeException re) {
log.warning("Unable to find %LOCALAPPDATA%, reverting to Environment variable " + re.getMessage());
// JENKINS-24265 / providing fallback to LOCALAPPDATA shouldn't be necessary, but this worked most
// of the cases and I am unable to test for all Windows configurations right now.
// This should be removed someday....
localAppData = EnvVars.masterEnvVars.get("LOCALAPPDATA");
log.fine("Found %LOCALAPPDATA% (from environment variable) under " + localAppData);
if (localAppData == null) {
Expand Down

0 comments on commit 132fa33

Please sign in to comment.