timbertson / opam2nix

Generate nix expressions from opam packages
MIT License
93 stars 28 forks source link

Change string representation of hashes #83

Open rgrinberg opened 2 years ago

rgrinberg commented 2 years ago

The wiki says the following:

If you find a hash that uses colon as a separator (:), don't use that. This relies on undocumented behavior and is not officially supported.

Should opam2nix switch to using dashes perhaps?

timbertson commented 2 years ago

Probably. But SRI hashes aren't implemented in nix-hash, you need nix which must first be enabled as an experimental feature. I could implement the SRI encoding myself, but it's not standard base64 so I'll just wait it out 🤷