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.

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

Cherry-pick OpenSSL 3.1.2 onto 2.4 #84

Merged
merged 7 commits into from
Aug 22, 2023

Conversation

fwcd
Copy link
Member

@fwcd fwcd commented Aug 21, 2023

Motivation: Trying to debug Mixxx with a 2.4 env on arm64 macOS crashes at launch due to a bug in OpenSSL 3.1.0, which presumably has been fixed in 3.1.2:

While the 2.5 branch is already based on a more recent version of the upstream master (see #78), the 2.4 branch still uses the buggy version of OpenSSL, hence this PR which merges the upstream master that includes OpenSSL 3.1.2.

@fwcd fwcd marked this pull request as ready for review August 21, 2023 19:10
@daschuer
Copy link
Member

This PR is huge. Since the beta, I have not merged from master to not have to much untested code in our release.

Copy link
Member

@daschuer daschuer left a comment

Choose a reason for hiding this comment

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

Can you only cherry pick the commits to the ports/openssl folder?

@fwcd
Copy link
Member Author

fwcd commented Aug 21, 2023

That sounds like a reasonable fix, I'll look into that. Should I place it in the overlay folder (to avoid messy merge conflicts in the future), with the potential downside of having to manually vendor it for now?

lbermes and others added 7 commits August 21, 2023 23:07
* Added option to enable fips build for openssl

I only implement and tested it for windows as I have no option to do this under linux

* Adjusted port version

* Updated to support only for Windows

* Corrected Formatting

* Corrected SHA for vcpkg x-add-version --all

* Changes requested by review

Removed the not needed FEATURE FIPS
Initialize INSTALL_FIPS as empty

* Corrected SHA for vcpkg x-add-version --all

* Removed Windows only support

* Corrected SHA for vcpkg x-add-version --all

* Added "supports": "!static"

As the provider conecpts needs dynamic linkage

* Corrected SHA for vcpkg x-add-version --all
* Corrected fipsmodule.cnf install

move it to the same position as openssl.cnf, both are only examples of openssl

* Adjusted port version

* Adjusted SHA for vcpkg x-add-version --all

---------

Co-authored-by: Billy Robert O'Neal III <[email protected]>
Do not try to install file that does not exist.

Fixes microsoft#31170

Signed-off-by: Frank Lichtenheld <[email protected]>
* [openssl] Use bash from PATH rather then /bin (microsoft#32288)

The configure script used to invoke python during the build process contains
a shebang that invokes /bin/bash independend of whatever bash would usually
resolve to.

This breaks compilation on FreeBSD, where bash isn't part of the operating
system.

Rather then /bin/bash, invoke /usr/bin/env bash, which resolves the bash
command by iterating PATH

* [openssl] regenerate versions registries (microsoft#32288)
* [openssl] update to 3.1.2

* update version

---------

Co-authored-by: Zhao Liu <[email protected]>
* [openssl] Fix win64 perlasm scheme

* CI
@daschuer
Copy link
Member

No just use git cherry-pick I have done that in this in the past and that will merge cleanly later. We use overlays for changes that are not part of upstream.

@fwcd fwcd changed the title Merge upstream master (into 2.4) Cherry-pick OpenSSL 3.1.2 onto 2.4 Aug 21, 2023
@fwcd
Copy link
Member Author

fwcd commented Aug 21, 2023

Done

Copy link
Member

@daschuer daschuer left a comment

Choose a reason for hiding this comment

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

LGTM, Thank you. Waiting for CI

@daschuer daschuer merged commit d4dab88 into mixxxdj:2.4 Aug 22, 2023
3 checks passed
@fwcd fwcd deleted the mixxx/merge-master-2.4 branch August 23, 2023 18:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

8 participants