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

Port build system to dune #39

Merged
merged 4 commits into from
Aug 21, 2018
Merged

Port build system to dune #39

merged 4 commits into from
Aug 21, 2018

Conversation

rgrinberg
Copy link
Contributor

As promised before, the release of dune makes #35 possible.

Signed-off-by: Rudi Grinberg <rudi.grinberg@gmail.com>
Signed-off-by: Rudi Grinberg <rudi.grinberg@gmail.com>
Signed-off-by: Rudi Grinberg <rudi.grinberg@gmail.com>
@andreypopp
Copy link

Glad to see this is being ported to dune, I'm having troubles building ssl under esy as the current build system handles OCAMLFIND_LDCONF supplying it an empty value by default or setting to dummy (instead of ignore) on --disable-ldconf.

@toots
Copy link
Member

toots commented Aug 6, 2018

I'm cool with merging it. @smimram, any thought on it?

Signed-off-by: Rudi Grinberg <rudi.grinberg@gmail.com>
@smimram
Copy link
Member

smimram commented Aug 21, 2018

I have no strong objection against this (but I am not really sure about what this switch to dune would bring). In short, @toots I leave it up to you! ;)

@toots
Copy link
Member

toots commented Aug 21, 2018

I'm not sure either, @smimram but I'm down to have the community help maintaining this package. Thanks for the feedback!

@toots toots merged commit e23a23d into savonet:master Aug 21, 2018
@rgrinberg rgrinberg deleted the dune branch August 21, 2018 15:05
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.

4 participants