rok commented on code in PR #48601:
URL: https://github.com/apache/arrow/pull/48601#discussion_r2669083428
##########
.github/workflows/cpp.yml:
##########
@@ -367,8 +367,12 @@ jobs:
export CMAKE_BUILD_PARALLEL_LEVEL=$NUMBER_OF_PROCESSORS
ci/scripts/cpp_build.sh "$(pwd)" "$(pwd)/build"
- name: Download Timezone Database
+ if: matrix.msystem_upper == 'CLANG64'
shell: bash
- run: ci/scripts/download_tz_database.sh
+ run: |
+ # TODO(GH-48743): Clang64 uses vendored date library which needs
tzdata
+ # https://github.com/apache/arrow/issues/48743
+ ci/scripts/download_tz_database.sh
Review Comment:
Same as [previous
comment](https://github.com/apache/arrow/pull/48601#discussion_r2669079994).
Clang64 on msys2 comes without std::chrono. We only skip tests that fail due to
gcc timezone bug. We keep other zone related tests (there are many).
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
