Xen project Mailing List

Hey Anil, thanks a lot for your response =)

2016-11-03 11:03 GMT+01:00 Anil Madhavapeddy <anil@xxxxxxxxxx>:

> The existing tests aren't too OPAM-friendly either, since they are not present in a `build-test` field (so the extra dependencies needed are not auto installable).
>

Ok, I've put all the dependencies required for the test in the opam file.

>
> The trunk version of ocaml-crunch has had the interface changes from the main mirage trunk applied, so you will need to `opam remote add dev https://github.com/mirage/mirage-dev.git` to get the development version of all the right packages.
>
> After that, you will need to add `mirage-types-lwt {test}` as a test dependency so that the right libraries are available in the opam file. Once you have that, the tests should work.

Indeed, it exits correctly, but the results are t1.{cmi,cmx,o} files, I think these should be linked with something else to see it working?

>
> The tests seem very basic at the moment, so writing a topkg + bos test that writes the .ml files into _build/_tests, invokes the compiler on it, and verifies that the resulting compiled ML outputs the crunched contents would be useful.
>

Sounds great, I've tried to implement it in [1] (probably the least idiomatic code ever :) please let me know how can I improve it), the last compilation generates a consumer binary from [2] that is supposed to use the crunched ml files. Not sure if this is the intended usage, I've been reading through mirage-www (as an example that uses ocaml-crunch) and mirage and haven't find the exact commands to get the contents from there, I guess that some lwt composition might be involved. Is this the right approach?

Cheers!

[1] https://github.com/mirage/ocaml-crunch/compare/master...fgimenez:topkg?expand=1#diff-fa96e02582bba62b4c7d8bac70d3e8e7
[2] https://github.com/mirage/ocaml-crunch/compare/master...fgimenez:topkg?expand=1#diff-75dff2ad9c3d30fad16c24c94d4bff0d

Re: [MirageOS-devel] topkg'ing ocaml-crunch