search

ocaml-ppx / ppx_tools

Tools for authors of ppx rewriters
MIT License
134 stars 39 forks source link

5.0+4.02.3 opam package seems broken for ocaml 4.02.3 #63

Open picojulien opened 6 years ago

picojulien commented 6 years ago

Hi, I tried to install ppx_tools.5.0+4.02.3 with opam (which is requiried by js_of_ocaml 2.8) and I got the following error. 

#=== ERROR while installing ppx_tools.5.0+4.02.3 ==============================#
# opam-version 1.2.2
# os           linux
# command      make all
# path         /home/julien/.opam/4.02.3/build/ppx_tools.5.0+4.02.3
# compiler     4.02.3
# exit-code    2
# env-file     /home/julien/.opam/4.02.3/build/ppx_tools.5.0+4.02.3/ppx_tools-3392-a00279.env
# stdout-file  /home/julien/.opam/4.02.3/build/ppx_tools.5.0+4.02.3/ppx_tools-3392-a00279.out
# stderr-file  /home/julien/.opam/4.02.3/build/ppx_tools.5.0+4.02.3/ppx_tools-3392-a00279.err
### stdout ###
# ocamlc -bin-annot -w +A-4-17-44-45 -I +compiler-libs -safe-string -c ast_convenience.mli
# ocamlc -bin-annot -w +A-4-17-44-45 -I +compiler-libs -safe-string -c ast_convenience.ml
# ocamlc -bin-annot -w +A-4-17-44-45 -I +compiler-libs -safe-string -c ast_mapper_class.mli
# ocamlc -bin-annot -w +A-4-17-44-45 -I +compiler-libs -safe-string -c ast_mapper_class.ml
# Makefile:69 : la recette pour la cible « ast_mapper_class.cmo » a échouée
### stderr ###
# File "ast_mapper_class.ml", line 337, characters 6-17:
# Error: This variant pattern is expected to have type Parsetree.pattern_desc
#        The constructor Ppat_effect does not belong to type Parsetree.pattern_desc
# make: *** [ast_mapper_class.cmo] Erreur 2
voila commented 6 years ago

+1

ghowardsit commented 6 years ago

I'm seeing something similar.

File "ast_mapper_class.ml", line 41, characters 40-41: Error: this expression has type Parsetree.object_field list but an expression was expected of type ('a Parsetree.attributes Parsetree.core_type) list

skull-squadron commented 6 years ago

It's already fixed in master, but a release is needed.

ITMT hackaround: opam pin add ppx_tools https://github.com/ocaml-ppx/ppx_tools.git