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

+angstrom.0.16.0 #24828

Merged
merged 3 commits into from
Dec 13, 2023
Merged

+angstrom.0.16.0 #24828

merged 3 commits into from
Dec 13, 2023

Conversation

seliopou
Copy link
Contributor

No description provided.

Copy link
Collaborator

@haochenx haochenx left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please also take a look at the failing lower-bounds checks, e.g.

(angstrom-async)

#=== ERROR while compiling angstrom-async.0.16.0 ==============================#
# context              2.2.0~alpha3 | linux/x86_64 | ocaml-base-compiler.4.08.1 | pinned(https://github.com/inhabitedtype/angstrom/archive/0.16.0.tar.gz)
# path                 ~/.opam/4.08/.opam-switch/build/angstrom-async.0.16.0
# command              ~/.opam/opam-init/hooks/sandbox.sh build dune build -p angstrom-async -j 71
# exit-code            1
# env-file             ~/.opam/log/angstrom-async-7-694e91.env
# output-file          ~/.opam/log/angstrom-async-7-694e91.out
### output ###
#       ocamlc async/.angstrom_async.objs/byte/angstrom_async.{cmo,cmt} (exit 2)
# (cd _build/default && /home/opam/.opam/4.08/bin/ocamlc.opt -w -40 -safe-string -g -bin-annot -I async/.angstrom_async.objs/byte -I /home/opam/.opam/4.08/lib/angstrom -I /home/opam/.opam/4.08/lib/async -I /home/opam/.opam/4.08/lib/async_extra -I /home/opam/.opam/4.08/lib/async_kernel -I /home/opam/.opam/4.08/lib/async_kernel/persistent_connection_kernel -I /home/opam/.opam/4.08/lib/async_rpc_kernel -I /home/opam/.opam/4.08/lib/async_unix -I /home/opam/.opam/4.08/lib/async_unix/thread_pool -I /home/opam/.opam/4.08/lib/async_unix/thread_safe_ivar -I /home/opam/.opam/4.08/lib/base -I /home/opam/.opam/4.08/lib/base/caml -I /home/opam/.opam/4.08/lib/base/md5 -I /home/opam/.opam/4.08/lib/base/shadow_stdlib -I /home/opam/.opam/4.08/lib/base_bigstring -I /home/opam/.opam/4.08/lib/base_quickcheck -I /home/opam/.opam/4.08/lib/bin_prot -I /home/opam/.opam/4.08/lib/bin_prot/shape -I /home/opam/.opam/4.08/lib/core -I /home/opam/.opam/4.08/lib/core/nano_mutex -I /home/opam/.opam/4.08/lib/core/squeue -I /home/opam/.opam/4.08/lib/core/uuid -I /home/opam/.opam/4.08/lib/core_kernel -I /home/opam/.opam/4.08/lib/core_kernel/base_for_tests -I /home/opam/.opam/4.08/lib/core_kernel/moption -I /home/opam/.opam/4.08/lib/core_kernel/thread_pool_cpu_affinity -I /home/opam/.opam/4.08/lib/core_kernel/thread_safe_queue -I /home/opam/.opam/4.08/lib/core_kernel/uopt -I /home/opam/.opam/4.08/lib/core_kernel/uuid -I /home/opam/.opam/4.08/lib/fieldslib -I /home/opam/.opam/4.08/lib/jane-street-headers -I /home/opam/.opam/4.08/lib/ocaml/threads -I /home/opam/.opam/4.08/lib/parsexp -I /home/opam/.opam/4.08/lib/ppx_assert/runtime-lib -I /home/opam/.opam/4.08/lib/ppx_bench/runtime-lib -I /home/opam/.opam/4.08/lib/ppx_compare/runtime-lib -I /home/opam/.opam/4.08/lib/ppx_enumerate/runtime-lib -I /home/opam/.opam/4.08/lib/ppx_expect/collector -I /home/opam/.opam/4.08/lib/ppx_expect/common -I /home/opam/.opam/4.08/lib/ppx_expect/config -I /home/opam/.opam/4.08/lib/ppx_hash/runtime-lib -I /home/opam/.opam/4.08/lib/ppx_inline_test/config -I /home/opam/.opam/4.08/lib/ppx_inline_test/runtime-lib -I /home/opam/.opam/4.08/lib/ppx_module_timer/runtime -I /home/opam/.opam/4.08/lib/ppx_sexp_conv/runtime-lib -I /home/opam/.opam/4.08/lib/protocol_version_header -I /home/opam/.opam/4.08/lib/result -I /home/opam/.opam/4.08/lib/sexplib -I /home/opam/.opam/4.08/lib/sexplib/unix -I /home/opam/.opam/4.08/lib/sexplib0 -I /home/opam/.opam/4.08/lib/spawn -I /home/opam/.opam/4.08/lib/splittable_random -I /home/opam/.opam/4.08/lib/stdio -I /home/opam/.opam/4.08/lib/time_now -I /home/opam/.opam/4.08/lib/typerep -I /home/opam/.opam/4.08/lib/variantslib -intf-suffix .ml -no-alias-deps -o async/.angstrom_async.objs/byte/angstrom_async.cmo -c -impl async/angstrom_async.ml)
# File "async/angstrom_async.ml", line 52, characters 14-24:
# 52 |     finalize (p.continue bigstring ~off:0 ~len:(String.length s) Complete) `Eof
#                    ^^^^^^^^^^
# Error: This function has type
#          Angstrom.bigstring ->
#          Angstrom.Unbuffered.more -> 'a Angstrom.Unbuffered.state
#        It is applied to too many arguments; maybe you forgot a `;'.
#     ocamlopt async/.angstrom_async.objs/native/angstrom_async.{cmx,o} (exit 2)
# (cd _build/default && /home/opam/.opam/4.08/bin/ocamlopt.opt -w -40 -safe-string -g -I async/.angstrom_async.objs/byte -I async/.angstrom_async.objs/native -I /home/opam/.opam/4.08/lib/angstrom -I /home/opam/.opam/4.08/lib/async -I /home/opam/.opam/4.08/lib/async_extra -I /home/opam/.opam/4.08/lib/async_kernel -I /home/opam/.opam/4.08/lib/async_kernel/persistent_connection_kernel -I /home/opam/.opam/4.08/lib/async_rpc_kernel -I /home/opam/.opam/4.08/lib/async_unix -I /home/opam/.opam/4.08/lib/async_unix/thread_pool -I /home/opam/.opam/4.08/lib/async_unix/thread_safe_ivar -I /home/opam/.opam/4.08/lib/base -I /home/opam/.opam/4.08/lib/base/caml -I /home/opam/.opam/4.08/lib/base/md5 -I /home/opam/.opam/4.08/lib/base/shadow_stdlib -I /home/opam/.opam/4.08/lib/base_bigstring -I /home/opam/.opam/4.08/lib/base_quickcheck -I /home/opam/.opam/4.08/lib/bin_prot -I /home/opam/.opam/4.08/lib/bin_prot/shape -I /home/opam/.opam/4.08/lib/core -I /home/opam/.opam/4.08/lib/core/nano_mutex -I /home/opam/.opam/4.08/lib/core/squeue -I /home/opam/.opam/4.08/lib/core/uuid -I /home/opam/.opam/4.08/lib/core_kernel -I /home/opam/.opam/4.08/lib/core_kernel/base_for_tests -I /home/opam/.opam/4.08/lib/core_kernel/moption -I /home/opam/.opam/4.08/lib/core_kernel/thread_pool_cpu_affinity -I /home/opam/.opam/4.08/lib/core_kernel/thread_safe_queue -I /home/opam/.opam/4.08/lib/core_kernel/uopt -I /home/opam/.opam/4.08/lib/core_kernel/uuid -I /home/opam/.opam/4.08/lib/fieldslib -I /home/opam/.opam/4.08/lib/jane-street-headers -I /home/opam/.opam/4.08/lib/ocaml/threads -I /home/opam/.opam/4.08/lib/parsexp -I /home/opam/.opam/4.08/lib/ppx_assert/runtime-lib -I /home/opam/.opam/4.08/lib/ppx_bench/runtime-lib -I /home/opam/.opam/4.08/lib/ppx_compare/runtime-lib -I /home/opam/.opam/4.08/lib/ppx_enumerate/runtime-lib -I /home/opam/.opam/4.08/lib/ppx_expect/collector -I /home/opam/.opam/4.08/lib/ppx_expect/common -I /home/opam/.opam/4.08/lib/ppx_expect/config -I /home/opam/.opam/4.08/lib/ppx_hash/runtime-lib -I /home/opam/.opam/4.08/lib/ppx_inline_test/config -I /home/opam/.opam/4.08/lib/ppx_inline_test/runtime-lib -I /home/opam/.opam/4.08/lib/ppx_module_timer/runtime -I /home/opam/.opam/4.08/lib/ppx_sexp_conv/runtime-lib -I /home/opam/.opam/4.08/lib/protocol_version_header -I /home/opam/.opam/4.08/lib/result -I /home/opam/.opam/4.08/lib/sexplib -I /home/opam/.opam/4.08/lib/sexplib/unix -I /home/opam/.opam/4.08/lib/sexplib0 -I /home/opam/.opam/4.08/lib/spawn -I /home/opam/.opam/4.08/lib/splittable_random -I /home/opam/.opam/4.08/lib/stdio -I /home/opam/.opam/4.08/lib/time_now -I /home/opam/.opam/4.08/lib/typerep -I /home/opam/.opam/4.08/lib/variantslib -intf-suffix .ml -no-alias-deps -o async/.angstrom_async.objs/native/angstrom_async.cmx -c -impl async/angstrom_async.ml)
# File "async/angstrom_async.ml", line 52, characters 14-24:
# 52 |     finalize (p.continue bigstring ~off:0 ~len:(String.length s) Complete) `Eof
#                    ^^^^^^^^^^
# Error: This function has type
#          Angstrom.bigstring ->
#          Angstrom.Unbuffered.more -> 'a Angstrom.Unbuffered.state
#        It is applied to too many arguments; maybe you forgot a `;'.

