Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

compiler future-proofing Release builds #1378

Merged
merged 2 commits into from
Feb 7, 2023
Merged

compiler future-proofing Release builds #1378

merged 2 commits into from
Feb 7, 2023

Conversation

baentsch
Copy link
Member

@baentsch baentsch commented Feb 3, 2023

As discussed in #1375 (comment).

An initial CI error is intentional to test the feature. Will follow up with a correction (removal of unused variable).

  • [no] Does this PR change the input/output behaviour of a cryptographic algorithm (i.e., does it change known answer test values)? (If so, a version bump will be required from x.y.z to x.(y+1).0.)
  • [no] Does this PR change the the list of algorithms available -- either adding, removing, or renaming? Does this PR otherwise change an API? (If so, PRs in oqs-provider, OQS-OpenSSL, OQS-BoringSSL, and OQS-OpenSSH will also need to be ready for review and merge by the time this is merged.)

@baentsch baentsch marked this pull request as ready for review February 3, 2023 15:36
@baentsch baentsch merged commit 04ff6fd into main Feb 7, 2023
@baentsch baentsch deleted the mb-strictwarnings branch February 7, 2023 16:48
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants