Skip to content

Expose keypair_derand in liboqs integration YML#886

Merged
mkannwischer merged 1 commit intopq-code-package:sw-liboqs-derandfrom
SWilson4:sw-liboqs-derand
Mar 21, 2025
Merged

Expose keypair_derand in liboqs integration YML#886
mkannwischer merged 1 commit intopq-code-package:sw-liboqs-derandfrom
SWilson4:sw-liboqs-derand

Conversation

@SWilson4
Copy link
Contributor

Summary:
Updates liboqs integration YML files to enable open-quantum-safe/liboqs#2070. Keeping as draft until that PR lands in case of changes.

Steps:
N/A

Performed local tests:

  • lint passing
  • tests all passing
  • tests bench passing
  • tests cbmc passing

Do you expect this change to impact performance: No

If yes, please provide local benchmarking results.

@hanno-becker
Copy link
Contributor

@SWilson4 will you give us a ping when this is ready?

@SWilson4 SWilson4 marked this pull request as ready for review March 21, 2025 13:16
@SWilson4 SWilson4 requested a review from a team as a code owner March 21, 2025 13:16
Signed-off-by: Spencer Wilson <spencer.wilson@uwaterloo.ca>
@SWilson4
Copy link
Contributor Author

@hanno-becker @mkannwischer open-quantum-safe/liboqs#2070 has landed, so this is ready to go.

Copy link
Contributor

@mkannwischer mkannwischer left a comment

Choose a reason for hiding this comment

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

LGTM! Let's get this into a local branch so we can run our full CI on it first.

@mkannwischer mkannwischer changed the base branch from main to sw-liboqs-derand March 21, 2025 13:23
@mkannwischer mkannwischer merged commit 75eb0d4 into pq-code-package:sw-liboqs-derand Mar 21, 2025
121 checks passed
mkannwischer added a commit that referenced this pull request Apr 2, 2025
This updates liboqs to the most recent commit in main:
https://github.com/open-quantum-safe/liboqs/tree/6337a8424deae09aa835ddd920faff83a8f0e1d7

open-quantum-safe/liboqs#2070
reintroduced patching for mlkem-native to expose the keypair_derand functions.
This patching is no longer needed since
#886 got merged.
This commit hence adds to the liboqs test in CI the removal of the patch
in the copy_from_upstream.yml.

Signed-off-by: Matthias J. Kannwischer <matthias@kannwischer.eu>
mkannwischer added a commit that referenced this pull request Apr 2, 2025
This updates liboqs to the most recent commit in main:
https://github.com/open-quantum-safe/liboqs/tree/6337a8424deae09aa835ddd920faff83a8f0e1d7

open-quantum-safe/liboqs#2070
reintroduced patching for mlkem-native to expose the keypair_derand functions.
This patching is no longer needed since
#886 got merged.
This commit hence adds to the liboqs test in CI the removal of the patch
in the copy_from_upstream.yml.

Signed-off-by: Matthias J. Kannwischer <matthias@kannwischer.eu>
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.

3 participants