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

Merge changes in distributed vim runtime #24

Open
krischik opened this issue Nov 4, 2022 · 2 comments
Open

Merge changes in distributed vim runtime #24

krischik opened this issue Nov 4, 2022 · 2 comments
Assignees
Labels
enhancement New feature or request

Comments

@krischik
Copy link
Owner

krischik commented Nov 4, 2022

There are a few changes in the ada plugin distributed with the vim runtime which need to merged.

It's mostly spelling mistakes and missing &cpo boilerplate code.

However the runtime version has support for gnatxref which needs further investigation. I believe gnatxref hat been superseded and is not a part of the GNAT toolchain any more and support has been removed by @thindil

@krischik krischik self-assigned this Nov 4, 2022
@krischik krischik added the enhancement New feature or request label Nov 4, 2022
@thindil
Copy link

thindil commented Nov 4, 2022

If I remember correctly, I removed the support for gnatxref mostly because gnatxref works far from perfect. Especially when it was looking for references in third-party packages. And I think it is true now that gnatxref is almost dead. It is still included in GNAT, just probably no longer maintained. Its role is taken now by Language Server Protocol implementation: https://github.com/AdaCore/ada_language_server.

But Vim doesn't have built-in support for LSP, so ALS require a third-party plugin to work. There are several available options, some of them mentioned in ALS README.md file. I use ALE: https://github.com/dense-analysis/ale, as it also has support for gnatpretty.

@TamaMcGlinn
Copy link

I'm not sure about Vim, but NeoVim does have built-in support for LSP, and I use that and these two to get excellent language support:

Plug 'TamaMcGlinn/nvim-lspconfig'
Plug 'TamaMcGlinn/nvim-lsp-gpr-selector'

The former is a fork just because Ada-related changes have been rejected for lspconfig because no other users use it, and the latter is my own addition for Alire support and support for multi-gpr projects.

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

No branches or pull requests

3 participants