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

Bundled ls #2

Merged
merged 2 commits into from
Dec 5, 2023
Merged

Bundled ls #2

merged 2 commits into from
Dec 5, 2023

Conversation

bafto
Copy link
Member

@bafto bafto commented Dec 5, 2023

Mit diesem Commit kann man optional die DDPLS binaries in das bin/ directory packen, um diese mit der extension zu bundlen.
Außerdem kann man in settings.json eine ddp.DDPLS.path Einstellung angeben, falls man eine eigene DDPLS Version benutzen möchte.

Die Extension würde dann so vorgehen:

  1. Wenn ddp.DDPLS.path gesetzt ist, diese binary benutzen
  2. ansonsten überprüfen ob /bin/DDPLS<.exe> existiert, und wenn ja diese binary benutzen
  3. ansonsten eine globale Installation von DDPLS benutzen (dafür muss DDPLS im PATH sein, so wie es momentan ist)

Das würde es ermöglichen Language Server updates zu veröffentlichen, ohne dass a) Nutzer von Hand die LS Binary herunterladen müssen oder b) wir einen komplett neuen DDP Release machen müssen.
Das wäre sehr praktisch für simple hot-fixes im parser.

@bafto bafto requested a review from NotLe0n December 5, 2023 16:27
@bafto bafto added the enhancement New feature or request label Dec 5, 2023
@bafto bafto merged commit 6dbd192 into main Dec 5, 2023
@NotLe0n NotLe0n deleted the bundled-ls branch December 5, 2023 17:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants