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

Update to cohttp.0.99 #467

Merged
merged 3 commits into from Aug 25, 2017
Merged

Conversation

samoht
Copy link
Member

@samoht samoht commented Jul 25, 2017

No description provided.

@samoht
Copy link
Member Author

samoht commented Aug 2, 2017

#=== ERROR while installing conduit-lwt.1.0.0 =================================#
# opam-version 1.2.2
# os           linux
# command      jbuilder build -p conduit-lwt -j 4
# path         /home/travis/.opam/4.04.2/build/conduit-lwt.1.0.0
# compiler     4.04.2
# exit-code    1
# env-file     /home/travis/.opam/4.04.2/build/conduit-lwt.1.0.0/conduit-lwt-24904-5788a7.env
# stdout-file  /home/travis/.opam/4.04.2/build/conduit-lwt.1.0.0/conduit-lwt-24904-5788a7.out
# stderr-file  /home/travis/.opam/4.04.2/build/conduit-lwt.1.0.0/conduit-lwt-24904-5788a7.err
### stderr ###
#     ocamlopt .ppx/ppx_sexp_conv/ppx.exe (exit 2)
# (cd _build/default && /home/travis/.opam/4.04.2/bin/ocamlopt.opt -o .ppx/ppx_sexp_conv/ppx.exe -I /home/travis/.opam/4.04.2/lib/base -I /home/travis/.opam/4.04.2/lib/base/caml -I /home/travis/.opam/4.04.2/lib/base/shadow_stdlib -I /home/travis/.opam/4.04.2/lib/ocaml-compiler-libs/common -I /home/travis/.opam/4.04.2/lib/ocaml-compiler-libs/shadow -I /home/travis/.opam/4.04.2/lib/ocaml-migrate-parsetree -I /home/travis/.opam/4.04.2/lib/ocaml-migrate-parsetree/driver-main -I /home/travis/.opam/4.04.2/lib/ocaml/compiler-libs -I /home/travis/.opam/4.04.2/lib/ppx_ast -I /home/travis/.opam/4.04.2/lib/ppx_core -I /home/travis/.opam/4.04.2/lib/ppx_driver -I /home/travis/.opam/4.04.2/lib/ppx_driver/print_diff -I /home/travis/.opam/4.04.2/lib/ppx_optcomp -I /home/travis/.opam/4.04.2/lib/ppx_sexp_conv -I /home/travis/.opam/4.04.2/lib/ppx_sexp_conv/expander -I /home/travis/.opam/4.04.2/lib/ppx_traverse_builtins -I /home/travis/.opam/4.04.2/lib/ppx_type_conv -I /home/travis/.opam/4.04.2/lib/result -I /home/travis/.opam/4.04.2/lib/sexplib/0 -I /home/travis/.opam/4.04.2/lib/stdio caml.cmxa shadow_stdlib.cmxa sexplib0.cmxa ocamlcommon.cmxa result.cmxa ocaml_common.cmxa ocaml_shadow.cmxa migrate_parsetree.cmxa base.cmxa ppx_ast.cmxa ppx_traverse_builtins.cmxa stdio.cmxa ppx_core.cmxa print_diff.cmxa ppx_optcomp.cmxa ppx_driver.cmxa ppx_sexp_conv_expander.cmxa ppx_type_conv.cmxa ppx_sexp_conv.cmxa migrate_parsetree_driver_main.cmxa)
# >> Fatal error: Asmlink.object_file_name: not found
# Fatal error: exception Misc.Fatal_error

@let-def this is using ocaml-migrate-parsetree 1.0.2. Any idea what is happening?

@samoht
Copy link
Member Author

samoht commented Aug 3, 2017

/cc @diml as well just in case that error means something to you :-)

@ghost
Copy link

ghost commented Aug 3, 2017

Looking at the compiler, this function seems to be called to resolve a .cmx or .cmxa file. All the files on the command line are cmxa coming from installed libraries, maybe some file didn't get installed for some reason?

@samoht
Copy link
Member Author

samoht commented Aug 3, 2017

Ha thanks. I guess there between install of conduit-lwt and re-installation of some ppx_ libraries:

removed   ppx_type_conv.v0.9.0
[ERROR] The compilation of conduit-lwt failed at "jbuilder build -p conduit-lwt

I'll make sure the right dependencies are there.

@samoht
Copy link
Member Author

samoht commented Aug 4, 2017

