From fa40715975ffd4874b1afad12b0f359e9f5b5ef9 Mon Sep 17 00:00:00 2001 From: "Alexander.Glukhov" Date: Fri, 22 Aug 2025 14:56:04 +0200 Subject: [PATCH] [gradle][eel][IJPL-203083] do not convert JDK path for remote execution There is no need to convert the JDK path from local to remote, as it is already remote. GitOrigin-RevId: ad3eca9c6edb46633c0948debb78170aa802edeb --- .../execution/target/GradleServerEnvironmentSetupImpl.kt | 8 +++----- 1 file changed, 3 insertions(+), 5 deletions(-) diff --git a/plugins/gradle/src/org/jetbrains/plugins/gradle/execution/target/GradleServerEnvironmentSetupImpl.kt b/plugins/gradle/src/org/jetbrains/plugins/gradle/execution/target/GradleServerEnvironmentSetupImpl.kt index d44e4fc58138..d63762b0137a 100644 --- a/plugins/gradle/src/org/jetbrains/plugins/gradle/execution/target/GradleServerEnvironmentSetupImpl.kt +++ b/plugins/gradle/src/org/jetbrains/plugins/gradle/execution/target/GradleServerEnvironmentSetupImpl.kt @@ -106,7 +106,6 @@ internal class GradleServerEnvironmentSetupImpl( val environmentConfigurationProvider = connection.environmentConfigurationProvider this.environmentConfiguration = environmentConfigurationProvider.environmentConfiguration - val targetPathMapper = environmentConfigurationProvider.pathMapper val request = if (environmentConfiguration.typeId == "local") LocalTargetEnvironmentRequest() else environmentConfiguration.createEnvironmentRequest(project) @@ -115,7 +114,7 @@ internal class GradleServerEnvironmentSetupImpl( targetBuildParametersBuilder.useInstallation(it) } - configureJdk(javaParameters, request, consumerOperationParameters, targetPathMapper, environmentConfiguration) + configureJdk(javaParameters, request, consumerOperationParameters, environmentConfiguration) progressIndicator.checkCanceled() val targetArguments = prepareTargetEnvironmentRequest( request, @@ -155,6 +154,7 @@ internal class GradleServerEnvironmentSetupImpl( targetEnvironmentProvider.uploadVolumes(progressIndicator) val pathsToMap = extractPathsToMapFromInitScripts(initScripts) + val targetPathMapper = environmentConfigurationProvider.pathMapper val pathMapperInitScript = createTargetPathMapperInitScript( request, targetPathMapper, @@ -277,7 +277,6 @@ internal class GradleServerEnvironmentSetupImpl( javaParameters: SimpleJavaParameters, request: TargetEnvironmentRequest, consumerOperationParameters: ConsumerOperationParameters, - targetPathMapper: PathMapper?, environmentConfiguration: TargetEnvironmentConfiguration, ) { if (request is LocalTargetEnvironmentRequest) { @@ -287,8 +286,7 @@ internal class GradleServerEnvironmentSetupImpl( } else { if (environmentConfiguration.runtimes.findByType(JavaLanguageRuntimeConfiguration::class.java) == null) { - val localJavaHomePath = consumerOperationParameters.javaHome.path - val targetJavaHomePath = targetPathMapper.maybeConvertToRemote(localJavaHomePath) + val targetJavaHomePath = FileUtil.toCanonicalPath(consumerOperationParameters.javaHome.path) val javaLanguageRuntimeConfiguration = JavaLanguageRuntimeConfiguration().apply { homePath = targetJavaHomePath } environmentConfiguration.addLanguageRuntime(javaLanguageRuntimeConfiguration) }