feat(jdtls): upgrade to vscode-java v1.40.0 with Java 21 for macOS ARM64 fix #128
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
The osx-arm64 configuration was incorrectly downloading the x64 binary (java@darwin-x64) and using x86_64 JRE paths instead of the proper ARM64 binary and aarch64 paths. Changes: - Download darwin-arm64 VSIX instead of darwin-x64 - Use macosx-aarch64 JRE paths instead of macosx-x86_64 This fixes JDTLS on Apple Silicon Macs (M1/M2/M3).
This commit upgrades Eclipse JDTLS dependencies to fix macOS ARM64 completions and bring modern Java support: Changes: - Update vscode-java from v1.23.0 to v1.40.0 for all platforms - Upgrade bundled JRE from Java 17 to Java 21.0.6 - Update JDTLS launcher to v1.6.1000.v20250131-0606 - Update lombok to v1.18.36 - Fix osx-arm64 to use correct darwin-arm64 binary and aarch64 paths - Add linux-arm64 support with config_linux_arm configuration Why Java 21 is required: - Eclipse JDTLS v1.40+ requires Java 21 as minimum runtime - The bundled JustJ JRE (Java 17) has broken jspawnhelper on macOS ARM64 - Java 21 from vscode-java has proper ARM64 process spawning support This fixes the macOS ARM64 issue where completions returned 0 items due to posix_spawn failures in the bundled Java 17 JRE. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude Opus 4.5 <[email protected]>
gruckion
changed the title
The `parent.children(recursive=True)` call can throw `psutil.NoSuchProcess` if child processes exit during iteration. This was causing TypeScript LSP tests to fail on macOS with "process PID not found" errors during server shutdown. Wrap the children iteration loop in a try-except to handle the race condition where child processes may exit between the time they're enumerated and when we try to signal them. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude Opus 4.5 <[email protected]>
Author
|
@microsoft-github-policy-service agree [company="Gruckion Limited"] |
|
@gruckion the command you issued was incorrect. Please try again. Examples are: and |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Summary
This PR fixes macOS ARM64 completions returning 0 items by:
Problem
On macOS ARM64 (Apple Silicon), Eclipse JDTLS completions returned 0 items while other features (hover, go-to-definition) worked correctly.
Root Causes
jspawnhelperon macOS ARM64, causingposix_spawnfailures when JDTLS tries to spawn processesChanges
runtime_dependencies.json:17.0.8.1-*) to Java 21 (21.0.6-*)1.6.500.v20230717-2134to1.6.1000.v20250131-0606config_linux_armconfigurationeclipse_jdtls.py:JavaSE-17toJavaSE-21initialize_params.json:Testing
Tested on macOS ARM64 (Apple Silicon M1):
Got 0 completionsGot 2 completions(includingnewServerNode,class)Related