pan3793 commented on code in PR #8214: URL: https://github.com/apache/hadoop/pull/8214#discussion_r2741349010
########## BUILDING.txt: ########## @@ -63,7 +63,17 @@ 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 + $ sudo ln -s /opt/apache-maven-3.9.11/bin/mvn /usr/bin + $ sudo nano /etc/profile.d/maven.sh + $ export M2_HOME=/opt/maven + $ export MAVEN_HOME=/opt/maven + $ export PATH=${M2_HOME}/bin:${PATH} + $ sudo chmod +x /etc/profile.d/maven.sh + $ source /etc/profile.d/maven.sh Review Comment: below setup is unnecessary if we create a soft link `/usr/bin/mvn`, please keep things simple ```suggestion ``` -- 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]
