Skip to content

Enable OTP 23 support with swappable native or Rust NIF keccack hash implementation#34

Open
rupurt wants to merge 1 commit into
exthereum:mainfrom
rupurt:otp-23-keccak
Open

Enable OTP 23 support with swappable native or Rust NIF keccack hash implementation#34
rupurt wants to merge 1 commit into
exthereum:mainfrom
rupurt:otp-23-keccak

Conversation

@rupurt

@rupurt rupurt commented Oct 18, 2020

Copy link
Copy Markdown

- keccakf1600 doesn't compile on Erlang OTP 23
(potatosalad/erlang-keccakf1600#5)
@rupurt rupurt changed the title Use native or Rust NIF for keccack hash Enable OTP 23 support with swappable native or Rust NIF keccack hash implementation Oct 18, 2020
@ayrat555

Copy link
Copy Markdown
Member

@rupurt I fixed this in https://github.com/poanetwork/ex_abi

@rupurt

rupurt commented Oct 19, 2020

Copy link
Copy Markdown
Author

@ayrat555 awesome, thank you. Have you tried to upstream it to the dependent libraries?

@ayrat555

Copy link
Copy Markdown
Member

No, but I think that's a good idea. I see only two libraries on hex.pm that depend on this version of abi implementation. it should just work (the only difficulty will be adding rust).

Btw, ex_abi is more stable and there are no known bugs there.

@rupurt

rupurt commented Oct 19, 2020

Copy link
Copy Markdown
Author

Sweet. This was the lib I was trying to use when I ran into the underlying problem https://github.com/hswick/exw3.

BTW cool to meet you @ayrat555 👋. Nice to see there is an active Elixir developer in the blockchain space 😄

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