Skip to content

Comments

Darwin platform allows to build on releases before Yosemite/ios 8.#16418

Closed
devnexen wants to merge 1 commit intoopenssl:masterfrom
devnexen:macos_old_releases
Closed

Darwin platform allows to build on releases before Yosemite/ios 8.#16418
devnexen wants to merge 1 commit intoopenssl:masterfrom
devnexen:macos_old_releases

Conversation

@devnexen
Copy link
Contributor

backport #16409

@github-actions github-actions bot added the severity: fips change The pull request changes FIPS provider sources label Aug 25, 2021
@t8m t8m added approval: review pending This pull request needs review by a committer branch: master Applies to master branch triaged: bug The issue/pr is/fixes a bug labels Aug 25, 2021

Choose a reason for hiding this comment

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

Should the version number here be 101000?

Availability.h (in the MacOSX10.10 SDK) defines:

--snip--
#define __MAC_10_0            1000
#define __MAC_10_1            1010
#define __MAC_10_2            1020
#define __MAC_10_3            1030
#define __MAC_10_4            1040
#define __MAC_10_5            1050
#define __MAC_10_6            1060
#define __MAC_10_7            1070
#define __MAC_10_8            1080
#define __MAC_10_9            1090
#define __MAC_10_10         101000
#define __MAC_10_10_2       101002
#define __MAC_10_10_3       101003
--snip--

Copy link
Contributor Author

Choose a reason for hiding this comment

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

true ... will update both PR.

@devnexen devnexen force-pushed the macos_old_releases branch from 8b72c4c to f1a01d1 Compare August 25, 2021 13:14
@paulidale paulidale 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 Aug 25, 2021
@openssl-machine openssl-machine added approval: ready to merge The 24 hour grace period has passed, ready to merge and removed approval: done This pull request has the required number of approvals labels Aug 27, 2021
@openssl-machine
Copy link
Collaborator

This pull request is ready to merge

openssl-machine pushed a commit that referenced this pull request Aug 27, 2021
backport #16409

Reviewed-by: Paul Dale <[email protected]>
Reviewed-by: Tomas Mraz <[email protected]>
(Merged from #16418)
@t8m
Copy link
Member

t8m commented Aug 27, 2021

Merged to master. Thank you for your contribution.

@t8m t8m closed this Aug 27, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

approval: ready to merge The 24 hour grace period has passed, ready to merge branch: master Applies to master branch severity: fips change The pull request changes FIPS provider sources triaged: bug The issue/pr is/fixes a bug

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants