kou opened a new pull request, #38340:
URL: https://github.com/apache/arrow/pull/38340

   ### Rationale for this change
   
   Our bundled GoogleTest has transitive dependency. So we can omit explicit 
`GTest::gtest` linking because `GTest::gtest_main`/`GTest::gmock` has 
transitive dependency.
   
   But GoogleTest related targets found by CMake's `FindGTest.cmake` don't have 
transitive dependency. So our code base needs to link `GTest::gtest` explicitly.
   
   We can remove needless links by setting transitive dependency to GoogleTest 
related targets found by CMake's `FindGTest.cmake`.
   
   ### What changes are included in this PR?
   
   * Set transitive dependencies to `GTest::gtest_main` and `GTest::gmock`
   * Remove needless links
   
   ### Are these changes tested?
   
   Yes.
   
   ### Are there any user-facing changes?
   
   No.


-- 
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: github-unsubscr...@arrow.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Reply via email to