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

Package variables in depexts #5836

Open
kit-ty-kate opened this issue Feb 14, 2024 · 1 comment
Open

Package variables in depexts #5836

kit-ty-kate opened this issue Feb 14, 2024 · 1 comment

Comments

@kit-ty-kate
Copy link
Member

Currently using package variables such as with-test in the depexts field is authorized but the variable itself seems to be undefined, and the documentation about whether or not package variables are authorized.

I think we should probably:

  • make opam lint trigger a warning when an undefined variable is present in the depexts field
  • make the documentation clearer that it is not accepted
  • think about accepting it in the future in opam 3.0 or something like that if we make opam keep track of the state of package variables (as it currently doesn't)

cc @sim642 who reported about the issue on Discord

@dra27
Copy link
Member

dra27 commented Feb 15, 2024

I think #5455 is related?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants