You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@arrow.apache.org by we...@apache.org on 2018/12/12 14:16:36 UTC

[arrow] branch master updated: ARROW-4002: [C++][Gandiva] Remove needless CMake version check

This is an automated email from the ASF dual-hosted git repository.

wesm pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/arrow.git


The following commit(s) were added to refs/heads/master by this push:
     new 67506d9  ARROW-4002: [C++][Gandiva] Remove needless CMake version check
67506d9 is described below

commit 67506d94b762d0ea3d26ba0e2df1399e566d145b
Author: Kouhei Sutou <ko...@clear-code.com>
AuthorDate: Wed Dec 12 08:16:30 2018 -0600

    ARROW-4002: [C++][Gandiva] Remove needless CMake version check
    
    I could build Gandiva with CMake 3.7.2 and LLVM 6.0.0 on Debian
    stretch. But I disabled Gandiva JNI.
    
    Author: Kouhei Sutou <ko...@clear-code.com>
    
    Closes #3161 from kou/cpp-gandiva-remove-cmake-version-check and squashes the following commits:
    
    1506c546c <Kouhei Sutou>  Remove needless CMake version check
---
 cpp/src/gandiva/CMakeLists.txt | 3 ---
 1 file changed, 3 deletions(-)

diff --git a/cpp/src/gandiva/CMakeLists.txt b/cpp/src/gandiva/CMakeLists.txt
index 5d75aa2..5ef5738 100644
--- a/cpp/src/gandiva/CMakeLists.txt
+++ b/cpp/src/gandiva/CMakeLists.txt
@@ -15,9 +15,6 @@
 # specific language governing permissions and limitations
 # under the License.
 
-# LLVM/Clang is required by multiple subdirs.
-cmake_minimum_required(VERSION 3.11)
-
 project(gandiva)
 
 find_package(LLVM)