Skip to content

Update charrua-{core,unix} to version 0.3.#5990

Merged
samoht merged 1 commit intoocaml:masterfrom
haesbaert:master
Apr 2, 2016
Merged

Update charrua-{core,unix} to version 0.3.#5990
samoht merged 1 commit intoocaml:masterfrom
haesbaert:master

Conversation

@haesbaert
Copy link
Copy Markdown
Contributor

@haesbaert haesbaert commented Mar 14, 2016

No description provided.

@camelus
Copy link
Copy Markdown
Contributor

camelus commented Mar 14, 2016

✅ All lint checks passed 4dcaa67
  • These packages passed lint tests: charrua-core.0.3, charrua-unix.0.3
✅ Installability check (4742 → 4744)
  • new installable packages (2): charrua-core.0.3 charrua-unix.0.3

@hannesm
Copy link
Copy Markdown
Member

hannesm commented Mar 17, 2016

the unix name should be charrua-unix.0.3 (., not - before 0) to make opam happy

@samoht
Copy link
Copy Markdown
Member

samoht commented Mar 20, 2016 

#=== ERROR while installing charrua-core.0.3 ==================================#
# opam-version 1.2.2
# os           linux
# command      sh build.sh
# path         /home/travis/.opam/system/build/charrua-core.0.3
# compiler     system (4.02.3)
# exit-code    10
# env-file     /home/travis/.opam/system/build/charrua-core.0.3/charrua-core-8605-805f32.env
# stdout-file  /home/travis/.opam/system/build/charrua-core.0.3/charrua-core-8605-805f32.out
# stderr-file  /home/travis/.opam/system/build/charrua-core.0.3/charrua-core-8605-805f32.err
### stdout ###
# ocamlfind ocamlopt unix.cmxa -I /usr/lib/ocaml/ocamlbuild /usr/lib/ocaml/ocamlbuild/ocamlbuildlib.cmxa -linkpkg myocamlbuild.ml /usr/lib/ocaml/ocamlbuild/ocamlbuild.cmx -o myocamlbuild
# ocamlfind ocamldep -package tcpip -package sexplib -package sexplib.syntax -package cstruct.unix -package ipaddr -package cstruct -package cstruct.syntax -syntax camlp4o -syntax pa_cstruct -modules lib/dhcp_wire.mli > lib/dhcp_wire.mli.depends
# + ocamlfind ocamldep -package tcpip -package sexplib -package sexplib.syntax -package cstruct.unix -package ipaddr -package cstruct -package cstruct.syntax -syntax camlp4o -syntax pa_cstruct -modules lib/dhcp_wire.mli > lib/dhcp_wire.mli.depends
# ocamlfind: Package `sexplib.syntax' not found
# Command exited with code 2.

@samoht
Copy link
Copy Markdown
Member

samoht commented Mar 21, 2016

@haesbaert there's a dependency to camlp4 which is missing.

@haesbaert
Copy link
Copy Markdown
Contributor Author

haesbaert commented Mar 21, 2016

Thanks, I was out this weekend, will have a look tonight.

On Monday, 21 March 2016, Thomas Gazagnaire notifications@github.com
wrote:

@haesbaert https://github.com/haesbaert there's a dependency to camlp4
which is missing.

—
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub
#5990 (comment)

@hannesm
Copy link
Copy Markdown
Member

hannesm commented Mar 21, 2016

🍷 :D

@samoht
Copy link
Copy Markdown
Member

samoht commented Apr 1, 2016

@haesbaert ping?

@haesbaert
Copy link
Copy Markdown
Contributor Author

haesbaert commented Apr 1, 2016

Sorry, though week, will sort it all tonight.

On Friday, 1 April 2016, Thomas Gazagnaire notifications@github.com wrote:

@haesbaert https://github.com/haesbaert ping?

—
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub
#5990 (comment)

@haesbaert haesbaert force-pushed the master branch 6 times, most recently from ce05b8b to 1813748 Compare April 2, 2016 09:21
@hannesm
Copy link
Copy Markdown
Member

hannesm commented Apr 2, 2016

travis is happy now with christianos recent commit... 👍

@samoht samoht merged commit 92a0e73 into ocaml:master Apr 2, 2016
@samoht
Copy link
Copy Markdown
Member

samoht commented Apr 2, 2016

Great, thanks!

@hannesm hannesm mentioned this pull request Apr 4, 2016
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@haesbaert @camelus @hannesm @samoht