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

4.07 support #47

Closed
copy opened this issue Jun 9, 2018 · 5 comments · Fixed by ocaml/opam-repository#12339
Closed

4.07 support #47

copy opened this issue Jun 9, 2018 · 5 comments · Fixed by ocaml/opam-repository#12339

Comments

@copy
Copy link

copy commented Jun 9, 2018

Currently, on 4.07.0+beta2+flambda:

jbuilder build @install
      ocamlc src/.base.objs/base__Formatter.{cmi,cmti} (exit 2)
(cd _build/default && /media/samsung/opam2/4.07.0+beta2+flambda/bin/ocamlc.opt -w -40 -g -bin-annot -I src/.base.objs -I /media/samsung/opam2/4.07.0+beta2+flambda/lib/sexplib0 -I compiler-stdlib/src/.caml.objs -I shadow-stdlib/src/.shadow_stdlib.objs -no-alias-deps -open Base__ -o src/.base.objs/base__Formatter.cmi -c -intf src/formatter.mli)
File "src/formatter.mli", line 9, characters 9-30:
Error: Unbound module Caml.Format
      ocamlc src/.base.objs/base__Import0.{cmi,cmo,cmt} (exit 2)
(cd _build/default && /media/samsung/opam2/4.07.0+beta2+flambda/bin/ocamlc.opt -w -40 -g -bin-annot -I src/.base.objs -I /media/samsung/opam2/4.07.0+beta2+flambda/lib/sexplib0 -I compiler-stdlib/src/.caml.objs -I shadow-stdlib/src/.shadow_stdlib.objs -no-alias-deps -open Base__ -o src/.base.objs/base__Import0.cmo -c -impl src/import0.ml)
File "src/import0.ml", line 7, characters 6-394:
Error: The signature constrained by `with' has no component named Pervasives
@ghost
Copy link

ghost commented Jun 11, 2018

Indeed, it needs a patch since the introduction of Stdlib. I believe @hhugo has a patch for this

@kit-ty-kate
Copy link
Contributor

Any news on that now that 4.07 has been released ?

@hhugo
Copy link
Contributor

hhugo commented Jul 11, 2018

Will look into that today

@ghost
Copy link

ghost commented Jul 12, 2018

This is fixed in the v0.11 branch. A similar fix will be pushed to master soon. We did a minor release of base: ocaml/opam-repository#12339

@copy
Copy link
Author

copy commented Jul 12, 2018

Thanks, can confirm that it's working. Follow-up: janestreet/core_kernel#84.

@copy copy closed this as completed Jul 12, 2018
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 a pull request may close this issue.

3 participants