-
Notifications
You must be signed in to change notification settings - Fork 6
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
MacOS Qt5 overlay clean up #45
Conversation
05a6f4e
to
3881cb4
Compare
overlay/osx/qt5-base/portfile.cmake
Outdated
if("${VCPKG_TARGET_ARCHITECTURE}" MATCHES "arm64") | ||
string(REPLACE | ||
"QMAKE_APPLE_DEVICE_ARCHS = x86_64" | ||
"QMAKE_APPLE_DEVICE_ARCHS = arm64" | ||
_tmp_contents ${_tmp_contents}) | ||
endif() |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
can you explain this? Is this a change from us or a patch from Qt5.15?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ups, this slips in from the M1 arm port.
I must have messed up the rebase.
I will rebase this again to remove that change.
3881cb4
to
8da1a73
Compare
Done. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
This is a set of patches that improves our Qt5.12 backport for MacOs