-
Notifications
You must be signed in to change notification settings - Fork 3
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
feat: Release coqorg/coq:8.20-rc1 #66
Conversation
Hi @ejgallego ! when you have some time, could you prepare a coq-serapi tag then, we'll be able to open a PR targeting extra-dev similar to this one:
and release 8.20+rc1 in docker-coq. Many thanks! |
@ejgallego unless you have reservations, I will prepare a SerAPI tag in the next day or so. |
@erikmd @ejgallego it doesn't look feasible for me to do a tag due to rocq-archive/coq-serapi@c4685e3 which makes |
Hi @palmskog ! OK. What make you think that it'd be tricky (or impossible (or a bad idea)) to still keep coq-serapi in docker-coq ? And what could we propose for projects that use docker-coq + alectryon? |
@erikmd since serlib has migrated to Coq-LSP, you are going to need tags of both Coq-LSP and SerAPI, and I don't know if these will be forthcoming. It's not obvious to me if Coq-LSP as a whole should be included in the Docker images, since it may be updated many times for a single Coq version (so images will have to change a lot). |
I do agree, let's remove it. We already have many packages available for 8.20+rc1 in both Nix and Opam, it makes no sense to further delay the Docker image. |
As you prefer folks, there are many different possibilities here; All the current SerAPI users have been either migrated to coq-lsp or will do soon, so unless someone would like to volunteer to maintain SerAPI the 8.20 release will be the last one. |
Hi, thanks for your comments.
@proux01 Note that Also, not-shipping SerAPI in-docker-coq anymore would be a major change (currently it is included for
@ejgallego IMHO it would be nice indeed.
Can you explain the rationale? Is For instance, I don't know what will be the impact on Alectryon, which still uses SerAPI AFAIK? (Cc @cpitclaudel) |
Not drop in as-is, but could be made if someone would like to port The main thing that have changed in 8.20 is that Sorry I didn't anticipate the docker problem for RCs when I did this move, but I really needed serlib in coq-lsp repos as to implement some critical functionality for some projects. One quick solution for SerAPI is actually to do the following for the 8.20 tag:
Unfortunately I cannot do this before July 8th due to a combo of holidays and really critical stuff; but the process is simple, mainly:
Note that for 8.20 a fully working SerAPI will be still kept. For 8.21, that will only happen if someone volunteers to maintain it. |
Wonderful! I didn't understood that. Let me announce the RC then. |
@ejgallego thanks a lot for your detailed explanation 👍 👍
Fully understand that!
OK, looks good! and there's no hurry: given what I said in my previous comment, no worries to wait till next week.
Okay 👌 |
friendly ping @ejgallego → if you can prepare a |
How would you evaluate the time needed to "maintain" SerAPI relatively to the next versions of Coq compared to the time needed to switch from SerAPI to what coq-lsp provides as a replacement? (For instance, how does it compare to the time already spent at making adaptations to elpi, equations, metacoq or coq-lsp in the CI?) To maintain SerAPI, is it a question of half an hour per release, of several hours, of several days? To switch to coq-lsp, is it a matter of a few hours, of a few days? |
If the dual repo setup is inconvenient, what about moving everything in the coq-lsp repo and archiving the SerAPI repo? IIUC Alectryon uses sertop, so we indeed need to either find a volunteer to maintain sertop (on top of flèche) or to port Alectryon to coq-lsp. |
Cc @cpitclaudel, in case he wasn't aware of the discussion above |
(rebased on master) |
Thanks @erikmd , I've seen the thread and I wrote about it here: https://coq.zulipchat.com/#narrow/stream/437203-Coq-Platform-docs/topic/Alectryon.20issues/near/455232324 |
Hi all, I just chatted with @ejgallego who summarized the status of the SerAPI / coq-lsp release:
Meanwhile for 8.20, @ejgallego just implemented solution 3 and pushed a version of Cc @palmskog, do you think you could help to release this branch in opam, as soon as the coq-serapi CI passes? |
@erikmd OK, when |
Thanks a lot @ejgallego! |
Thanks to you! |
Thanks a lot!! :) |
Answer to this @herbelin is that it depends on the expertise and experience of the team in charge of that. I'd prefer tho to discuss maintenance topics on their own issue / meeting, after catching up it seems to me that the discussion is happening in a scattered way which makes it hard to follow.
IMHO there is no point for Alectryon to keep using |
cf. @proux01's comment in coq/coq#18882 (comment)
8.20-rc1 is now available in
core-dev
: coq/opam#3058bignums is already available in
extra-dev
: coq/opam#3054IINM, the last required ingredient before merging the PR is coq-serapi