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

adapt to coq/coq#15220 #34

Merged
merged 1 commit into from Feb 3, 2022
Merged

Conversation

gares
Copy link
Contributor

@gares gares commented Dec 1, 2021

To be merge when coq/coq#15220 is.

@gares gares mentioned this pull request Dec 1, 2021
1 task
Makefile Outdated Show resolved Hide resolved
src/Rewriter/Util/plugins/RewriterBuild.v Outdated Show resolved Hide resolved
_CoqProject.in Outdated Show resolved Hide resolved
Makefile.local.common Show resolved Hide resolved
.gitignore Show resolved Hide resolved
@JasonGross
Copy link
Collaborator

The .v89 files seem to be missing (RewriterBuild.v.v89, at the very least)

@gares
Copy link
Contributor Author

gares commented Dec 3, 2021

done, please re-run ci

Copy link
Collaborator

@JasonGross JasonGross left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks! All checks but master pass; let me know when to merge.

@gares gares force-pushed the dynlink-w-findlib branch 3 times, most recently from 6bdc30e to f27ac68 Compare January 27, 2022 11:20
@SkySkimmer
Copy link
Contributor

Please merge now

@JasonGross
Copy link
Collaborator

@SkySkimmer How long does it take for the docker image to be updated? I'm still getting "Unknown option src/Rewriter/Util/plugins/META.coq-rewriter"

@SkySkimmer
Copy link
Contributor

No idea
When did you start the run?

@JasonGross
Copy link
Collaborator

About an hour after you commented. I've told it to try again now

@SkySkimmer
Copy link
Contributor

green

@JasonGross JasonGross merged commit 2bd8776 into mit-plv:master Feb 3, 2022
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.

None yet

3 participants