janestreet / sexplib

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

Compilation fails with ocaml 4.03.0+trunk (sexplib.113.00.00) #17

Closed smondet closed 7 years ago

smondet commented 8 years ago
+ /Users/me/.opam/4.03.0+trunk/bin/ocamlfind ocamlc -c -g -annot -bin-annot -package bigarray -I lib -o lib/conv.cmo lib/conv.ml
File "lib/conv.ml", line 95, characters 12-25:
Warning 3: deprecated: String.create
Use Bytes.create instead.
File "lib/conv.ml", line 96, characters 24-43:
Warning 3: deprecated: String.set
Use Bytes.set instead.
File "lib/conv.ml", line 187, characters 30-34:
Error: This expression has type Obj.t but an expression was expected of type
         extension_constructor
Command exited with code 2.
ghost commented 8 years ago

Yes, a lot of our code doesn't compile with trunk. Hopefully we should be able to push trunk-following branches soon.

avsm commented 8 years ago

@diml any chance of a pull req constraining the appropriate Core packages in OPAM to <4.03.0? would really help with automated testing.

ghost commented 8 years ago

@avsm, I'll add the constraint to all versions of all JS packages. This error is likely to affect all versions of sexplib and most of our packages depend on it anyway...

avsm commented 8 years ago

sounds good to me.

dbuenzli commented 8 years ago

A lot of things unfortunately depend on this package. Would it be possible to have a fix and make a new release for 4.03. The same problem reported by @smondet persists.

trefis commented 8 years ago

We are currently preparing a new release of our libraries which will work with 4.03.

msprotz commented 8 years ago

@trefis if you get a chance to push just a commit that would fix this, that would help with my testing of opam-on-windows + 4.03.0 -- I'm willing to maintain a private repo for testing purposes that would point at the git master revision of sexplib in order to proceed with my testing

trefis commented 8 years ago

@msprotz the 4.03 branch already contain such a patch. I'm also hoping to get 4.03 compatible versions of our package into opam soon (cf. https://github.com/ocaml/opam-repository/pull/6094 )

msprotz commented 8 years ago

oh excellent I'll just wait until the pull request is accepted, then

FlorentBecker commented 7 years ago

What is the current status of this?

ghost commented 7 years ago

113.33.00+4.03 builds with 4.03 and 4.04.