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.

Sign up for GitHub

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

Jump to bottom

Don't link clients against PPX libraries #316

Merged
merged 3 commits into from
Jun 9, 2017
Merged

Don't link clients against PPX libraries #316

merged 3 commits into from
Jun 9, 2017

Commits on Jun 5, 2017

  1. META: don't link clients against cstruct.ppx 

    The package cstruct.ppx contains helper functions for the PPX rewriter
itself. Linking this code into applications makes them larger and also brings
in a dependency on compiler-libs which pollutes the global module namespace
with names like Types.

See mirage/mirage-www#556

Signed-off-by: David Scott <dave@recoil.org>
    @djs55
    djs55 committed Jun 5, 2017
    Configuration menu
    Copy the full SHA
    ab62419 View commit details
    Browse the repository at this point in the history

  2. Regenerate OASIS 

    Signed-off-by: David Scott <dave@recoil.org>
    @djs55
    djs55 committed Jun 5, 2017
    Configuration menu
    Copy the full SHA
    f62195e View commit details
    Browse the repository at this point in the history

  3. opam: depend on cstruct-lwt 

    From cstruct.3.0.0 onwards, the `cstruct.lwt` subpackage requires the
top-level package `cstruct-lwt`. This is part of the great "make all
ocamlfind subpackages into separate top-level opam packages"
simplification. This patch adds the necessary dependency on `cstruct-lwt`

Signed-off-by: David Scott <dave@recoil.org>
    @djs55
    djs55 committed Jun 5, 2017
    Configuration menu
    Copy the full SHA
    352f835 View commit details
    Browse the repository at this point in the history