Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix build issues for NetBeans Language Server VSCode extension on Windows platform #7709

Merged
merged 1 commit into from
Sep 12, 2024

Conversation

Achal1607
Copy link
Collaborator

Description

This PR addresses build issues for the NetBeans Language Server VSCode extension on the Windows platform.

  1. Ant Targets: Certain Ant targets, such as ant build-vscode-ext, were failing on Windows due to missing suffixes required by the Windows environment. This has been corrected to ensure compatibility.

  2. NPM Compile Script: The npm run compile command was not functioning correctly on Windows due to delimiter issues. This PR fixes the delimiter handling, allowing the command to run successfully on Windows.

Changes Made

  • Updated Ant target configurations to include necessary suffixes for Windows compatibility.
  • Modified the npm run compile script to handle delimiter issue.

@lahodaj lahodaj added LSP [ci] enable Language Server Protocol tests VSCode Extension [ci] enable VSCode Extension tests Java [ci] enable extra Java tests (java.completion, java.source.base, java.hints, refactoring.java, form) labels Sep 2, 2024
@apache apache locked and limited conversation to collaborators Sep 2, 2024
@apache apache unlocked this conversation Sep 2, 2024
@Achal1607
Copy link
Collaborator Author

Can anybody please re-trigger failed tests for this PR?

Copy link
Contributor

@lahodaj lahodaj left a comment

Choose a reason for hiding this comment

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

Looks sensible to me. Thanks!

@lahodaj lahodaj merged commit 1f07376 into apache:master Sep 12, 2024
67 of 68 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Java [ci] enable extra Java tests (java.completion, java.source.base, java.hints, refactoring.java, form) LSP [ci] enable Language Server Protocol tests VSCode Extension [ci] enable VSCode Extension tests
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants