Skip to content

Fixing C symbol names#7

Merged
kazu-yamamoto merged 2 commits into
masterfrom
cbits
Jun 19, 2023
Merged

Fixing C symbol names#7
kazu-yamamoto merged 2 commits into
masterfrom
cbits

Conversation

@kazu-yamamoto
Copy link
Copy Markdown
Owner

@shlevy @angerman Should fix #6.

@shlevy
Copy link
Copy Markdown

shlevy commented Jun 19, 2023

Great, thanks for the speedy turnaround! Testing now.

@shlevy
Copy link
Copy Markdown

shlevy commented Jun 19, 2023

@kazu-yamamoto Linking still fails due to this definition in ed25519-donna-batchverify.h:

/* not actually used for anything other than testing */
unsigned char batch_point_buffer[3][32];

@shlevy
Copy link
Copy Markdown

shlevy commented Jun 19, 2023

#8 will include additional symbol renames needed to get our project building.

@kazu-yamamoto kazu-yamamoto merged commit ac6496f into master Jun 19, 2023
@kazu-yamamoto kazu-yamamoto deleted the cbits branch June 19, 2023 12:20
kazu-yamamoto added a commit that referenced this pull request Jun 19, 2023
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.

C symbol names clash with cryptonite

2 participants