Skip to content

Commit cba7d0a

Browse files
committed
Cleanup after dropping 8.20
1 parent 96db1b4 commit cba7d0a

File tree

97 files changed

+177
-689
lines changed

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

97 files changed

+177
-689
lines changed

.github/workflows/ci.yml

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -20,11 +20,10 @@ jobs:
2020
- 'rocq/rocq-prover:dev'
2121
- 'rocq/rocq-prover:9.1'
2222
- 'rocq/rocq-prover:9.0'
23-
- 'coqorg/coq:8.20'
2423
profile:
2524
- dev
2625
include:
27-
- image: 'coqorg/coq:8.20'
26+
- image: 'rocq/rocq-prover:9.0'
2827
profile: fatalwarnings
2928
fail-fast: false # don't stop jobs if one fails
3029
steps:

Makefile

Lines changed: 12 additions & 34 deletions
Original file line numberDiff line numberDiff line change
@@ -1,59 +1,37 @@
11
dune_wrap = $(shell command -v coqc > /dev/null || echo "etc/with-rocq-wrap.sh")
22
dune = $(dune_wrap) dune $(1) $(DUNE_$(1)_FLAGS) --stop-on-first-error
3-
DUNE_IN_FILES = $(shell find . -name "dune.in" | sed -e 's/.in$$//')
43

54
# This makefile is mostly calling dune and dune doesn't like
65
# being called in parralel, so we enforce -j1
76
.NOTPARALLEL:
87

9-
all: $(DUNE_IN_FILES)
8+
all:
109
$(call dune,build)
1110
$(call dune,build) builtin-doc
1211
.PHONY: all
1312

14-
# simplify this and get rid of the dune.in files once we require Rocq >= 9.0
15-
%dune: %dune.in
16-
@rm -f $@
17-
@echo "; generated by make, do not edit\n" > $@
18-
@if test -e .coq-dune-files || \
19-
(command -v coqc > /dev/null && (coqc --version | grep -q '8.20')) ; then \
20-
sed -e 's/@@STDLIB_THEORY@@//' $< | \
21-
sed -e 's/@@STDLIB@@//' | \
22-
sed -e 's/@@ROCQ_RUNTIME@@/coq-core/g' >> $@ ; \
23-
else \
24-
sed -e 's/@@STDLIB_THEORY@@/(theories Stdlib)/' $< | \
25-
sed -e 's/@@STDLIB@@/Stdlib/' | \
26-
sed -e 's/@@ROCQ_RUNTIME@@/rocq-runtime/g' >> $@ ; \
27-
fi
28-
@chmod a-w $@
29-
30-
dune-files: $(DUNE_IN_FILES)
13+
dune-files:
14+
echo "no longer doing anything"
3115
.PHONE: dune-files
3216

33-
coq-dune-files:
34-
touch .coq-dune-files
35-
$(MAKE) dune-files
36-
$(RM) .coq-dune-files
37-
.PHONE: coq-dune-files
38-
39-
build-core: $(DUNE_IN_FILES)
17+
build-core:
4018
$(call dune,build) theories
4119
$(call dune,build) builtin-doc
4220
.PHONY: build-core
4321

44-
build-apps: $(DUNE_IN_FILES)
22+
build-apps:
4523
$(call dune,build) $$(find apps -type d -name theories)
4624
.PHONY: build-apps
4725

48-
build: $(DUNE_IN_FILES)
26+
build:
4927
$(call dune,build) -p rocq-elpi @install
5028
$(call dune,build) builtin-doc
5129
.PHONY: build
5230

5331
all-tests: test-core test-stdlib test-apps test-apps-stdlib
5432
.PHONY: all-tests
5533

56-
test-core: $(DUNE_IN_FILES)
34+
test-core:
5735
$(call dune,runtest) tests
5836
$(call dune,build) tests
5937
# Check for build reproducibility
@@ -65,22 +43,22 @@ test-core: $(DUNE_IN_FILES)
6543
rm md5sum_accumulate1vo_1 md5sum_accumulate1vo_2
6644
.PHONY: test-core
6745

68-
test-apps: $(DUNE_IN_FILES)
46+
test-apps:
6947
$(call dune,build) $$(find apps -type d -name tests)
7048
.PHONY: test-apps
7149

72-
test-apps-stdlib: $(DUNE_IN_FILES)
50+
test-apps-stdlib:
7351
$(call dune,build) $$(find apps -type d -name tests-stdlib)
7452
.PHONY: test-apps-stdlib
7553

76-
test-stdlib: $(DUNE_IN_FILES)
54+
test-stdlib:
7755
$(call dune,build) tests-stdlib
7856
.PHONY: test-stdlib
7957

8058
all-examples: examples examples-stdlib
8159
.PHONY: all-examples
8260

83-
examples: $(DUNE_IN_FILES)
61+
examples:
8462
$(call dune,build) examples
8563
.PHONY: examples
8664

@@ -108,7 +86,7 @@ clean:
10886
$(call dune,clean)
10987
.PHONY: clean
11088

111-
install: $(DUNE_IN_FILES)
89+
install:
11290
$(call dune,install) rocq-elpi
11391
.PHONY: install
11492

