PR#1: Enable tests - rpms/ocaml-sexplib

rpms / ocaml-sexplib

#1 Enable tests

Opened 3 years ago by defolos. Modified 3 years ago

rpms/ defolos/ocaml-sexplib master into rawhide

Dan Čermák • 3 years ago

7b5161c

ocaml-sexplib.spec

file modified

		`@@ -66,6 +66,9 @@`
		`find %{buildroot}%{_libdir}/ocaml -name \*.cmxs -exec chmod 0755 {} \+`
		`%endif`

		`+ %check`
		`+ dune test`
		`+`
		`%files`
		`%doc CHANGES.md README.org`
		`%license COPYRIGHT.txt LICENSE.md LICENSE-Tywith.txt THIRD-PARTY.txt`

defolos commented 3 years ago

Depends on the update of parsexp and sexplib0.

rjones commented 3 years ago

Seems OK (albeit fails CI for some reason). Did you want me to do anything?

defolos commented 3 years ago

The CI fails because the updates for parsexp and sexplib0 are not yet in Rawhide.

defolos commented 3 years ago

So please don't merge this yet, as I don't even know if the tests can be run.

rebased onto 7b5161c

3 years ago

defolos commented 3 years ago

Jerry already updated sexplib, the only thing left from this PR is the test enablement.

jjames commented 3 years ago

I see 0/0 tests run when I try this. It looks like the tests require base_quickcheck, core_kernel, expect_test_helpers_core, and sexp_grammar_validation, none of which we have in Fedora.