devonfw / IDEasy

Tool to automate the setup and updates of a development environment for any project (Successor of devonfw-ide).
Apache License 2.0
8 stars 18 forks source link

JDK settings for intellij are not working properly #601

Open jan-vcapgemini opened 2 weeks ago

jan-vcapgemini commented 2 weeks ago

Expected behavior

As an IDEasy intellij user I expect IDEasy to configure my JDK settings in Project Settings/Project and Plattform Settings/SDKs properly.

Actual behavior

IDEasy does not replace or utilize the environment variable DEVON_IDE_HOME or IDE_HOME properly so the path to the JAVA_SDK is not valid for intellij.

image

image

Steps to reproduce (bug) / Use Case of feature request (enhancement)

  1. Remove all settings from intellij e.g. .intellij and .idea from workspaces/main
  2. Start intellij from git bash using ide intellij
  3. Check the file workspaces/main/.intellij/config/options/jdktable.xml

Related/Dependent Issues

Comments/Hints:

Might be related to:

https://github.com/devonfw/ide-settings/blob/8f6da8c195e2932976374913dd34ecdd344614df/intellij/workspace/update/idea.properties#L3

https://github.com/devonfw/ide-settings/blob/8f6da8c195e2932976374913dd34ecdd344614df/intellij/workspace/update/.intellij/config/options/jdk.table.xml#L6-L7

https://github.com/devonfw/ide-settings/blob/8f6da8c195e2932976374913dd34ecdd344614df/intellij/workspace/update/.intellij/config/options/jdk.table.xml#L16

Shouldn't this be $[IDE_HOME] instead?

Affected version:

hohwille commented 1 week ago

IDEasy does not replace or utilize the environment variable DEVON_IDE_HOME or IDE_HOME properly so the path to the JAVA_SDK is not valid for intellij.

Syntax has to be $[IDE_HOME] instead of ${IDE_HOME}. It seems I missed some spots in PR https://github.com/devonfw/ide-settings/pull/51/files Sorry for that... I just fixed this and other leftovers I have missed. Could you recheck if the bug is fixed (after ide update) and we can close?

jan-vcapgemini commented 5 days ago

I've tried to start intellij after running ide update. It looks like the SDK is now working but I'm wondering why it does not use Java and added temurin-21 instead. Both SDKs use the same path to java though.

image image image