cleanup

facebookincubator · Apr 20, 2024 · 5935723 · 5935723
1 parent 5ba1b62
commit 5935723
Show file tree

Hide file tree

Showing 4 changed files with 4 additions and 4 deletions.
diff --git a/CMake/resolve_dependency_modules/README.md b/CMake/resolve_dependency_modules/README.md
@@ -22,7 +22,7 @@ by Velox. See details on bundling below.
 | xz                | default         | No       |
 | zstd              | default         | No       |
 | openssl           | default         | No       |
-| protobuf          | 26 (exact)      | Yes      |
+| protobuf          | 26              | Yes      |
 | boost             | 1.77.0          | Yes      |
 | flex              | 2.5.13          | No       |
 | bison             | 3.0.4           | No       |

diff --git a/CMake/resolve_dependency_modules/protobuf.cmake b/CMake/resolve_dependency_modules/protobuf.cmake
@@ -13,7 +13,7 @@
 # limitations under the License.
 include_guard(GLOBAL)
 
-set(VELOX_PROTOBUF_BUILD_VERSION 21.4)
+set(VELOX_PROTOBUF_BUILD_VERSION 26.1)
 set(VELOX_PROTOBUF_BUILD_SHA256_CHECKSUM
     6c5e1b0788afba4569aeebb2cfe205cb154aa01deacaba0cd26442f3b761a836)
 string(

diff --git a/CMakeLists.txt b/CMakeLists.txt
@@ -471,7 +471,7 @@ if(${VELOX_BUILD_MINIMAL_WITH_DWIO}
    OR ${VELOX_ENABLE_SUBSTRAIT})
   # Locate or build protobuf.
   set_source(Protobuf)
-  resolve_dependency(Protobuf 3.21 EXACT)
+  resolve_dependency(Protobuf 3.26)
   include_directories(${Protobuf_INCLUDE_DIRS})
 endif()
 

diff --git a/scripts/setup-centos8.sh b/scripts/setup-centos8.sh
@@ -122,7 +122,7 @@ function install_fmt {
 }
 
 function install_protobuf {
-  wget_and_untar https://github.com/protocolbuffers/protobuf/releases/download/v26.4/protobuf-all-21.4.tar.gz protobuf
+  wget_and_untar https://github.com/protocolbuffers/protobuf/releases/download/v26.1/protobuf-26.1.tar.gz protobuf
   (
     cd protobuf
     ./configure --prefix=/usr