TravisCI errors have been fixed by ocaml/opam-repository#10011

Hum strange failure for irmin-unix tests on windows:

Fatal error: exception (Failure "Nocrypto_entropy_unix: no random device")

@hannesm any idea what went wrong and I could fix this?

@hannesm
Copy link
Member

hannesm commented Aug 4, 2017

well, see the code at https://github.com/mirleft/ocaml-nocrypto/blob/master/unix/nocrypto_entropy_unix.ml#L14 -- I've no clue about windows APIs and how to earn random numbers there... the current implementation attempts to open /dev/urandom and /dev/random (in that order), which seems to be unavailable on windows

@samoht
Copy link
Member Author

samoht commented Aug 25, 2017

Great a new error:

#=== ERROR while installing cohttp.0.99.0 =====================================#
# opam-version 1.2.2
# os           linux
# command      jbuilder build -p cohttp -j 4
# path         /home/travis/.opam/4.03.0/build/cohttp.0.99.0
# compiler     4.03.0
# exit-code    1
# env-file     /home/travis/.opam/4.03.0/build/cohttp.0.99.0/cohttp-23106-a0321b.env
# stdout-file  /home/travis/.opam/4.03.0/build/cohttp.0.99.0/cohttp-23106-a0321b.out
# stderr-file  /home/travis/.opam/4.03.0/build/cohttp.0.99.0/cohttp-23106-a0321b.err
### stderr ###
# [...]
#       ocamlc cohttp/src/cohttp__Link.{cmo,cmt} (exit 2)
# (cd _build/default && /home/travis/.opam/4.03.0/bin/ocamlc.opt -w -40 -g -bin-annot -I /home/travis/.opam/4.03.0/lib/base -I /home/travis/.opam/4.03.0/lib/base/caml -I /home/travis/.opam/4.03.0/lib/base/shadow_stdlib -I /home/travis/.opam/4.03.0/lib/base64 -I /home/travis/.opam/4.03.0/lib/bytes -I /home/travis/.opam/4.03.0/lib/fieldslib -I /home/travis/.opam/4.03.0/lib/ocaml -I /home/travis/.opam/4.03.0/lib/ppx_deriving -I /home/travis/.opam/4.03.0/lib/re -I /home/travis/.opam/4.03.0/lib/result -I /home/travis/.opam/4.03.0/lib/sexplib -I /home/travis/.opam/4.03.0/lib/sexplib/0 -I /home/travis/.opam/4.03.0/lib/stringext -I /home/travis/.opam/4.03.0/lib/uri -no-alias-deps -I cohttp/src -open Cohttp -o cohttp/src/cohttp__Link.cmo -c -impl cohttp/src/link.pp.ml)
# File "cohttp/src/link.ml", line 123, characters 10-11:
# Error: Multiple definition of the module name Fields.
#        Names must be unique in a given structure or signature.
#     ocamlopt cohttp/src/cohttp__Link.{cmx,o} (exit 2)
# (cd _build/default && /home/travis/.opam/4.03.0/bin/ocamlopt.opt -w -40 -g -I /home/travis/.opam/4.03.0/lib/base -I /home/travis/.opam/4.03.0/lib/base/caml -I /home/travis/.opam/4.03.0/lib/base/shadow_stdlib -I /home/travis/.opam/4.03.0/lib/base64 -I /home/travis/.opam/4.03.0/lib/bytes -I /home/travis/.opam/4.03.0/lib/fieldslib -I /home/travis/.opam/4.03.0/lib/ocaml -I /home/travis/.opam/4.03.0/lib/ppx_deriving -I /home/travis/.opam/4.03.0/lib/re -I /home/travis/.opam/4.03.0/lib/result -I /home/travis/.opam/4.03.0/lib/sexplib -I /home/travis/.opam/4.03.0/lib/sexplib/0 -I /home/travis/.opam/4.03.0/lib/stringext -I /home/travis/.opam/4.03.0/lib/uri -no-alias-deps -I cohttp/src -open Cohttp -o cohttp/src/cohttp__Link.cmx -c -impl cohttp/src/link.pp.ml)
# File "cohttp/src/link.ml", line 123, characters 10-11:
# Error: Multiple definition of the module name Fields.
#        Names must be unique in a given structure or signature.

@samoht samoht merged commit 49ae6d1 into mirage:master Aug 25, 2017
@samoht samoht deleted the update-conduit-and-cohttp branch August 25, 2017 15:05
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

2 participants