Commits

Sutou Kouhei authored 27a292f9b9a
PARQUET-1692: [C++] Don't use the same CMake variable name for thirdparty version and found version https://github.com/apache/arrow/pull/5900 introduced a new problem. If Thrift is found but found Thrift is old, `THRIFT_VERSION` in `FindThrift.cmake` overrides our `THRIFT_VERSION` in `versions.txt`. https://github.com/apache/arrow/pull/5946 tried to fix this problem by using another variable name, `FOUND_THRIFT_VERSION` in `FindThrift.cmake`. This can fix this but the root cause of this problem is using the same variable name for found library version and build target library version. This change uses `ARROW_${LIBRARY_NAME}_BUILD_VERSION` for build target library version and `${LIBRARY_NAME}_VERSION` for found library version (no change). This avoids variable conflicts. I choose `ARROW_${LIBRARY_NAME}_BUILD_VERSION` because we already use `ARROW_${LIBRARY_NAME}_REQUIRED_VERSION` such as `ARROW_BOOST_REQUIRED_VERSION`. Closes #5958 from kou/cpp-rename-bulid-version-variable-name and squashes the following commits: 625d11cc1 <Sutou Kouhei> Set TRHIFT_VERSION ce3e295b1 <Sutou Kouhei> Format d7807bcc3 <Sutou Kouhei> Improve variable name a4578ac65 <Sutou Kouhei> Use ARROW_${LIBRARY}_BUILD_VERSION for build version variable Authored-by: Sutou Kouhei <kou@clear-code.com> Signed-off-by: Sutou Kouhei <kou@clear-code.com>