Merge the pact-crypto lib to the main tree - #3
Conversation
jmcardon
left a comment
There was a problem hiding this comment.
So:
WITHOUT_CRYPTO macro had a use: building for profiling builds with cost centers. At the time when we tried this, GHC had a bug which prevented profiling builds to be used with our lib.
Please prove that profiling builds work in this PR after removing this macro.
Probably you didn't use the WITHOUT_CRYPTO flags since at least 2 years… 😄 😉 ..
That's why I believe, it doesn't make sense anymore to have it separated. |
I remember I already did it.. But let me check again. |
I've merged the the pact-cypto lib into the main tree, moving everything from Pact.Core.Crypto -> to Pact.Crypto (like the WebAuthn implementation).
Base64 has been moved moved to the directly Pact.Core, and indirect import through Pact.Hash removed.
Preprocessor directive WITHOUT_CRYPTO (and related cases) have been removed.
A warning (shadowed variable) has been fixed in Keccak.
I hope you like the color of bicycle shed @jmcardon .. 😃