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

Cabal revisions #393

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Cabal revisions #393

wants to merge 3 commits into from

Conversation

qrilka
Copy link
Contributor

@qrilka qrilka commented Dec 4, 2018

Our final target is to use this in stack2nix, this PR depends on NixOS/hackage-db#9 so I guess we'll need hackage-db bounds updated when it will get merged

@peti
Copy link
Member

peti commented Dec 4, 2018

I don't think I follow ... why does this PR depend on stack?

@qrilka
Copy link
Contributor Author

qrilka commented Dec 4, 2018

Oh, dummy copy-paste, sorry, link corrected

@qrilka
Copy link
Contributor Author

qrilka commented Dec 5, 2018

@peti in ced478e I've added support for Stack's size part after SHA (there it's used as a measure against overflow attacks). Also in SHA256 is assumed to be in standard hexadecimal notation (Nix's base32 is used internally).
Please let me know what you think about this. Also it would be great to get some understanding of the situation with NixOS/hackage-db#9

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

Successfully merging this pull request may close these issues.

2 participants