Skip to content

Commit

Permalink
Extend the test suite (with coq 8.12.1 and coq-native)
Browse files Browse the repository at this point in the history
* Follow-up of math-comp@52988b6

* Now possible after the merge of coq/opam#1835
  • Loading branch information
erikmd authored and thery committed Jan 3, 2023
1 parent 4f18dc2 commit 4cc5391
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 6 deletions.
4 changes: 1 addition & 3 deletions .github/workflows/ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -35,9 +35,7 @@ jobs:
. configure # sourcing mandatory
coq_version
endGroup
# Note: Replace with "if coq_native_compiler"
# to also test with Coq 8.12.1+
if coq_native_compiler_default; then
if coq_native_compiler; then
startGroup "Workaround permission issue"
sudo chown -R coq:coq . # <--(§)
endGroup
Expand Down
4 changes: 1 addition & 3 deletions configure
Original file line number Diff line number Diff line change
Expand Up @@ -36,9 +36,7 @@ coq_native_compiler() {
}

main() {
# Note: Replace with "if coq_native_compiler"
# to also test with Coq 8.12.1+
if coq_native_compiler_default; then
if coq_native_compiler; then
echo 'coq-native support enabled!' >&2
sed -e 's/;coq-native-disabled; \?//' "$dir/src/dune.in" > "$dir/src/dune"
else
Expand Down

0 comments on commit 4cc5391

Please sign in to comment.