-
Notifications
You must be signed in to change notification settings - Fork 6.3k
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
[libmysql] Fix port on arm64-osx (compiler checks) #35930
[libmysql] Fix port on arm64-osx (compiler checks) #35930
Conversation
@microsoft-github-policy-service agree |
ports/libmysql/compiler-check.patch
Outdated
- CHECK_C_SOURCE_RUNS(" | ||
- int main() | ||
- { | ||
- return (__clang_major__ < 10); | ||
- }" HAVE_SUPPORTED_CLANG_VERSION) |
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.
IMO either this should become part of cross-build.patch
(because check_c_source_runs
is a cross-build blocker), or we simply add -DFORCE_UNSUPPORTED_COMPILER=1
to the cmake options.
(Sidenote: Is the XCode version equal to __clang_major__
?)
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.
Not sure how to modify it. Are you suggesting to keep it? It doesn't work for me as expected - HAVE_SUPPORTED_CLANG_VERSION
is empty after the function executes.
But the HAVE_SUPPORTED_CLANG_VERSION_COMPILED
is set to true
.
Regarding __clang_major__
, it's equal to XCode version (according to wiki). The conditional I'm suggesting using CMAKE_CXX_COMPILER_VERSION
works the same way.
8.0.34#1 is declared with 20cb416a4659c1f3d6cbf08fef69ebf51eb8e15c, but the local port has a different SHA 167e024cc411a99094f62e58913991caca22237f.
to add the new version. |
@Cheney-W done, thanks. The 8.0.34#1 was not released yet. I updated the database during the checks in the previous commit, but the hash diverged somehow. |
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 but someone else must approve.
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.
Will merge when build comes back
FIxes #35876
./vcpkg x-add-version --all
and committing the result.