(angstrom-lwt-unix)

[ERROR] The compilation of cstruct.2.4.1 failed at "make".

#=== ERROR while compiling cstruct.2.4.1 ======================================#
# context              2.2.0~alpha3 | linux/x86_64 | ocaml-base-compiler.4.06.1 | file:///home/opam/opam-repository
# path                 ~/.opam/4.06/.opam-switch/build/cstruct.2.4.1
# command              ~/.opam/opam-init/hooks/sandbox.sh build make
# exit-code            2
# env-file             ~/.opam/log/cstruct-7-af8af0.env
# output-file          ~/.opam/log/cstruct-7-af8af0.out
### output ###
# ocaml setup.ml -build 
# ocamlfind ocamlopt unix.cmxa -I /home/opam/.opam/4.06/lib/ocamlbuild /home/opam/.opam/4.06/lib/ocamlbuild/ocamlbuildlib.cmxa -linkpkg myocamlbuild.ml /home/opam/.opam/4.06/lib/ocamlbuild/ocamlbuild.cmx -o myocamlbuild
# /home/opam/.opam/4.06/bin/ocamlfind ocamlc -g -ccopt -O3 -ccopt -Wall -package sexplib -package ocplib-endian.bigstring -package ocplib-endian -package bytes -package bigarray -c lib/cstruct_stubs.c
# mv cstruct_stubs.o lib/cstruct_stubs.o
# /home/opam/.opam/4.06/bin/ocamlfind ocamlmklib -o lib/cstruct_stubs lib/cstruct_stubs.o
# /home/opam/.opam/4.06/bin/ocamlfind ocamldep -package sexplib -package ocplib-endian.bigstring -package ocplib-endian -package bytes -package bigarray -modules lib/cstruct.mli > lib/cstruct.mli.depends
# /home/opam/.opam/4.06/bin/ocamlfind ocamlc -c -g -annot -bin-annot -principal -ccopt -O3 -ccopt -Wall -package sexplib -package ocplib-endian.bigstring -package ocplib-endian -package bytes -package bigarray -I lib -o lib/cstruct.cmi lib/cstruct.mli
# /home/opam/.opam/4.06/bin/ocamlfind ocamldep -package sexplib -package ocplib-endian.bigstring -package ocplib-endian -package bytes -package bigarray -modules lib/cstruct.ml > lib/cstruct.ml.depends
# /home/opam/.opam/4.06/bin/ocamlfind ocamlc -c -g -annot -bin-annot -principal -ccopt -O3 -ccopt -Wall -package sexplib -package ocplib-endian.bigstring -package ocplib-endian -package bytes -package bigarray -I lib -o lib/cstruct.cmo lib/cstruct.ml
# + /home/opam/.opam/4.06/bin/ocamlfind ocamlc -c -g -annot -bin-annot -principal -ccopt -O3 -ccopt -Wall -package sexplib -package ocplib-endian.bigstring -package ocplib-endian -package bytes -package bigarray -I lib -o lib/cstruct.cmo lib/cstruct.ml
# File "lib/cstruct.ml", line 154, characters 0-138:
# Warning 3: deprecated: [@@noalloc] should be used instead of "noalloc"
# File "lib/cstruct.ml", line 156, characters 0-132:
# Warning 3: deprecated: [@@noalloc] should be used instead of "noalloc"
# File "lib/cstruct.ml", line 158, characters 0-132:
# Warning 3: deprecated: [@@noalloc] should be used instead of "noalloc"
# File "lib/cstruct.ml", line 160, characters 0-132:
# Warning 3: deprecated: [@@noalloc] should be used instead of "noalloc"
# File "lib/cstruct.ml", line 162, characters 0-132:
# Warning 3: deprecated: [@@noalloc] should be used instead of "noalloc"
# File "lib/cstruct.ml", line 164, characters 0-117:
# Warning 3: deprecated: [@@noalloc] should be used instead of "noalloc"
# File "lib/cstruct.ml", line 166, characters 0-102:
# Warning 3: deprecated: [@@noalloc] should be used instead of "noalloc"
# /home/opam/.opam/4.06/bin/ocamlfind ocamlc -a -dllib -lcstruct_stubs -cclib -lcstruct_stubs lib/cstruct.cmo -o lib/cstruct.cma
# /home/opam/.opam/4.06/bin/ocamlfind ocamlopt -c -g -annot -bin-annot -principal -ccopt -O3 -ccopt -Wall -package sexplib -package ocplib-endian.bigstring -package ocplib-endian -package bytes -package bigarray -I lib -o lib/cstruct.cmx lib/cstruct.ml
# + /home/opam/.opam/4.06/bin/ocamlfind ocamlopt -c -g -annot -bin-annot -principal -ccopt -O3 -ccopt -Wall -package sexplib -package ocplib-endian.bigstring -package ocplib-endian -package bytes -package bigarray -I lib -o lib/cstruct.cmx lib/cstruct.ml
# File "lib/cstruct.ml", line 154, characters 0-138:
# Warning 3: deprecated: [@@noalloc] should be used instead of "noalloc"
# File "lib/cstruct.ml", line 156, characters 0-132:
# Warning 3: deprecated: [@@noalloc] should be used instead of "noalloc"
# File "lib/cstruct.ml", line 158, characters 0-132:
# Warning 3: deprecated: [@@noalloc] should be used instead of "noalloc"
# File "lib/cstruct.ml", line 160, characters 0-132:
# Warning 3: deprecated: [@@noalloc] should be used instead of "noalloc"
# File "lib/cstruct.ml", line 162, characters 0-132:
# Warning 3: deprecated: [@@noalloc] should be used instead of "noalloc"
# File "lib/cstruct.ml", line 164, characters 0-117:
# Warning 3: deprecated: [@@noalloc] should be used instead of "noalloc"
# File "lib/cstruct.ml", line 166, characters 0-102:
# Warning 3: deprecated: [@@noalloc] should be used instead of "noalloc"
# /home/opam/.opam/4.06/bin/ocamlfind ocamlopt -a -cclib -lcstruct_stubs lib/cstruct.cmx -o lib/cstruct.cmxa
# /home/opam/.opam/4.06/bin/ocamlfind ocamlopt -shared -I lib/ lib/cstruct.cmxa lib/cstruct.cmx -o lib/cstruct.cmxs
# /home/opam/.opam/4.06/bin/ocamlfind ocamldep -package sexplib -package ocplib-endian.bigstring -package ocplib-endian -package bytes -package bigarray -package lwt.unix -modules lwt/lwt_cstruct.mli > lwt/lwt_cstruct.mli.depends
# /home/opam/.opam/4.06/bin/ocamlfind ocamlc -c -g -annot -bin-annot -principal -I lib -package sexplib -package ocplib-endian.bigstring -package ocplib-endian -package bytes -package bigarray -package lwt.unix -I lwt -I lib -o lwt/lwt_cstruct.cmi lwt/lwt_cstruct.mli
# + /home/opam/.opam/4.06/bin/ocamlfind ocamlc -c -g -annot -bin-annot -principal -I lib -package sexplib -package ocplib-endian.bigstring -package ocplib-endian -package bytes -package bigarray -package lwt.unix -I lwt -I lib -o lwt/lwt_cstruct.cmi lwt/lwt_cstruct.mli
# ocamlfind: Error from package `threads': Missing -thread or -vmthread switch
# Command exited with code 2.
# E: Failure("Command ''/home/opam/.opam/4.06/bin/ocamlbuild' lib/libcstruct_stubs.a lib/dllcstruct_stubs.so lib/cstruct.cma lib/cstruct.cmxa lib/cstruct.a lib/cstruct.cmxs lwt/lwt_cstruct.cma lwt/lwt_cstruct.cmxa lwt/lwt_cstruct.a lwt/lwt_cstruct.cmxs unix/unix_cstruct.cma unix/unix_cstruct.cmxa unix/unix_cstruct.a unix/unix_cstruct.cmxs ppx/ppx_cstruct.cmx ppx/ppx_cstruct.cmo ppx/standalone.native -use-ocamlfind -tag debug' terminated with error code 10")
# make: *** [Makefile:7: build] Error 1

