Skip to content

bugfix: avoid usage of removed package:get_package_path()#29

Open
sivaplaysmC wants to merge 1 commit intoiabdelkareem:mainfrom
sivaplaysmC:main
Open

bugfix: avoid usage of removed package:get_package_path()#29
sivaplaysmC wants to merge 1 commit intoiabdelkareem:mainfrom
sivaplaysmC:main

Conversation

@sivaplaysmC
Copy link

This change fixes error on csharp nvim lsp startup, by using the old $MASON environment variable.

Please merge this ASAP - the removed api is causing startup errors.

mason-org/mason.nvim#33 (reply in thread)

@ Alan-John-Byrne

Sadly this no longer works. The get_install_path method was completely removed.

I had the same problem, there is a changelog note: https://github.com/mason-org/mason.nvim/blob/7c7318e8bae7e3536ef6b9e86b9e38e74f2e125e/CHANGELOG.md?plain=1#L65

I'm using vim.fn.expand "$MASON/packages/codelldb" for example.

Thanks for the heads up 👍

This change fixes error on csharp nvim lsp startup, by using the old
$MASON environment variable.

```
return vim.fn.expand("$MASON/packages/omnisharp/omnisharp")
return package:get_install_path() .. "/omnisharp"
```
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.

2 participants

Comments