pan3793 commented on code in PR #8214: URL: https://github.com/apache/hadoop/pull/8214#discussion_r2744342730
########## 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: @steveloughran, if you look at the context, no installation does such verification. can we make the guide simple to match the context for now? later we can consider Maven Wrapper https://maven.apache.org/tools/wrapper/, then `./mvnw` would automatically trigger the requested version of Maven downloading, which is a much simpler way. (my previous try was abandoned because I didn't figure out how to let Yetus invoke `./mvnw` instead of `mvn` under `$PATH`, but it's simple if we keep `mvn` for `Yetus` and add a `./mvnw` as an alternative for developers) -- 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]
