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

Temporarily drop EdDSA support until linking is sorted #66

Merged
merged 1 commit into from
Nov 29, 2016
Merged

Conversation

olorin
Copy link
Contributor

@olorin olorin commented Nov 29, 2016

Packages which depend on tinfoil currently rely on linking with
libsodium, which makes the executables they build nonportable. The
right solution to this is linking to a static version of libsodium
(#65) and/or split the package (#59), but until this can be fully
implemented and pushed through to master, this is a temporary fix so
we can build working client executables (ref
https://github.com/ambiata/ambiata-cli/issues/59).

! @markhibberd

Packages which depend on tinfoil currently rely on linking with
libsodium, which makes the executables they build nonportable. The
right solution to this is linking to a static version of libsodium
(#65) and/or split the package (#59), but until this can be fully
implemented and pushed through to master, this is a temporary fix so
we can build working client executables (ref
https://github.com/ambiata/ambiata-cli/issues/59).
@markhibberd
Copy link
Contributor

👍 seems fair

@olorin olorin merged commit 79b7d0d into master Nov 29, 2016
@olorin olorin deleted the topic/lnk branch November 29, 2016 05:37
olorin added a commit to haskell-mafia/zodiac that referenced this pull request Nov 29, 2016
So we're not linking with libsodium unnecessarily.

haskell-mafia/tinfoil#66
olorin added a commit to haskell-mafia/zodiac that referenced this pull request Nov 29, 2016
So we're not linking with libsodium unnecessarily.

haskell-mafia/tinfoil#66
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