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

Patch kaputt install to check for ocamlopt explicitly #7176

Merged
merged 1 commit into from Aug 15, 2016

Conversation

Projects
None yet
2 participants
@rleonid
Copy link
Contributor

rleonid commented Aug 11, 2016

A little bit of context. While migrating to using the Travis CI scripts, I ran into the problem of ocambuild and ocamlopt being installed in separate directories due to the system OCaml install via brew. Kaputt's configuration assumes that they are installed in the same directory and will not compile and install native code even though a native compiler is available.

@dsheets

This comment has been minimized.

Copy link
Member

dsheets commented Aug 12, 2016

Have you contacted the upstream maintainer? I believe it is Xavier Clerc. If he does not want to maintain this package, perhaps you could take over? It would be nice to have a released version that doesn't have multiple patches applied downstream.

@rleonid

This comment has been minimized.

Copy link
Contributor Author

rleonid commented Aug 12, 2016

Reported the issue here since @xclerk doesn't actually mention the original developer.

@dsheets Unfortunately, at the moment I am a bit reticent to take on more titles. As mentioned in the context, this is a solution to a corner case that I think will make the project installable on more configurations.

@dsheets dsheets merged commit e40793c into ocaml:master Aug 15, 2016

1 check passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment