Skip to content

hermes-1.3.4.3 appears in 00-index, but tarball not available for download #436

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

Closed
snoyberg opened this issue Oct 15, 2015 · 4 comments
Closed

Comments

@snoyberg
Copy link
Contributor

This is currently breaking multiple mirroring scripts, which do sanity checks (which are rightfully failing). The tarball should either be made available, or the .cabal file removed from the 00-index tarball.

snoyberg added a commit to commercialhaskell/all-cabal-hashes-tool that referenced this issue Oct 15, 2015
@hvr
Copy link
Member

hvr commented Oct 15, 2015

(related to #382 (comment))

@hvr
Copy link
Member

hvr commented Oct 15, 2015

FWIW, For facilities I'm building on top of a monotonically append-only index-log, removing .cabal files from the tarball without leaving a placeholder to retain the unique serial/event numbering (alas the timestamps can't be used for that) would be fatal.

However, I'm talking about the 01-index.tar file (where I think there's something like a tar-level deletion entry which can be appended). For the 00-index.tar file I see no problem in physically removing the .cabal entry.

@snoyberg
Copy link
Contributor Author

Yeah, I was worried that the new tarball approach may have had something to do with this.

I'll write down for the record that this is now the second time I'm fixing up something that's broken that's "Hackage Security" related :(

snoyberg added a commit to commercialhaskell/all-cabal-metadata-tool that referenced this issue Oct 15, 2015
@adamgundry
Copy link
Member

@snoyberg I don't believe that this is related to the Hackage Security work.

A user uploaded the problematic package by mistake, and it contained code that was not supposed to be publicly distributed, so the tarball was removed from the server by the admins. Deletion is not yet properly supported (cf. #112, #382) so it's not as easy to remove the .cabal file. Unfortunately this also means that there are a few broken links where the server reports an internal error.

It would be good to implement support for deleting a package (version), removing the .cabal file completely from the 00-index.tar and by replacing it with a sentinel in the 01-index.tar, as @hvr suggests.

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