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

Some opam updates #73

Merged
merged 5 commits into from
Feb 9, 2022
Merged

Some opam updates #73

merged 5 commits into from
Feb 9, 2022

Conversation

rjbou
Copy link
Contributor

@rjbou rjbou commented Feb 8, 2022

on syntax, plugin & file detection

let installed = split((system("opam switch -s -i 2> /dev/null")), "\n")
call map(installed, 'opam#chomp(v:val)')
return join(installed, "\n")
let switches = split((system("opam switch --short 2> /dev/null")), "\n")
Copy link
Contributor Author

Choose a reason for hiding this comment

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

-i i no more available, is it required to have only installed ones?

Copy link
Member

Choose a reason for hiding this comment

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

i don't think so

syn keyword opamKeyword1 syntax
syn keyword opamKeyword1 tags
syn keyword opamKeyword1 version
syn match opamKeyword2 "\v(bug-reports|post-messages|opam-version|dev-repo|build)"
Copy link
Contributor Author

Choose a reason for hiding this comment

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

What is the reason to have both opamKeyword1 and opamKeyword2 ?

Copy link
Member

Choose a reason for hiding this comment

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

No idea. You can try including it in opamKeyword1 and see if it works.

@rgrinberg rgrinberg merged commit 335ebc6 into ocaml:master Feb 9, 2022
@rjbou rjbou mentioned this pull request Oct 25, 2022
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