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

[new release] Add Coq 8.11.1 #16146

Merged
merged 2 commits into from
Apr 5, 2020
Merged

Conversation

anton-trunov
Copy link
Contributor

Adapted from the PR #15764 introducing Coq 8.11.0.

CoqIDE 8.11.1 will be added in a separate PR to avoid holding Coq up in case of issues.

@camelus
Copy link
Contributor

camelus commented Apr 5, 2020

Commit: 4e5d1ac

@anton-trunov has posted 6 contributions.

☀️ All lint checks passed 4e5d1ac
  • These packages passed lint tests: coq.8.11.1

☀️ Installability check (+1)
  • new installable packages (1): coq.8.11.1

🌤️ 1 ignored non-opam files:
  • packages/coq/coq.8.11.1/files/coq.install

@Zimmi48
Copy link
Contributor

Zimmi48 commented Apr 5, 2020

Thanks! Note coq/coq#11178 where it was suggested to disable native-compute on macOS in the opam package. I have no idea whether opam supports system-specific build instructions though!

@anton-trunov
Copy link
Contributor Author

I'm wondering if there is a shorter way to express "-native-compiler" {os = "macos"} "no" {os = "macos"}.

@kit-ty-kate
Copy link
Member

I'm wondering if there is a shorter way to express "-native-compiler" {os = "macos"} "no" {os = "macos"}.

I'm unsure, maybe. But for now i think this will be perfect.

Thanks!

@kit-ty-kate kit-ty-kate merged commit a10e0ac into ocaml:master Apr 5, 2020
@anton-trunov anton-trunov deleted the coq-8.11.1 branch April 5, 2020 13:31
@anton-trunov
Copy link
Contributor Author

@kit-ty-kate Awesome, thank you!

@anton-trunov
Copy link
Contributor Author

FWIW, it turns out that filters work on "per-line" basis, see ocaml/opam#4127

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.

4 participants