pan3793 commented on code in PR #8214: URL: https://github.com/apache/hadoop/pull/8214#discussion_r2740450683
########## BUILDING.txt: ########## @@ -63,7 +63,8 @@ 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.9 or later. + Download and Install Maven manually from https://maven.apache.org/install.html Review Comment: let's replace this with the real commands ```suggestion $ 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 $ sudo ln -s /opt/apache-maven-3.9.11/bin/mvn /usr/bin ``` Oops, I have the wrong memory, the dev container actually uses Maven 3.9.11, not 3.9.9, sorry for the misleading. Could you update the docs to use 3.9.11? -- 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]
