Skip to content

mirage-types.lwt -> mirage-types-lwt#176

Merged
avsm merged 1 commit intomirage:mirage-devfrom
yomimono:types-lwt
Dec 16, 2016
Merged

mirage-types.lwt -> mirage-types-lwt#176
avsm merged 1 commit intomirage:mirage-devfrom
yomimono:types-lwt

Conversation

@yomimono
Copy link
Copy Markdown
Contributor

No description provided.

@hannesm
Copy link
Copy Markdown
Member

hannesm commented Dec 14, 2016

I don't understand the .5 breakage (where vchan is unhappy)... .3 and .4 should be fixed by mirage/mirage-dev#261 and mirage/ocaml-dns#113

@yomimono
Copy link
Copy Markdown
Contributor Author

I can reproduce the vchan install failure locally; I'll get it sorted.

@yomimono
Copy link
Copy Markdown
Contributor Author

yomimono commented Dec 14, 2016

.5 still broken after I fixed vchan, and the reason looks more serious:

# File "src/dog_listen.ml", line 98, characters 2-41:
# Error: Unbound value Irmin_unix.install_dir_polling_listener
# Command exited with code 2.

@samoht, maybe you know why?

@samoht
Copy link
Copy Markdown
Member

samoht commented Dec 14, 2016

dog is not supposed to work with irmin 0.12, I guess I forgot to fix that dependency

@yomimono
Copy link
Copy Markdown
Contributor Author

@samoht is it OK to merge this, as without it the branch is more broken?

@avsm
Copy link
Copy Markdown
Member

avsm commented Dec 16, 2016

I'm going ahead and merging this to unblock things

@avsm avsm merged commit a944909 into mirage:mirage-dev Dec 16, 2016
@avsm
Copy link
Copy Markdown
Member

avsm commented Dec 16, 2016

PR to upper bound dog.0.2.1 in ocaml/opam-repository#8115

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