JDK path changes (#296)

* Environment variable JAVA_HOME does not include 'bin' in path.

* Trying to fix spaces problem.

---------

Co-authored-by: saibulusu <[email protected]>

Author: saibulusu

.pipelines/azure-pipelines-linux.yml

@@ -16,7 +16,7 @@ resources:
     options: --entrypoint=""
 
 variables:
-  VcVersion : 1.14.14
+  VcVersion : 1.14.15
   ROOT: $(Build.SourcesDirectory)
   CDP_DEFINITION_BUILD_COUNT: $[counter('', 0)] # needed for onebranch.pipeline.version task https://aka.ms/obpipelines/versioning
   ENABLE_PRS_DELAYSIGN: 1

.pipelines/azure-pipelines.yml

@@ -18,7 +18,7 @@ pool:
   vmImage: windows-latest
 
 variables:
-  VcVersion : 1.14.14
+  VcVersion : 1.14.15
   ROOT: $(Build.SourcesDirectory)
   CDP_DEFINITION_BUILD_COUNT: $[counter('', 0)] # needed for onebranch.pipeline.version task https://aka.ms/obpipelines/versioning
   ENABLE_PRS_DELAYSIGN: 1

src/VirtualClient/VirtualClient.Dependencies/JDKPackageDependencyInstallation.cs

@@ -80,7 +80,7 @@ protected override async Task ExecuteAsync(EventContext telemetryContext, Cancel
                     javaExecutable = this.PlatformSpecifics.Combine(javaDirectory, "java.exe");
                 }
 
-                this.SetEnvironmentVariable(EnvironmentVariable.JAVA_HOME, javaDirectory, EnvironmentVariableTarget.Process);
+                this.SetEnvironmentVariable(EnvironmentVariable.JAVA_HOME, jdkPackage.Path, EnvironmentVariableTarget.Process);
                 this.SetEnvironmentVariable(EnvironmentVariable.JAVA_EXE, javaExecutable, EnvironmentVariableTarget.Process);
                 this.SetEnvironmentVariable(EnvironmentVariable.PATH, javaDirectory, append: true);