search

janestreet / sexplib

Automated S-expression conversion
MIT License
147 stars 27 forks source link

Sexplib 112.06.00 fails to install with camlp4 .4.02.0 #13

Closed Emm closed 9 years ago

Emm commented 9 years ago 
# ...[truncated]
# /home/emm/.opam/toml/bin/ocamlfind ocamldep -ppopt -DOCAML_4_02 -syntax camlp4o -syntax camlp4o -package camlp4.macro -package bigarray -modules lib/conv.ml > lib/conv.ml.depends
# + /home/emm/.opam/toml/bin/ocamlfind ocamldep -ppopt -DOCAML_4_02 -syntax camlp4o -syntax camlp4o -package camlp4.macro -package bigarray -modules lib/conv.ml > lib/conv.ml.depends
# File "lib/conv.ml", line 214, characters 4-5:
# Parse error: [match_case0] expected (in [match_case])
# File "lib/conv.ml", line 1:
# Error: Error while running external preprocessor
# Command line: camlp4 '-I' '/home/emm/.opam/toml/lib/ocaml/camlp4' '-I' '/home/emm/.opam/toml/lib/ocaml/camlp4' '-parser' 'o' '-parser' 'op' '-printer' 'p' '-parser' 'Camlp4MacroParser' '-DOCAML_4_02' 'lib/conv.ml' > /tmp/ocamlppe43f32
# 
# Command exited with code 2.
# Makefile:21: recipe for target 'build' failed
ghost commented 9 years ago

It should work with camlp4 4.02.0+1

Emm commented 9 years ago

It does. The opam dependencies should be fixed, though.

ghost commented 9 years ago

The opam dependency is fixed. You might want to look at the comment in ocaml/opam-repository#3510 though.

Emm commented 9 years ago

@diml, that's exactly what happened. Didn't know this compiler version had been removed.