williamboman / mason.nvim

Portable package manager for Neovim that runs everywhere Neovim runs. Easily install and manage LSP servers, DAP servers, linters, and formatters.
Apache License 2.0
7.23k stars 258 forks source link

feat(linker): use relative targets for exec scripts #1643

Closed henry-hsieh closed 3 months ago

henry-hsieh commented 4 months ago

Closes #1156 and improves #1525.

henry-hsieh commented 3 months ago

Thanks for the PR! I've however already implemented a similar fix locally! It's largely the same except that the resulting wrapper executable can be executed from any context - I believe this solution would make the wrapper script error if executed directly (it has to be executed from the symlink in bin/ for the path resolution to function).

Yes, that's true. Hope the fix can be uploaded soon!