Skip to content

frodo-kem: enable cross in CI#97

Closed
tarcieri wants to merge 2 commits intomasterfrom
frodo-kem/ci-cross
Closed

frodo-kem: enable cross in CI#97
tarcieri wants to merge 2 commits intomasterfrom
frodo-kem/ci-cross

Conversation

@tarcieri
Copy link
Copy Markdown
Member

Ensures big endian targets are supported by testing on PPC32

Ensures big endian targets are supported by testing on PPC32
@tarcieri tarcieri requested a review from mikelodder7 January 21, 2025 20:08
@tarcieri
Copy link
Copy Markdown
Member Author

To make this work it can't link against safe-oqs-sys it appears

@mikelodder7
Copy link
Copy Markdown
Contributor

That's okay, its just for testing. Perhaps we just conditionally test it against safe-oqs when under specific OS.

@tarcieri
Copy link
Copy Markdown
Member Author

Perhaps it could be gated on a special cfg

tarcieri added a commit that referenced this pull request Nov 10, 2025
Closes: #97 #109 #165

It would be good to be able to retain these equivalence tests, but
they're pretty onerous for anyone trying to develop any KEM.

It also was breaking cross (#97) which can now be enabled for
`frodo-kem`.
tarcieri added a commit that referenced this pull request Nov 10, 2025
Closes: #97 #109 #165

It would be good to be able to retain these equivalence tests, but
they're pretty onerous for anyone trying to develop any KEM.

It also was breaking cross (#97) which can now be enabled for
`frodo-kem`.
@tarcieri tarcieri closed this Nov 10, 2025
@tarcieri tarcieri deleted the frodo-kem/ci-cross branch February 1, 2026 00:22
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