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 to jbuilder #30

Merged
merged 6 commits into from
Jun 6, 2017
Merged

Port to jbuilder #30

merged 6 commits into from
Jun 6, 2017

Conversation

avsm
Copy link
Member

@avsm avsm commented May 24, 2017

This needs careful review and testing to make sure that the new configurator based cflags are correct

@avsm avsm changed the title [do not merge] Port to jbuilder Port to jbuilder Jun 5, 2017
@hannesm
Copy link
Member

hannesm commented Jun 5, 2017

could you please elaborate on whether there was any testing done with xen and/or solo5 unikernels using this PR? if not, please compile and test some unikernels and report back. thx.

@djs55
Copy link
Member

djs55 commented Jun 5, 2017

Do we have any example unikernel configurations in opam-repository? Now that the mirage tool doesn't need to run opam as part of the build, and now that mirage configure generates a name.opam file (all very nice improvements IMHO), I think we should be able to publish some -- is that correct? If we had a few example unikernels in the repo, perhaps we could more easily test they build as part of the CI revdeps steps?

@hannesm
Copy link
Member

hannesm commented Jun 5, 2017 via email

@avsm
Copy link
Member Author

avsm commented Jun 5, 2017

could you please elaborate on whether there was any testing done with xen and/or solo5 unikernels using this PR?

solo5 yeah -- I'm provisioning a freebsd box to test out the virtio backends atm.

@avsm
Copy link
Member Author

avsm commented Jun 5, 2017

yes a separate remote with all the configuration steps in a new repo would be great. will take a look at this.

@avsm
Copy link
Member Author

avsm commented Jun 5, 2017

Woohoo! FreeBSD Virtio DNS server succeeds fine with this PR in the pin list. Just a minor issue Solo5/solo5#197 found

@avsm
Copy link
Member Author

avsm commented Jun 6, 2017

And the Linux Xen and UKVM examples in static_tls build and work as well with this pin. The configuration here is now simpler as well due to using Configurator (rather than the ocamlbuild tags), but something may have regressed despite all these checks due to the different cclib options. I'm going to merge this so it's in master and let it settle for a few days while I inspect more command lines to be sure...

@avsm avsm merged commit b60d2df into mirage:master Jun 6, 2017
@avsm avsm deleted the jbuilder branch June 6, 2017 17:40
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.

None yet

3 participants