pan3793 commented on code in PR #7780:
URL: https://github.com/apache/hadoop/pull/7780#discussion_r2188865898


##########
BUILDING.txt:
##########
@@ -434,14 +436,15 @@ Installing required dependencies for clean install of 
macOS 10.14:
 * Install native libraries, only openssl is required to compile native code,
 you may optionally install zlib, lz4, etc.
   $ brew install openssl
-* Protocol Buffers 3.21.12 (required to compile native code)
-  $ curl -L 
https://github.com/protocolbuffers/protobuf/archive/refs/tags/v3.21.12.tar.gz > 
protobuf-3.21.12.tar.gz
-  $ tar -zxvf protobuf-3.21.12.tar.gz && cd protobuf-3.21.12
-  $ ./autogen.sh
-  $ ./configure
-  $ make
-  $ make check
-  $ make install
+* Protocol Buffers 3.25.5 (required to build native code)
+  $ curl -L 
https://github.com/protocolbuffers/protobuf/archive/refs/tags/v3.25.5.tar.gz > 
protobuf-3.25.5.tar.gz
+  $ curl -L 
https://github.com/abseil/abseil-cpp/archive/refs/tags/20230802.1.tar.gz > 
abseil-cpp-20230802.1.tar.gz
+  $ tar -zxvf protobuf-3.25.5.tar.gz
+  $ tar -zxvf abseil-cpp-20230802.1.tar.gz --strip-components 1 -C 
protobuf-3.25.5/third_party/abseil-cpp
+  $ cd protobuf-3.25.5
+  $ cmake -S . -B build -Dprotobuf_BUILD_TESTS=OFF
+  $ cmake --build build --parallel $(nproc)
+  $ cmake --install build
   $ protoc --version

Review Comment:
   now it returns 25.5 (seems protobuf changed version policy recently)
   
   ```
   chengpan@a1847ca27697:~/hadoop$ protoc --version
   libprotoc 25.5
   ```



-- 
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: common-issues-unsubscr...@hadoop.apache.org

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


---------------------------------------------------------------------
To unsubscribe, e-mail: common-issues-unsubscr...@hadoop.apache.org
For additional commands, e-mail: common-issues-h...@hadoop.apache.org

Reply via email to