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

Please create a tag for Coq 8.19 in Coq Platform 2024.01 #1831

Closed
rtetley opened this issue Mar 18, 2024 · 15 comments
Closed

Please create a tag for Coq 8.19 in Coq Platform 2024.01 #1831

rtetley opened this issue Mar 18, 2024 · 15 comments

Comments

@rtetley
Copy link

rtetley commented Mar 18, 2024

The Coq team released Coq 8.19.0 on January 24th, 2024.
The corresponding Coq Platform release 2024.01 should be released before April 30th, 2024.
It can be delayed in case of difficulties until May 15th, 2024, but this should be an exception.

This issue is to inform you that to our (possibly a few days old) best knowledge the latest released version of your project (0.0.24) does not work with Coq 8.19.0.
We tried to remove version restrictions in opam files and possibly make or configure files, but this did not suffice.

Please note that in Coq Platform CI (unlike Coq CI) we test only released / tagged versions. Coq CI is currently testing commit 4d9fadc
47058dc on repository https://github.com/mit-plv/fiat-crypto
https://github.com/mit-plv/fiat-crypto - which likely means that this commit does work in Coq CI.

Could you please create a tag and opam package, or communicate us any existing tag that works with Coq branch v8.19, preferably before March 31st, 2024?
In case we might have to delay the Coq Platform release cause of issues with your project, we would prefer to be informed about the situation as early as possible.

In case the tag and opam package are available before March 31st, 2024, it will be included in an early Coq Platform beta release of the for Coq 8.19.0.

The working branch of Coq Platform, can be found here main.
It contains package pick ~8.19~2024.01+beta1 which already supports Coq version 8.19.0 and contains already working (possibly patched / commit pinned) Coq Platform packages.

Please don't close this issue, even after creating the new tag and/or opam package.
We will close the issue after updating Coq Platform.
This is a measure of 'double book keeping' in order to avoid that we ship the wrong version.

In any case, Coq Platform won't be released before this issue is closed!

Thanks!

P.S.: this issue has been created automatically based on CI status.

CC: coq/platform#405

@samuelgruetter
Copy link
Contributor

I could create tags for these dependencies. I don't really care which versions end up in the platform. @JasonGross should I just pick whatever commit the fiat-crypto submodules point to?

@JasonGross
Copy link
Collaborator

Plausibly it makes sense to merge mit-plv/rupicola#100 (and mit-plv/rupicola#89?) first, but otherwise, yes @samuelgruetter , please tag whatever the master branch of rupicola points to (which should be the same as the fiat-crypto submodules)

@samuelgruetter
Copy link
Contributor

I just created the following tags/releases:

@rtetley I will not write separate replies in the issues that you opened in these dependencies, please consider this thread here in fiat-crypto as our replies for the repos of the submodules

@JasonGross
Copy link
Collaborator

I've released

and made coq/opam#2992 to put all these packages on opam

@rtetley
Copy link
Author

rtetley commented Mar 20, 2024

Perfect ! I'll wait for merge in opam and update the pick !

@JasonGross
Copy link
Collaborator

I've merged the opam PR

@rtetley
Copy link
Author

rtetley commented Mar 26, 2024

Merged coq/platform#407

@rtetley rtetley closed this as completed Mar 26, 2024
@andres-erbsen
Copy link
Contributor

So I've tied up a number of loose ends in fiat-crypto over the last week. Would it be possible to bump the fiat-crypto version that goes into the upcoming coq platform? 🙏

@andres-erbsen andres-erbsen reopened this Apr 4, 2024
@rtetley
Copy link
Author

rtetley commented Apr 4, 2024

No worries ! I don't see any new release though ?

@JasonGross
Copy link
Collaborator

@andres-erbsen should I tag a new release of Fiat Crypto on current master and update opam packages? (And if current master is what you want used, could you or @samuelgruetter tag corresponding releases of the submodules?)

@rtetley
Copy link
Author

rtetley commented Apr 22, 2024

@andres-erbsen , @JasonGross, @samuelgruetter, quick ping ! What is the status here ? Cheers !

@JasonGross
Copy link
Collaborator

I've tagged Fiat Cryptography v0.1.3. Please keep using rewriter 0.1.11. @andres-erbsen or @samuelgruetter could you tag releases for the other submodules again? Then I (or someone else) can update opam. Thank you!

@andres-erbsen
Copy link
Contributor

I have tagged the dependencies at respective heads (which I checked to be in sync modulo CI changes):

Thank you for accommodating us!

@rtetley
Copy link
Author

rtetley commented Apr 23, 2024

No worries at all. Should go through with coq/platform#411

@rtetley rtetley closed this as completed Apr 23, 2024
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

No branches or pull requests

4 participants