Skip to content

Conversation

@samoht
Copy link
Member

@samoht samoht commented Dec 26, 2016

Reverts #8155

@samoht samoht merged commit 056fb4d into master Dec 26, 2016
@samoht samoht deleted the revert-8155-ocamlbuild.0.10.0 branch December 26, 2016 16:29
@camelus
Copy link
Contributor

camelus commented Dec 26, 2016

✅ No new or changed opam files 37f9c2e

✅ Installability check (5893 → 5892)
  • removed installable packages (1): ocamlbuild.0.10.0

@samoht
Copy link
Member Author

samoht commented Dec 26, 2016

Reverting as it breaks mtime builds:

#=== ERROR while installing mtime.0.8.3 =======================================#
# opam-version 1.2.2
# os           linux
# command      ocaml pkg/build.ml native=true native-dynlink=true jsoo=false
# path         /home/travis/.opam/4.03.0/build/mtime.0.8.3
# compiler     4.03.0
# exit-code    10
# env-file     /home/travis/.opam/4.03.0/build/mtime.0.8.3/mtime-24235-865888.env
# stdout-file  /home/travis/.opam/4.03.0/build/mtime.0.8.3/mtime-24235-865888.out
# stderr-file  /home/travis/.opam/4.03.0/build/mtime.0.8.3/mtime-24235-865888.err
### stdout ###
# [...]
# ocamlfind ocamlopt -a -cclib -lmtime_stubs -cclib -lrt src-os/mtime_base.cmx src-os/mtime.cmx -o src-os/mtime.cmxa
# ocamlfind ocamlc -c -g -bin-annot -safe-string -I src-os -o src-os/mtime.cmo src-os/mtime.ml
# ocamlfind ocamlc -a -dllib -lmtime_stubs -cclib -lrt src-os/mtime_base.cmo src-os/mtime.cmo -o src-os/mtime.cma
# ocamlfind ocamlopt -shared -linkall src-os/libmtime_stubs.a src-os/mtime.cmxa -o src-os/mtime.cmxs
# + ocamlfind ocamlopt -shared -linkall src-os/libmtime_stubs.a src-os/mtime.cmxa -o src-os/mtime.cmxs
# /usr/bin/ld: cannot find -lmtime_stubs
# collect2: ld returned 1 exit status
# File "caml_startup", line 1:
# Error: Error during linking
# Command exited with code 2.

I reproduced it locally on 4.03, need to investigate a bit more. /cc @gasche and @dbuenzli

@samoht
Copy link
Member Author

samoht commented Dec 26, 2016

Same issue on 4.04.

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 this pull request may close these issues.

3 participants