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

feat: introduce Nim impl of generateAlias #42

Merged
merged 1 commit into from
Aug 31, 2020

Conversation

michaelsbradleyjr
Copy link
Contributor

@michaelsbradleyjr michaelsbradleyjr commented Aug 17, 2020

Closes #32


This PR depends on #41 and on a commit in a PR branch in status-go, so it's labeled don't merge yet for the time being.

It's also important to note that because of this PR's introduction of nim-secp256k1 as a dependency in vendor/ it's necessary to switch from building/linking status-go as a static library to building/linking as a shared library.

@michaelsbradleyjr
Copy link
Contributor Author

michaelsbradleyjr commented Aug 17, 2020

I just realized I need some conditionals for Windows / Linux re: running the C tests; in the case of Windows need to augment PATH and in the case of Linux need to spec LD_LIBRARY_PATH. I'll do that shortly and force push.

✅ It took a little more adjusting than setting some env vars, but it's done!

@michaelsbradleyjr michaelsbradleyjr force-pushed the feat/nim-generatealias branch 8 times, most recently from e4ef049 to 10ed8a5 Compare August 19, 2020 16:55
@michaelsbradleyjr michaelsbradleyjr changed the base branch from feat/nim-hashmessage to master August 31, 2020 19:02
@michaelsbradleyjr michaelsbradleyjr merged commit 6c2265f into master Aug 31, 2020
@michaelsbradleyjr michaelsbradleyjr deleted the feat/nim-generatealias branch November 18, 2020 21:03
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.

implement generateAlias in nim
1 participant