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

Fix conduit build with mirage3 #190

Closed
wants to merge 6 commits into from

Conversation

avsm
Copy link
Member

@avsm avsm commented Dec 30, 2016

No description provided.

@rgrinberg
Copy link
Member

rgrinberg commented Dec 30, 2016

could you rebase on master?

@avsm
Copy link
Member Author

avsm commented Dec 30, 2016

merged against master

@@ -7,7 +7,7 @@ bug-reports: "https://github.com/mirage/ocaml-conduit/issues"
tags: "org:mirage"
license: "ISC"

build: [["./configure"] [make]]
build: [["env" "CONDUIT_DISCOVER_VERBOSE=1" "./configure"] [make]]
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I don't mind if we get rid of this variable and always write the "verbose" output as well.

@samoht
Copy link
Member

samoht commented Jan 3, 2017

Travis errors are due to missing depexts on m4 on Fedora and Alpine.

@samoht samoht changed the base branch from master to mirage-dev January 3, 2017 13:33
@samoht samoht changed the base branch from mirage-dev to master January 3, 2017 13:33
@samoht
Copy link
Member

samoht commented Jan 3, 2017

#196 should be merged into master, and I'll rebase this PR on mirage-dev.

@samoht
Copy link
Member

samoht commented Jan 3, 2017

Rebased on mirage-dev in #197

@samoht samoht closed this Jan 3, 2017
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.

3 participants