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.

Sign up for GitHub

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

Jump to bottom

update oqsprovider/liboqs to v0.7.2 #19184

Closed
wants to merge 1 commit into from
Closed

update oqsprovider/liboqs to v0.7.2 #19184

wants to merge 1 commit into from

Conversation

baentsch
Copy link
Contributor

@baentsch baentsch commented Sep 9, 2022

Updates oqsprovider to current liboqs release as per discussion in #18899 (comment)

mattcaswell
mattcaswell approved these changes Sep 9, 2022
View reviewed changes
Copy link
Member

@mattcaswell mattcaswell left a comment

Choose a reason for hiding this comment

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

Approved assuming CIs are happy with it.

@mattcaswell mattcaswell added branch: master Merge to master branch approval: review pending This pull request needs review by a committer labels Sep 9, 2022
@hlandau hlandau added approval: done This pull request has the required number of approvals and removed approval: review pending This pull request needs review by a committer labels Sep 9, 2022
beldmit
beldmit approved these changes Sep 9, 2022
View reviewed changes
Copy link
Member

@beldmit beldmit left a comment

Choose a reason for hiding this comment

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

LGTM

@kiiwyz
Copy link

kiiwyz commented Sep 9, 2022

😘

kiiwyz
kiiwyz reviewed Sep 9, 2022
View reviewed changes
Copy link

@kiiwyz kiiwyz left a comment

Choose a reason for hiding this comment

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

♥️

@openssl-machine
Copy link
Collaborator

24 hours has passed since 'approval: done' was set, but as this PR has been updated in that time the label 'approval: ready to merge' is not being automatically set. Please review the updates and set the label manually.

@t8m
Copy link
Member

t8m commented Sep 12, 2022

Merged to master branch. Thank you for your contribution.

@t8m t8m closed this Sep 12, 2022
openssl-machine pushed a commit that referenced this pull request Sep 12, 2022
@baentsch @t8m
update oqsprovider/liboqs to v0.7.2
0b66422 
Reviewed-by: Matt Caswell <matt@openssl.org>
Reviewed-by: Hugo Landau <hlandau@openssl.org>
Reviewed-by: Dmitry Belyavskiy <beldmit@gmail.com>
Reviewed-by: Richard Levitte <levitte@openssl.org>
Reviewed-by: Tomas Mraz <tomas@openssl.org>
(Merged from #19184)
sftcd pushed a commit to sftcd/openssl that referenced this pull request Sep 24, 2022
@baentsch @sftcd
update oqsprovider/liboqs to v0.7.2
642e5cd 
Reviewed-by: Matt Caswell <matt@openssl.org>
Reviewed-by: Hugo Landau <hlandau@openssl.org>
Reviewed-by: Dmitry Belyavskiy <beldmit@gmail.com>
Reviewed-by: Richard Levitte <levitte@openssl.org>
Reviewed-by: Tomas Mraz <tomas@openssl.org>
(Merged from openssl#19184)
@baentsch baentsch mentioned this pull request Sep 29, 2022
Closed
t8m pushed a commit to t8m/openssl that referenced this pull request Nov 4, 2022
@baentsch @t8m
update oqsprovider/liboqs to v0.7.2
a9de7f3 
Reviewed-by: Matt Caswell <matt@openssl.org>
Reviewed-by: Hugo Landau <hlandau@openssl.org>
Reviewed-by: Dmitry Belyavskiy <beldmit@gmail.com>
Reviewed-by: Richard Levitte <levitte@openssl.org>
Reviewed-by: Tomas Mraz <tomas@openssl.org>
(Merged from openssl#19184)

(cherry picked from commit 0b66422)
t8m pushed a commit to t8m/openssl that referenced this pull request Nov 4, 2022
@baentsch @t8m
update oqsprovider/liboqs to v0.7.2
6aafb11 
Reviewed-by: Matt Caswell <matt@openssl.org>
Reviewed-by: Hugo Landau <hlandau@openssl.org>
Reviewed-by: Dmitry Belyavskiy <beldmit@gmail.com>
Reviewed-by: Richard Levitte <levitte@openssl.org>
Reviewed-by: Tomas Mraz <tomas@openssl.org>
(Merged from openssl#19184)

(cherry picked from commit 0b66422)
t8m pushed a commit to t8m/openssl that referenced this pull request Nov 9, 2022
@baentsch @t8m
update oqsprovider/liboqs to v0.7.2
b80ef9d 
Reviewed-by: Matt Caswell <matt@openssl.org>
Reviewed-by: Hugo Landau <hlandau@openssl.org>
Reviewed-by: Dmitry Belyavskiy <beldmit@gmail.com>
Reviewed-by: Richard Levitte <levitte@openssl.org>
Reviewed-by: Tomas Mraz <tomas@openssl.org>
(Merged from openssl#19184)

(cherry picked from commit 0b66422)
openssl-machine pushed a commit that referenced this pull request Nov 11, 2022
@baentsch @t8m
update oqsprovider/liboqs to v0.7.2
afb67ad 
Reviewed-by: Matt Caswell <matt@openssl.org>
Reviewed-by: Hugo Landau <hlandau@openssl.org>
Reviewed-by: Dmitry Belyavskiy <beldmit@gmail.com>
Reviewed-by: Richard Levitte <levitte@openssl.org>
Reviewed-by: Tomas Mraz <tomas@openssl.org>
(Merged from #19184)

(cherry picked from commit 0b66422)
beldmit pushed a commit to beldmit/openssl that referenced this pull request Dec 26, 2022
@baentsch @beldmit
update oqsprovider/liboqs to v0.7.2
f49ea10 
Reviewed-by: Matt Caswell <matt@openssl.org>
Reviewed-by: Hugo Landau <hlandau@openssl.org>
Reviewed-by: Dmitry Belyavskiy <beldmit@gmail.com>
Reviewed-by: Richard Levitte <levitte@openssl.org>
Reviewed-by: Tomas Mraz <tomas@openssl.org>
(Merged from openssl#19184)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@kiiwyz kiiwyz kiiwyz left review comments

@levitte levitte levitte approved these changes

@hlandau hlandau hlandau approved these changes

@t8m t8m t8m approved these changes

@mattcaswell mattcaswell mattcaswell approved these changes

@beldmit beldmit beldmit approved these changes

Assignees
No one assigned
Labels
approval: done This pull request has the required number of approvals branch: master Merge to master branch
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
8 participants
@baentsch @kiiwyz @openssl-machine @t8m @levitte @hlandau @mattcaswell @beldmit