[C++][CMake] cpp/src/arrow/util/logging.cc misses glog::glog dependency
#40331
Comments
kou
added a commit
to kou/arrow
that referenced
this issue
Mar 4, 2024
kou
added a commit
that referenced
this issue
Mar 4, 2024
#40332) ### Rationale for this change `cpp/src/arrow/util/logging.cc` may use glog. So `arrow_util` OBJECT target(s) must depend on `glog::glog`. ### What changes are included in this PR? Add missing `glog::glog` dependency to `${ARROW_UTIL_TARGETS}`. ### Are these changes tested? Yes. ### Are there any user-facing changes? Yes. * GitHub Issue: #40331 Authored-by: Sutou Kouhei <kou@clear-code.com> Signed-off-by: Sutou Kouhei <kou@clear-code.com>
|
Issue resolved by pull request 40332 |
thisisnic
pushed a commit
to thisisnic/arrow
that referenced
this issue
Mar 8, 2024
…ow_util (apache#40332) ### Rationale for this change `cpp/src/arrow/util/logging.cc` may use glog. So `arrow_util` OBJECT target(s) must depend on `glog::glog`. ### What changes are included in this PR? Add missing `glog::glog` dependency to `${ARROW_UTIL_TARGETS}`. ### Are these changes tested? Yes. ### Are there any user-facing changes? Yes. * GitHub Issue: apache#40331 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.
This is a problem by #39824.
cpp/src/arrow/util/logging.ccmay use glog. Soarrow_utilOBJECT target(s) must depend onglog::glog.Component(s)
C++
The text was updated successfully, but these errors were encountered: