Skip to content

remove tm-grammars.1.0.0#29528

Closed
davesnx wants to merge 1 commit intoocaml:masterfrom
davesnx:remove-tm-grammars-1-0-0
Closed

remove tm-grammars.1.0.0#29528
davesnx wants to merge 1 commit intoocaml:masterfrom
davesnx:remove-tm-grammars-1-0-0

Conversation

@davesnx
Copy link
Contributor

@davesnx davesnx commented Mar 9, 2026

Summary

  • remove tm-grammars.1.0.0 from opam-repository
  • the published release tarball URL is currently broken, so the package is not installable

If the upstream 1.0.0 tag/release is restored with the original tarball/checksum, we can submit a re-add PR instead.

@jmid
Copy link
Member

jmid commented Mar 10, 2026

Thanks!

In general we try to avoid removing packages:
https://github.com/ocaml/opam-repository/tree/master/governance/policies#1-removal-of-packages-should-be-avoided

Instead, we try to find a copy of the tar-ball in one of the caches as described here
https://github.com/ocaml/opam-repository/wiki/FAQ#how-to-find-lost-archives-of-packages
then upload the restored tar-ball to https://github.com/ocaml/opam-source-archives/
and finally send a PR here to update the URL to point to opam-source-archives instead (but keeping the checksum)
and thereby ensure immutability of the already published package 🤷

(when I tried myself just now, the caches were failing me however...)

@davesnx
Copy link
Contributor Author

davesnx commented Mar 11, 2026

I tried to point to a new checksum, since the original got removed #29525

@jmid
Copy link
Member

jmid commented Mar 11, 2026

I tried to point to a new checksum, since the original got removed #29525

I suspect we are misunderstanding each other.
Here's a prior PR #28407 doing what I tried to explain above.
Notice how it is changing the tar-ball URL, but leaving the checksum unmodified.

mseri added a commit that referenced this pull request Mar 17, 2026
Mark the package as unavailable due to a broken release tarball URL, see #29528
@mseri
Copy link
Member

mseri commented Mar 17, 2026

Superseded by the linked PR

@mseri mseri closed this Mar 17, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants