Skip to content

Fix opam installing packages without checking their checksum when the local cache is corrupted in some cases#5538

Merged
kit-ty-kate merged 4 commits intoocaml:masterfrom
kit-ty-kate:fix-local-cache-corruption
May 8, 2023
Merged

Fix opam installing packages without checking their checksum when the local cache is corrupted in some cases#5538
kit-ty-kate merged 4 commits intoocaml:masterfrom
kit-ty-kate:fix-local-cache-corruption

Conversation

@kit-ty-kate
Copy link
Copy Markdown
Member

Reported by @roburio
Pre-checked by @rjbou
Ready for integration in #5444

@kit-ty-kate kit-ty-kate added this to the 2.2.0~alpha milestone May 8, 2023
@kit-ty-kate kit-ty-kate force-pushed the fix-local-cache-corruption branch 5 times, most recently from 14995f8 to 9d598fe Compare May 8, 2023 18:55
Copy link
Copy Markdown
Collaborator

@rjbou rjbou left a comment

Choose a reason for hiding this comment

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

Thanks!

kit-ty-kate and others added 4 commits May 8, 2023 21:19
…nd behave correctly when corrupted

Co-authored-by: Raja Boujbel <raja.boujbel@ocamlpro.com>
…and install packages without checking their checksum
@kit-ty-kate kit-ty-kate force-pushed the fix-local-cache-corruption branch from 9d598fe to 4d4b1e0 Compare May 8, 2023 20:19
@kit-ty-kate kit-ty-kate merged commit 7a538fd into ocaml:master May 8, 2023
@kit-ty-kate kit-ty-kate deleted the fix-local-cache-corruption branch May 8, 2023 23:52
kit-ty-kate added a commit to kit-ty-kate/opam that referenced this pull request May 9, 2023
…tion

Fix opam installing packages without checking their checksum when the local cache is corrupted in some cases
rjbou pushed a commit to kit-ty-kate/opam that referenced this pull request May 12, 2023
…tion

Fix opam installing packages without checking their checksum when the local cache is corrupted in some cases
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.

2 participants