ocaml / oasis

Cabal like system for OCaml
Other
125 stars 45 forks source link

Build broken with 4.03.0+trunk #313

Closed gildor478 closed 3 years ago

gildor478 commented 3 years ago

This bug has been migrated from artifact #1519 on forge.ocamlcore.org. It was assigned to user102. It was closed on 2016-04-29 07:24:19.

user21778 posted on 2015-12-10 20:21:40:

Here is the extract of the error log:

File "/Users/sebastienmondet/.opam/4.03.0+trunk/lib/ocaml/ocamlbuild/ocamlbuildlib.cma(Ocamlbuild_pack)", line 1: Warning 31: files /Users/sebastienmondet/.opam/4.03.0+trunk/lib/ocaml/ocamlbuild/ocamlbuildlib.cma(Ocamlbuild_pack) and /Users/sebastienmondet/.opam/4.03.0+trunk/lib/ocaml/ocamlbuild/ocamlbuildlib.cma(Ocamlbuild_pack) both define a module named Ocamlbuild_pack File "none", line 1: Error: Some fatal warnings were triggered (1 occurrences) Command exited with code 2.

user102 replied on 2015-12-10 21:31:52:

This error means that EVERY project shipping with the myocamlbuild version generated by OASIS will break !

As of OCaml 4.02 String.uncapitalize_ascii doesn't exist, so it also means that ALL project will have to depend on OCaml >= 4.03.

I would expect to have a few version of OCaml with both uncapitalize and uncapitalize_ascii, rather than a drastic change...

user21778 replied on 2015-12-10 21:38:44:

Both uncapitalize and uncapitalize_ascii are there, but the former is just deprecated (with the [@@deprecated] attribute).

The problem here seems to be compiling with some warning turned into an error.

(BTW, I would have guessed the problem is with warning 31 and not warning 3)

user21778 replied on 2015-12-10 21:41:06:

Confirming that the error comes from warining 31.

In the logs I see that other files compiled fine with the String.capitalize.

The error happens when linking src/cli/Main.byte with ocamlc ... -o ....

user102 replied on 2015-12-10 23:07:32:

ocamlbuild.cma is probably included twice (for an unknown reason).

Can you give the full command line when compiling Main.byte ?

user21778 replied on 2015-12-10 23:12:14:

Here is what I have:

user21778 replied on 2015-12-10 23:13:51:

Attaching the whole logs form opam

user102 replied on 2015-12-11 00:57:28:

Here is the duplicate: -I +ocamlbuild ocamlbuildlib.cma and -package ocamlbuild

I suppose one of them should be removed.

user112 replied on 2016-04-21 18:06:15:

Is there an immediate plan for fixing oasis itself under 4.03.0? It's supposed to be released soon.

user102 replied on 2016-04-21 19:32:22:

Now that I have access to travis + OPAM build all this is a lot more easy to test and fix.

Will fix it for next release.

user923 replied on 2016-04-26 13:55:25:

Would it be possible to have a release reasonably quickly ?

user102 replied on 2016-04-26 14:37:37:

Yes, but I have no clue how I can make it backward compatible with OCaml < 4.03.00 (which is a must have).

gildor478 commented 3 years ago

oasis-2938-384e1f.out.txt oasis-2938-384e1f.err.txt