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 packages from c-cube/qcheck at 0.18.1 #20636

Conversation

c-cube
Copy link
Contributor

@c-cube c-cube commented Feb 3, 2022

This pull-request concerns:
-qcheck.0.18.1: Compatibility package for qcheck
-qcheck-alcotest.0.18.1: Alcotest backend for qcheck
-qcheck-core.0.18.1: Core qcheck library
-qcheck-ounit.0.18.1: OUnit backend for qcheck



🐫 Pull-request generated by opam-publish v2.1.0

@mseri
Copy link
Member

mseri commented Feb 3, 2022


#=== ERROR while compiling qcheck-core.0.18.1 =================================#
# context              2.0.10 | linux/x86_64 | ocaml-base-compiler.4.13.1 | file:///home/opam/opam-repository
# path                 ~/.opam/4.13/.opam-switch/build/qcheck-core.0.18.1
# command              ~/.opam/opam-init/hooks/sandbox.sh build dune build -p qcheck-core -j 31
# exit-code            1
# env-file             ~/.opam/log/qcheck-core-51-de96f2.env
# output-file          ~/.opam/log/qcheck-core-51-de96f2.out
### output ###
# File "src/ppx_deriving_qcheck/dune", line 3, characters 14-33:
# 3 |  (public_name ppx_deriving_qcheck)
#                   ^^^^^^^^^^^^^^^^^^^
# Error: The current scope doesn't define package "ppx_deriving_qcheck".
# The only packages for which you can declare elements to be installed in this
# directory are:
# - qcheck          (because of qcheck.opam)
# - qcheck-alcotest (because of qcheck-alcotest.opam)
# - qcheck-core     (because of qcheck-core.opam)
# - qcheck-ounit    (because of qcheck-ounit.opam)

@c-cube c-cube force-pushed the opam-publish-qcheck-qcheck-alcotest-qcheck-core-qcheck-ounit.0.18.1 branch from fe272e1 to 7b6048d Compare February 3, 2022 15:21
@mseri
Copy link
Member

mseri commented Feb 3, 2022

The checksum needs to be updated: - qcheck-core.0.18.1: Bad checksum

@c-cube c-cube force-pushed the opam-publish-qcheck-qcheck-alcotest-qcheck-core-qcheck-ounit.0.18.1 branch from 7b6048d to 2bfdd11 Compare February 3, 2022 17:31
@mseri
Copy link
Member

mseri commented Feb 3, 2022

I think this failure is safe to ignore, do you agree?

#=== ERROR while compiling qcheck-ounit.0.18.1 ================================#
# context              2.1.2 | linux/x86_32 | ocaml-base-compiler.4.13.1 | pinned(https://github.com/c-cube/qcheck/archive/v0.18.1.tar.gz)
# path                 ~/.opam/4.13/.opam-switch/build/qcheck-ounit.0.18.1
# command              ~/.opam/opam-init/hooks/sandbox.sh build dune runtest -p qcheck-ounit -j 31
# exit-code            1
# env-file             ~/.opam/log/qcheck-ounit-36-f6b706.env
# output-file          ~/.opam/log/qcheck-ounit-36-f6b706.out
### output ###
#     ocamlopt example/ounit/QCheck_ounit_test.exe
# /usr/bin/ld: example/ounit/.QCheck_ounit_test.eobjs/native/dune__exe__QCheck_ounit_test.o: warning: relocation in read-only section `.text'
# /usr/bin/ld: warning: creating DT_TEXTREL in a PIE
# File "example/ounit/output.txt.expected", line 1, characters 0-0:
#          git (internal) (exit 1)
# (cd _build/default && /usr/bin/git --no-pager diff --no-index --color=always -u example/ounit/output.txt.expected example/ounit/output.txt)
# diff --git a/example/ounit/output.txt.expected b/example/ounit/output.txt
# index cb6e442..26f4b3f 100644
# --- a/example/ounit/output.txt.expected
# +++ b/example/ounit/output.txt
# @@ -6,7 +6,7 @@ Error: tests:2:error_raise_exn (in the log).
#  
#  
#  test `error_raise_exn` raised exception `Dune__exe__QCheck_ounit_test.Error`
# -on `0 (after 63 shrink steps)`
# +on `0 (after 31 shrink steps)`
#  
#  ------------------------------------------------------------------------------

@c-cube
Copy link
Contributor Author

c-cube commented Feb 3, 2022

Ah, on 32 bits, I think it's safe to ignore indeed. Expect tests can be hard.

@mseri mseri merged commit bd6bdfd into ocaml:master Feb 3, 2022
@c-cube c-cube deleted the opam-publish-qcheck-qcheck-alcotest-qcheck-core-qcheck-ounit.0.18.1 branch February 3, 2022 19:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants