Skip to content

Conversation

kevaundray
Copy link
Contributor

No description provided.

@kevaundray
Copy link
Contributor Author

Only nim will be passing tests since I changed the c function names and only modified that code to match.

@kevaundray
Copy link
Contributor Author

c functions needed to be changed because the linker was detecting duplicate symbols with c-kzg (since the c functions are named exactly the same thing)

proc getInstallDir*(): string =
when defined(macosx):
when defined(aarch64) or defined(amd64):
when defined(aarch64) or defined(amd64) or defined(arm64):
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

In nimbus, it was returning arm64 instead of aarch64

@kevaundray kevaundray force-pushed the kw/nim-release-branch branch 2 times, most recently from 1c3d0f2 to fc339dc Compare July 3, 2024 15:08
@kevaundray
Copy link
Contributor Author

Renaming the methods and the the nim changes can be ported to master

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.

1 participant