-
Notifications
You must be signed in to change notification settings - Fork 3.5k
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
[CI][C++] MinGW jobs failed with GTest::gmock_main related error #43134
Comments
CMake version may be related. Success: https://github.com/apache/arrow/actions/runs/9752783251/job/26916919495#step:7:87
Failure: https://github.com/apache/arrow/actions/runs/9775229516/job/26985190635?pr=43133#step:7:95
|
kou
added a commit
to kou/arrow
that referenced
this issue
Jul 3, 2024
9 tasks
kou
added a commit
to kou/arrow
that referenced
this issue
Jul 3, 2024
kou
added a commit
to kou/arrow
that referenced
this issue
Jul 3, 2024
kou
added a commit
to kou/arrow
that referenced
this issue
Jul 3, 2024
kou
added a commit
to kou/arrow
that referenced
this issue
Jul 4, 2024
kou
added a commit
to kou/arrow
that referenced
this issue
Jul 4, 2024
This probably can be added to 17.0.0 (marking it with the 17.0.0 milestone) |
kou
added a commit
to kou/arrow
that referenced
this issue
Jul 5, 2024
kou
added a commit
that referenced
this issue
Jul 5, 2024
### Rationale for this change `google_cloud_cpp_mocks` depends on `GTest::gmock_main` but it's built without `BUILD_TESTING`. google-cloud-cpp finds GoogleTest only with `BUILD_TESTING`. ### What changes are included in this PR? The recent google-cloud-cpp doesn't build `google_cloud_cpp_mocks` without `BUILD_TESTING`. Note that we can't use 2.23.0 or later because they can't be built with MinGW-w64. See also: * mingw-w64/mingw-w64#49 * googleapis/google-cloud-cpp#14436 ### Are these changes tested? Yes. ### Are there any user-facing changes? Yes. * GitHub Issue: #43134 Authored-by: Sutou Kouhei <kou@clear-code.com> Signed-off-by: Sutou Kouhei <kou@clear-code.com>
Issue resolved by pull request 43136 |
raulcd
pushed a commit
that referenced
this issue
Jul 5, 2024
### Rationale for this change `google_cloud_cpp_mocks` depends on `GTest::gmock_main` but it's built without `BUILD_TESTING`. google-cloud-cpp finds GoogleTest only with `BUILD_TESTING`. ### What changes are included in this PR? The recent google-cloud-cpp doesn't build `google_cloud_cpp_mocks` without `BUILD_TESTING`. Note that we can't use 2.23.0 or later because they can't be built with MinGW-w64. See also: * mingw-w64/mingw-w64#49 * googleapis/google-cloud-cpp#14436 ### Are these changes tested? Yes. ### Are there any user-facing changes? Yes. * GitHub Issue: #43134 Authored-by: Sutou Kouhei <kou@clear-code.com> Signed-off-by: Sutou Kouhei <kou@clear-code.com>
zanmato1984
pushed a commit
to zanmato1984/arrow
that referenced
this issue
Jul 9, 2024
…che#43136) ### Rationale for this change `google_cloud_cpp_mocks` depends on `GTest::gmock_main` but it's built without `BUILD_TESTING`. google-cloud-cpp finds GoogleTest only with `BUILD_TESTING`. ### What changes are included in this PR? The recent google-cloud-cpp doesn't build `google_cloud_cpp_mocks` without `BUILD_TESTING`. Note that we can't use 2.23.0 or later because they can't be built with MinGW-w64. See also: * mingw-w64/mingw-w64#49 * googleapis/google-cloud-cpp#14436 ### Are these changes tested? Yes. ### Are there any user-facing changes? Yes. * GitHub Issue: apache#43134 Authored-by: Sutou Kouhei <kou@clear-code.com> Signed-off-by: Sutou Kouhei <kou@clear-code.com>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Describe the bug, including details regarding any error messages, version, and platform.
https://github.com/apache/arrow/actions/runs/9775229516/job/26985190635?pr=43133#step:7:1001
Component(s)
C++, Continuous Integration
The text was updated successfully, but these errors were encountered: