steveloughran commented on code in PR #8214: URL: https://github.com/apache/hadoop/pull/8214#discussion_r2742939793
########## BUILDING.txt: ########## @@ -63,7 +63,11 @@ Refer to dev-support/docker/Dockerfile): $ sudo apt-get update $ sudo apt-get -y install openjdk-17-jdk * Maven - $ sudo apt-get -y install maven + Most package managers do not install required version of Maven i.e. 3.9.11 or later. + Following commands download and install Maven manually Review Comment: `Use the following commands to download and install Maven manually` ########## BUILDING.txt: ########## @@ -63,7 +63,11 @@ Refer to dev-support/docker/Dockerfile): $ sudo apt-get update $ sudo apt-get -y install openjdk-17-jdk * Maven - $ sudo apt-get -y install maven + Most package managers do not install required version of Maven i.e. 3.9.11 or later. Review Comment: Replace with ``` Do not as of January 2026 install an up to date version of Maven. ``` ########## BUILDING.txt: ########## @@ -63,7 +63,11 @@ Refer to dev-support/docker/Dockerfile): $ sudo apt-get update $ sudo apt-get -y install openjdk-17-jdk * Maven - $ sudo apt-get -y install maven + Most package managers do not install required version of Maven i.e. 3.9.11 or later. + Following commands download and install Maven manually + $ curl -L https://repo.maven.apache.org/maven2/org/apache/maven/apache-maven/3.9.11/apache-maven-3.9.11-bin.tar.gz > apache-maven-3.9.11-bin.tar.gz + $ tar -zxvf apache-maven-3.9.11-bin.tar.gz -C /opt Review Comment: add a checksum/signature validation stage if we are putting in code for people to run, or at least recommend it. ```bash curl -L https://repo.maven.apache.org/maven2/org/apache/maven/apache-maven/3.9.11/apache-maven-3.9.11-bin.tar.gz.asc > apache-maven-3.9.11-bin.tar.gz.asc curl -L https://downloads.apache.org/maven/KEYS > KEYS gpg --import KEYS gpg --verify apache-maven-3.9.11-bin.tar.gz.asc ``` -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