README.md

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,4 @@
11
[![CI](https://github.com/LPCIC/coq-elpi/actions/workflows/ci.yml/badge.svg)](https://github.com/LPCIC/coq-elpi/actions/workflows/ci.yml)
2-
[![Nix 8.20](https://github.com/LPCIC/coq-elpi/actions/workflows/nix-action-coq-8.20.yml/badge.svg)](https://github.com/LPCIC/coq-elpi/actions/workflows/nix-action-coq-8.20.yml)
32
[![Nix 9.0](https://github.com/LPCIC/coq-elpi/actions/workflows/nix-action-rocq-9.0.yml/badge.svg)](https://github.com/LPCIC/coq-elpi/actions/workflows/nix-action-rocq-9.0.yml)
43
[![Nix 9.1](https://github.com/LPCIC/coq-elpi/actions/workflows/nix-action-rocq-9.1.yml/badge.svg)](https://github.com/LPCIC/coq-elpi/actions/workflows/nix-action-rocq-9.1.yml)
54
[![Nix master](https://github.com/LPCIC/coq-elpi/actions/workflows/nix-action-coq-master.yml/badge.svg)](https://github.com/LPCIC/coq-elpi/actions/workflows/nix-action-coq-master.yml)

_CoqProject

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -10,8 +10,6 @@
1010

1111
-R theories elpi
1212
-R _build/default/theories elpi
13-
-Q theories-stdlib elpi_stdlib
14-
-Q _build/default/theories-stdlib elpi_stdlib
1513

1614
-Q elpi elpi_elpi
1715
-Q _build/default/elpi elpi_elpi

apps/coercion/src/dune.in renamed to apps/coercion/src/dune

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -3,7 +3,7 @@
33
(public_name rocq-elpi.coercion)
44
(flags :standard -w -27)
55
(preprocess (pps ppx_optcomp -- -cookie "ppx_optcomp.env=env ~coq:(Defined \"%{coq:version.major}.%{coq:version.minor}\")"))
6-
(libraries @@ROCQ_RUNTIME@@.plugins.ltac @@ROCQ_RUNTIME@@.vernac rocq-elpi.elpi))
6+
(libraries rocq-runtime.plugins.ltac rocq-runtime.vernac rocq-elpi.elpi))
77

88
(coq.pp
99
(modules rocq_elpi_coercion_hook))

apps/coercion/tests/test_open.v

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
From elpi.apps Require Import coercion.
2-
From elpi.core Require Import ssreflect.
2+
From Corelib Require Import ssreflect.
33

44
Ltac my_solver := try ((repeat apply: le_n_S); apply: le_0_n).
55

apps/cs/src/dune.in renamed to apps/cs/src/dune

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -3,7 +3,7 @@
33
(public_name rocq-elpi.cs)
44
(flags :standard -w -27)
55
(preprocess (pps ppx_optcomp -- -cookie "ppx_optcomp.env=env ~coq:(Defined \"%{coq:version.major}.%{coq:version.minor}\")"))
6-
(libraries @@ROCQ_RUNTIME@@.plugins.ltac @@ROCQ_RUNTIME@@.vernac rocq-elpi.elpi))
6+
(libraries rocq-runtime.plugins.ltac rocq-runtime.vernac rocq-elpi.elpi))
77

88
(coq.pp
99
(modules rocq_elpi_cs_hook))

apps/cs/src/rocq_elpi_cs_hook.mlg

Lines changed: 2 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -7,7 +7,7 @@ open Elpi_plugin
77
open Rocq_elpi_arg_syntax
88
open Rocq_elpi_vernacular
99

10-
[%%if coq = "8.20" || coq = "9.0" || coq = "9.1"]
10+
[%%if coq = "9.0" || coq = "9.1"]
1111
let structure_projection_nparams env ind = Structures.Structure.projection_nparams ind
1212
[%%else]
1313
let structure_projection_nparams env ind = Structures.Structure.projection_nparams env ind
@@ -65,20 +65,14 @@ let elpi_cs_hook program env sigma ((proji, u), params1, c1) (t2, args2) =
6565
| API.Execute.Failure -> None
6666
end
6767

68-
[%%if coq = "8.20"]
69-
let adapt_hook f : Evarconv.hook = fun env sigma (s,l,t) x -> f env sigma (s,Some l,t) x
70-
[%%else]
71-
let adapt_hook f : Evarconv.hook = f
72-
[%%endif]
73-
7468
let add_cs_hook =
7569
let cs_hook_program = Summary.ref ~name:"elpi-cs" None in
7670
let cs_hook env sigma proj pat =
7771
match !cs_hook_program with
7872
| None -> None
7973
| Some h -> elpi_cs_hook h env sigma proj pat in
8074
let name = "elpi-cs" in
81-
Evarconv.register_hook ~name (adapt_hook cs_hook);
75+
Evarconv.register_hook ~name cs_hook;
8276
let inCs =
8377
let cache program =
8478
cs_hook_program := Some program;

apps/derive/tests-stdlib/dune

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2,6 +2,6 @@
22
(package rocq-elpi-tests-stdlib)
33
(name elpi_apps_derive_tests_stdlib)
44
(flags :standard -w -default-output-directory)
5-
(theories elpi elpi.apps.derive elpi.apps.derive.tests elpi_stdlib))
5+
(theories elpi elpi.apps.derive elpi.apps.derive.tests Stdlib))
66

77
(include_subdirs qualified)

apps/derive/tests/test_bcongr.v

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,3 @@
1-
From elpi.core Require Import Bool.
21
From elpi.apps Require Import derive.bcongr.
32

43
From elpi.apps Require Import test_derive_corelib test_projK.

0 commit comments

Comments
 (0)