Skip to content

Comments

Revert "TEST: make and use a fipsinstall script"#11592

Closed
levitte wants to merge 1 commit intoopenssl:masterfrom
levitte:remove-fipsinstall.pl
Closed

Revert "TEST: make and use a fipsinstall script"#11592
levitte wants to merge 1 commit intoopenssl:masterfrom
levitte:remove-fipsinstall.pl

Conversation

@levitte
Copy link
Member

@levitte levitte commented Apr 21, 2020

Unfortunately, this won't work on MacOS because of system integrity
measures on that platform, which clears DYLD_LIBRARY_PATH before
starting a sub-process executable.

Ref: https://developer.apple.com/library/archive/documentation/Security/Conceptual/System_Integrity_Protection_Guide/RuntimeProtections/RuntimeProtections.html

This reverts commit ae6b654.

Unfortunately, this won't work on MacOS because of system integrity
measures on that platform, which clears DYLD_LIBRARY_PATH before
starting a sub-process executable.

Ref: https://developer.apple.com/library/archive/documentation/Security/Conceptual/System_Integrity_Protection_Guide/RuntimeProtections/RuntimeProtections.html

This reverts commit ae6b654.
@levitte levitte added branch: master Applies to master branch approval: review pending This pull request needs review by a committer labels Apr 21, 2020
@levitte levitte added this to the 3.0.0 milestone Apr 21, 2020
@mattcaswell mattcaswell 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 Apr 21, 2020
@levitte
Copy link
Member Author

levitte commented Apr 21, 2020

Hmmm, shouldn't this be urgent?

@t8m t8m added the severity: urgent Fixes an urgent issue (exempt from 24h grace period) label Apr 21, 2020
@t8m
Copy link
Member

t8m commented Apr 21, 2020

It should.

@levitte
Copy link
Member Author

levitte commented Apr 21, 2020

I'll merge it later this afternoon

@paulidale
Copy link
Contributor

Agreed, urgent.

openssl-machine pushed a commit that referenced this pull request Apr 21, 2020
Unfortunately, this won't work on MacOS because of system integrity
measures on that platform, which clears DYLD_LIBRARY_PATH before
starting a sub-process executable.

Ref: https://developer.apple.com/library/archive/documentation/Security/Conceptual/System_Integrity_Protection_Guide/RuntimeProtections/RuntimeProtections.html

This reverts commit ae6b654.

Reviewed-by: Matt Caswell <[email protected]>
(Merged from #11592)
@levitte
Copy link
Member Author

levitte commented Apr 21, 2020

Merged.

1e78a50 Revert "TEST: make and use a fipsinstall script"

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

approval: done This pull request has the required number of approvals branch: master Applies to master branch severity: urgent Fixes an urgent issue (exempt from 24h grace period)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants