Skip to content

Remove hardcoded fallback to tem from ms JDK#1316

Merged
chrmarti merged 4 commits intodevcontainers:mainfrom
Mathiyarasy:dev/Mathi/Java
Apr 7, 2025
Merged

Remove hardcoded fallback to tem from ms JDK#1316
chrmarti merged 4 commits intodevcontainers:mainfrom
Mathiyarasy:dev/Mathi/Java

Conversation

@Mathiyarasy
Copy link
Copy Markdown
Contributor

@Mathiyarasy Mathiyarasy commented Apr 2, 2025

Feature Name

  • Java

Description of Changes

  • Fixes Issue: Java feature failed to install #1308
  • Latest JDK Distro version 24 is released.
  • Fallback mechanism from ms to tem have hardcoded JDK versions(8,18,22,23)
  • This causes issue as the latest version 24 is not among them

Changelog

  • Made changes to install.sh file
  • Made changes to test scenarios file

Copy link
Copy Markdown
Contributor

@Kaniska244 Kaniska244 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi @Mathiyarasy ,

The change looks good. Kindly check the below point also once.
In devcontainer-feature.json file, please bump up the version & also lets add the current Java lts version(21) in the below list of proposals.

      "proposals": [
        "latest",
        "none",
        "17",
        "11",
        "8"
      ],

Perhaps we could also add lts as a version here.

Comment thread src/java/install.sh Outdated
@Mathiyarasy Mathiyarasy marked this pull request as ready for review April 3, 2025 11:26
@Mathiyarasy Mathiyarasy requested a review from a team as a code owner April 3, 2025 11:26
Copy link
Copy Markdown
Contributor

@Kaniska244 Kaniska244 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Changes are fine.

Copy link
Copy Markdown
Contributor

@chrmarti chrmarti left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great, thanks!

@chrmarti
Copy link
Copy Markdown
Contributor

chrmarti commented Apr 7, 2025

Merging to unblock other work.

@chrmarti chrmarti merged commit 66385f3 into devcontainers:main Apr 7, 2025
12 checks passed
@Mathiyarasy Mathiyarasy deleted the dev/Mathi/Java branch April 15, 2025 07:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants