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

Irmin on xen - current state #348

Closed
hannesm opened this issue Apr 7, 2016 · 6 comments
Closed

Irmin on xen - current state #348

hannesm opened this issue Apr 7, 2016 · 6 comments

Comments

@hannesm
Copy link
Member

hannesm commented Apr 7, 2016

it doesn't compile anymore... bin_prot (used by mirage-tc) needs to be patched (as you did here for a 112.35 version https://github.com/samoht/bin_prot/tree/112.35.00+xen), mirage-tc has an upper bound on bin_prot < 113.01.00, but pa_sexp_conv requires (via type_conv) a bin_prot >= 113.00.00.

any plan to remove or upgrade the bin_prot dependency from mirage-tc (or remove the mirage-tc dependency from irmin)?

@hannesm
Copy link
Member Author

hannesm commented Apr 7, 2016

the above relationship between bin_prot and type_conv is wrong. an old type_conv requires an old bin_prot, and pa_sexp_conv is not happy with that type_conv

@hannesm
Copy link
Member Author

hannesm commented Apr 7, 2016

(mirage-tc should be fine with newer bin_prot ocaml/opam-repository#6205)

@hannesm
Copy link
Member Author

hannesm commented Apr 7, 2016

and for bin_prot I started to add a xen sublibrary at https://github.com/hannesm/bin_prot/tree/113.33.00+xen but somehow I do not manage to convince the build system thereof to compile and install it...

@hannesm
Copy link
Member Author

hannesm commented Apr 8, 2016

solved: https://github.com/hannesm/bin_prot/tree/113.33.00+xen now works (compiles, installs the xen sublibrary as wel); and using the result I can link a Canopy unikernel! :)

@hannesm hannesm closed this as completed Apr 8, 2016
@hannesm
Copy link
Member Author

hannesm commented Apr 8, 2016

would nevertheless be nice to get an opinion from @samoht about bin_prot...

@samoht
Copy link
Member

samoht commented Apr 8, 2016

I'm in favour of making that dependency optional.

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

No branches or pull requests

2 participants