Skip to content

[qtbase] set CMAKE_OSX_DEPLOYMENT_TARGET#43607

Closed
JavierMatosD wants to merge 4 commits intomicrosoft:masterfrom
JavierMatosD:investigate_qt
Closed

[qtbase] set CMAKE_OSX_DEPLOYMENT_TARGET#43607
JavierMatosD wants to merge 4 commits intomicrosoft:masterfrom
JavierMatosD:investigate_qt

Conversation

@JavierMatosD
Copy link
Copy Markdown
Contributor

@JavierMatosD JavierMatosD commented Feb 3, 2025

Attempt to mitigate https://codereview.qt-project.org/c/qt/qtbase/+/606910

I confirmed that this issue occurs on macOS 15 and can be mitigated by setting VCPKG_OSX_DEPLOYMENT_TARGET. I also verified that downstream ports remain functional by successfully building libqglviewer.

BillyONeal added a commit to BillyONeal/vcpkg that referenced this pull request Feb 3, 2025
Copy link
Copy Markdown
Member

@BillyONeal BillyONeal left a comment

Choose a reason for hiding this comment

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

Confirming that this works by looking at build results in #43509 . (To be clear, this PR should merge first. But the build results that matter are in that one)

@BillyONeal BillyONeal added the depends:different-pr This PR or Issue depends on a PR which has been filed label Feb 4, 2025
BillyONeal added a commit to BillyONeal/vcpkg that referenced this pull request Feb 5, 2025
BillyONeal added a commit to BillyONeal/vcpkg that referenced this pull request Feb 5, 2025
@BillyONeal
Copy link
Copy Markdown
Member

qtmultimedia broke for the same reason so I gave 2bfaa8e a shot

@FrankXie05 FrankXie05 added info:needs-maintainer-attention Lets the current 'on rotation' vcpkg maintainer know they need to look at this. info:internal labels Feb 5, 2025
@BillyONeal
Copy link
Copy Markdown
Member

2bfaa8e passed arm64 testing and is already included in that PR, so I'm going to close that one but mark you as a co-author of the other one.

@BillyONeal BillyONeal closed this Feb 5, 2025
@fwcd fwcd mentioned this pull request Feb 19, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

depends:different-pr This PR or Issue depends on a PR which has been filed info:needs-maintainer-attention Lets the current 'on rotation' vcpkg maintainer know they need to look at this.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants