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

Add corrected-suffix to the list of known variables when upgrading #1840

Merged
1 commit merged into from Feb 13, 2019
Merged

Add corrected-suffix to the list of known variables when upgrading #1840

1 commit merged into from Feb 13, 2019

Conversation

ghost
Copy link

@ghost ghost commented Feb 13, 2019

This fixes support for [@@deriving_inline] with older versions of ppxlib.

Signed-off-by: Jeremie Dimino <jeremie@dimino.org>
@ghost ghost merged commit 4d88e45 into ocaml:master Feb 13, 2019
rgrinberg added a commit to rgrinberg/opam-repository that referenced this pull request Feb 21, 2019
CHANGES:

- Add `${corrected-suffix}`, `${library-name}` and a few other
  variables to the list of variables to upgrade. This fixes the
  support for various framework producing corrections (ocaml/dune#1840, ocaml/dune#1853,
  @diml)

- Fix `$ dune subst` failing because the build directory wasn't set. (ocaml/dune#1854, fix
  ocaml/dune#1846, @rgrinberg)

- Configurator: Add warning to `Pkg_config.query` when a full package expression
  is used. Add `Pkg_config.query_expr` for cases when the full power of
  pkg-config's querying is needed (ocaml/dune#1842, fix ocaml/dune#1833, @rgrinberg)

- Fix unavailable, optional implementations eagerly breaking the build (ocaml/dune#1857,
  fix ocaml/dune#1856, @rgrinberg)
This pull request was closed.
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