search

gregtatcam / imaplet-lwt

imaplet revamp with Lwt
19 stars 6 forks source link

opam: need upper bound on sexplib (or switch to ppx) #11

Closed djs55 closed 8 years ago

djs55 commented 8 years ago

I saw the following build failure:

#=== ERROR while installing imaplet-lwt.0.1.15 ================================#
# opam-version         1.2.2 (2dee5d69c0bd29731e6cb2ee537f0adf72404ba4)
# os                   linux
# command              ./configure --prefix=/home/opam/.opam/4.02.3
# path                 /home/opam/.opam/4.02.3/build/imaplet-lwt.0.1.15
# compiler             4.02.3
# exit-code            1
# env-file             /home/opam/.opam/4.02.3/build/imaplet-lwt.0.1.15/imaplet-lwt-12895-50de41.env
# stdout-file          /home/opam/.opam/4.02.3/build/imaplet-lwt.0.1.15/imaplet-lwt-12895-50de41.out
# stderr-file          /home/opam/.opam/4.02.3/build/imaplet-lwt.0.1.15/imaplet-lwt-12895-50de41.err
### stderr ###
# [...]
# W: Failure("Command ''/home/opam/.opam/4.02.3/bin/ocamlfind' query -format %d sexplib.syntax > '/tmp/oasis-e22241.txt'' terminated with error code 2")
# ocamlfind: Package `sexplib.syntax' not found
# W: Failure("Command ''/home/opam/.opam/4.02.3/bin/ocamlfind' query -format %d sexplib.syntax > '/tmp/oasis-ab1e32.txt'' terminated with error code 2")
# ocamlfind: Package `sexplib.syntax' not found
# W: Failure("Command ''/home/opam/.opam/4.02.3/bin/ocamlfind' query -format %d sexplib.syntax > '/tmp/oasis-88df6b.txt'' terminated with error code 2")
# ocamlfind: Package `sexplib.syntax' not found
# W: Failure("Command ''/home/opam/.opam/4.02.3/bin/ocamlfind' query -format %d sexplib.syntax > '/tmp/oasis-2c6158.txt'' terminated with error code 2")
# E: Cannot find findlib package fieldslib.syntax
# E: Cannot find findlib package sexplib.syntax
# E: Failure("2 configuration errors")

From https://travis-ci.org/mirage/ocaml-dns/jobs/123727125

I believe recent releases of sexplib and fieldslib are ppx-only, so an upper-bound is probably required. Probably adding camlp4 as an explicit dependency would also be a good idea.

gregtatcam commented 8 years ago

Thanks, I added upper-bound on sexplib and fieldslib. I created the pull request on opam repository...

gregtatcam commented 8 years ago

opam update has been merged.