packages/angstrom-async/angstrom-async.0.16.0/opam Outdated Show resolved Hide resolved
packages/angstrom-lwt-unix/angstrom-lwt-unix.0.16.0/opam Outdated Show resolved Hide resolved
packages/angstrom-unix/angstrom-unix.0.16.0/opam Outdated Show resolved Hide resolved
packages/angstrom/angstrom.0.16.0/opam Outdated Show resolved Hide resolved
@haochenx
Copy link
Collaborator

haochenx commented Nov 26, 2023

This PR also (rightfully?) removes dependency to result, which seems to cause a few revdeps (which is not this PR's fault in principle: they should have declared dependency on result themselves.)

We probably want to patch those packages as well (but probably not in this PR.)

Edit: issue filed: #24853

Co-authored-by: Haochen M. Kotoi-Xie <hx@kxc.inc>
@mseri
Copy link
Member

mseri commented Dec 13, 2023

The lower bounds are fixed, the rest of the failures are unrelated. @seliopou can you update the opam file for angstrom-async in your repo?

@mseri mseri merged commit 947b108 into ocaml:master Dec 13, 2023
1 of 2 checks passed
seliopou added a commit to inhabitedtype/angstrom that referenced this pull request Jan 5, 2024
@seliopou seliopou deleted the angstrom.0.16.0 branch January 5, 2024 02:25
seliopou added a commit to inhabitedtype/angstrom that referenced this pull request Sep 11, 2024
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