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

switch opam ci to docker #733

Merged
merged 1 commit into from
Dec 15, 2024
Merged

switch opam ci to docker #733

merged 1 commit into from
Dec 15, 2024

Conversation

gares
Copy link
Contributor

@gares gares commented Dec 12, 2024

No description provided.

.github/workflows/main.yml Outdated Show resolved Hide resolved
.github/workflows/main.yml Outdated Show resolved Hide resolved
.github/workflows/main.yml Outdated Show resolved Hide resolved
@gares
Copy link
Contributor Author

gares commented Dec 13, 2024

This is where the stdlib is:

...
/home/runner/work/coq-elpi/coq-elpi/_opam/lib/coq/theories/Strings/BinaryString.vo
/home/runner/work/coq-elpi/coq-elpi/_opam/lib/coq/theories/Strings/PrimStringAxioms.vo
find: ‘/proc/tty/driver’: Permission denied
/home/runner/work/coq-elpi/coq-elpi/_opam/lib/coq/user-contrib/Ltac2/Pattern.vo
/home/runner/work/coq-elpi/coq-elpi/_opam/lib/coq/user-contrib/Ltac2/Std.vo
/home/runner/work/coq-elpi/coq-elpi/_opam/lib/coq/user-contrib/Ltac2/RedFlags.vo
...

@proux01 is it supposed to be in coq/ (not rocq)?

@proux01
Copy link
Contributor

proux01 commented Dec 13, 2024

Yes, it is still coq, this hasn't been renamed yes (and I fer we'll have a hard time doing that renaming with dune).

.github/workflows/main.yml Outdated Show resolved Hide resolved
@gares
Copy link
Contributor Author

gares commented Dec 14, 2024

The error is now different:

dune build  -p coq-elpi @install
Error: path outside the workspace:
../../../_opam/lib/coq-core/plugins/ltac/ltac_plugin.cmxs from
default/theories
-> required by _build/default/theories/elpi.glob
-> required by _build/install/default/lib/coq/user-contrib/elpi/elpi.glob
-> required by _build/default/coq-elpi.install
-> required by alias install

I think it was a recent change to coqdep to spit recursive dependencies.
Am I right @SkySkimmer ?

The peculiarity here could be that setup-ocaml puts the opam root in ./, next to _build, eg inside dune's workspace

.github/workflows/main.yml Outdated Show resolved Hide resolved
@gares gares force-pushed the gares-patch-1 branch 3 times, most recently from 077bd36 to 445061c Compare December 15, 2024 21:20
@gares gares changed the title Update main.yml switch opam ci to docker Dec 15, 2024
@gares gares merged commit d108d2c into master Dec 15, 2024
23 checks passed
@gares gares deleted the gares-patch-1 branch December 15, 2024 21:35
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.

2 participants