steveloughran commented on code in PR #8214: URL: https://github.com/apache/hadoop/pull/8214#discussion_r2755562878
########## 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: ok, if mvnw is listed as the install mechanism, it's a lot simpler to detail as it does the validation. Docs can just say "use an up to date version of maven; the mvnw command will download one and validate it before execution" -- 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]
