-
Notifications
You must be signed in to change notification settings - Fork 3.4k
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
ARROW-7929: [C++] Align CMake target names to upstreams #6480
Conversation
This changes the declared imported targets for glog, gtest and thrift to match the exported names produced by each projects generated _Name_Targets.cmake file. This makes it possible to use `CMAKE_FIND_PACKAGE_PREFER_CONFIG=ON` in conjunction with `ARROW_DEPENDENCY_SOURCE=SYSTEM`.
Thanks for opening a pull request! Could you open an issue for this pull request on JIRA? Then could you also rename pull request title in the following format?
See also: |
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.
Looks solid, just one comment:
@kszucs Some Ursabot builds are failed. Could you confirm whether this is Ursabot related problem or this PR related problem? BTW, I want to rebuild failed Ursabot builds manually. What do I need to do for it? |
@kou login on the buildbot site and restart manually with the rebuild button, or... |
@ursabot build |
@kszucs Thanks! I didn't know |
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.
+1
No description provided.