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

Launch the Anoma node and client from the elixir REPL #3172

Merged
merged 1 commit into from
Nov 14, 2024

Conversation

paulcadman
Copy link
Collaborator

@paulcadman paulcadman commented Nov 13, 2024

This PR changes how we launch the Anoma Client to avoid a bug with linking cryptographic APIs.

libsodium cryptographic APIs like sign-detached cannot currently be called from within the Anoma node or client binaries. Until this is solved we must start both the node and client from the elixir REPL. Previously we were starting the node using the REPL and the client using the binary.

This commit changes the start.exs script we were using to start the node to now start both a node and a client.

After this change we can enable Anoma compilation test test077.

The output of juvix dev anoma node --anoma-dir ANOMA_DIR is now:

Anoma node and client successfully started
Listening on port 51748

libsodium cryptographic APIs like sign-detached cannot currently be
called from within the Anoma node or client binaries.

Until this is solved we must start both the node and client from the
elixir REPL.

Previously we were starting the node using the REPL and the client using
the binary.

This commit changes the `start.exs` script we were using to start the
node to now start both a node and a client.
@paulcadman paulcadman self-assigned this Nov 13, 2024
@paulcadman paulcadman added this to the 0.6.9 milestone Nov 13, 2024
@paulcadman paulcadman requested a review from lukaszcz November 13, 2024 18:59
@janmasrovira janmasrovira merged commit 29041dc into main Nov 14, 2024
4 checks passed
@janmasrovira janmasrovira deleted the anoma-client-repl-start branch November 14, 2024 08:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants