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

chore: Developer branch for nim #44

Open
wants to merge 15 commits into
base: master
Choose a base branch
from
Open

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)

@@ -9,7 +9,7 @@ const

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