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

fix compile error (SIZE_MAX not found) on HP-UX #21562

Closed
wants to merge 1 commit into from
Closed

fix compile error (SIZE_MAX not found) on HP-UX #21562

wants to merge 1 commit into from

Conversation

johnkohl-hcl
Copy link
Contributor

Fixes #21554

CLA: trivial

Checklist

mattcaswell
mattcaswell approved these changes Jul 26, 2023
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.

I agree this is trivial

@mattcaswell mattcaswell added branch: master Merge to master branch approval: review pending This pull request needs review by a committer labels Jul 26, 2023
hlandau
hlandau approved these changes Jul 26, 2023
View reviewed changes
Copy link
Member

@hlandau hlandau left a comment

Choose a reason for hiding this comment

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

Agree trivial

@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 Jul 26, 2023
@paulidale paulidale added cla: trivial One of the commits is marked as 'CLA: trivial' triaged: bug The issue/pr is/fixes a bug labels Jul 27, 2023
@paulidale
Copy link
Contributor

Agreed trivial.

@paulidale paulidale added the tests: exempted The PR is exempt from requirements for testing label Jul 27, 2023
@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.

@paulidale
Copy link
Contributor

Merged, thanks for the fix.

@paulidale paulidale closed this Jul 28, 2023
openssl-machine pushed a commit that referenced this pull request Jul 28, 2023
@johnkohl-hcl @paulidale
fix compile error (SIZE_MAX not found) on HP-UX
7f14656 
Fixes #21554

CLA: trivial

Reviewed-by: Hugo Landau <hlandau@openssl.org>
Reviewed-by: Matt Caswell <matt@openssl.org>
Reviewed-by: Paul Dale <pauli@openssl.org>
(Merged from #21562)
@johnkohl-hcl johnkohl-hcl deleted the ssl_priority_queue_fix_hpux branch July 28, 2023 12:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@hlandau hlandau hlandau approved these changes

@mattcaswell mattcaswell mattcaswell approved these changes

@paulidale paulidale paulidale 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 cla: trivial One of the commits is marked as 'CLA: trivial' tests: exempted The PR is exempt from requirements for testing triaged: bug The issue/pr is/fixes a bug
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

compile error on HP-UX (ssl/priority_queue.c) using HP compilers
5 participants
@johnkohl-hcl @paulidale @openssl-machine @hlandau @mattcaswell