-
Notifications
You must be signed in to change notification settings - Fork 56
Bump Arrow port version for compatibility with vcpkg master #593
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
Conversation
|
Bumping the port version fixed the Linux native library builds, but the macos-15-intel tests are now failling. There was a runner image update (https://github.com/actions/runner-images/releases/tag/macos-15%2F2025.1126), which includes an XCode update. And it looks like Edit: Looks like the default XCode version is 16.4 so I don't think that update is relevant. I'm not sure what has changed to cause this but it still looks related to the runner image update as the tests still fail after reverting vcpkg to the same version used in the last good build. Both builds seem to use the same C++ compiler version (17.0.0.17000013). |
|
Similar looking issue here indicates we might be linking to the wrong C++ standard library: https://stackoverflow.com/questions/77374525/c-exception-not-caught-on-clang-17-0-3 |
|
I think we should merge this PR to fix the Linux builds and follow up in another issue for the MacOS test failures. |
|
Closing, this has been superseded by #599 |
Looks like setting
CMAKE_SYSTEM_PROCESSORin the older port version caused builds to fail in CI due to microsoft/vcpkg#48496 even though we aren't cross compiling: https://github.com/G-Research/ParquetSharp/actions/runs/19750737197/job/56593190898#step:12:5078