Baunsgaard commented on code in PR #2388:
URL: https://github.com/apache/systemds/pull/2388#discussion_r2800044780


##########
docs/site/install.md:
##########
@@ -21,99 +21,126 @@ limitations under the License.
 {% endcomment %}
 -->
 
-This guide helps in the install and setup of SystemDS from source code.
-
-- [Install](#install)
-  - [Windows](#windows)
-  - [Ubuntu 22.04](#ubuntu-2204)
-    - [Testing](#testing)
-  - [MAC](#mac)
-- [2. Build the project](#2-build-the-project)
-- [3. Run A Component Test](#3-run-a-component-test)
+# Install SystemDS from Source
 
-Once the individual versions is set up skip to the common part of building the 
system.
+This guide helps in the install and setup of SystemDS from source code.
 
 ---
 
-## Install
+- [1. Install on Windows](#1-install-on-windows)
+- [2. Install on Ubuntu](#2-install-on-ubuntu-2204--2404)
+- [3. Install on macOS](#3-install-on-macos)
+- [4. Build the Project](#4-build-the-project)
+- [5. Run a Component Test](#5-run-a-component-test)
+- [6. Next Steps](#6-next-steps)
+
+Once the individual environment is set up, you can continue with the common 
build steps below.
 
 ---
 
-### Windows
+# 1. Install on Windows
 
-First setup java and maven to compile the system note the java version is 17, 
we suggest using Java OpenJDK 17.
+First setup Java and maven to compile the system note the Java version is 17, 
we suggest using Java OpenJDK 17.
 
 - <https://openjdk.org/>
 - <https://maven.apache.org/download.cgi?.>
 
-Setup your environment variables with JAVA_HOME and MAVEN_HOME. Using these 
variables add the JAVA_HOME/bin and MAVEN_HOME/bin to the path environment 
variable. An example of setting it for java can be found here: 
<https://www.thewindowsclub.com/set-java_home-in-windows-10>
+Setup your environment variables with JAVA_HOME and MAVEN_HOME. Using these 
variables add the JAVA_HOME/bin and MAVEN_HOME/bin to the path environment 
variable. An example of setting it for Java can be found here: 
<https://www.thewindowsclub.com/set-java_home-in-windows-10>
 
-To run the system we also have to setup some Hadoop and spark specific 
libraries. These can be found in the SystemDS repository. To add this, simply 
take out the files, or add 'src/test/config/hadoop_bin_windows/bin' to PATH. 
Just like for JAVA_HOME set a HADOOP_HOME to the environment variable without 
the bin part, and add the %HADOOP_HOME%/bin to path.
+To run the system we also have to setup some Hadoop and Spark specific 
libraries. These can be found in the SystemDS repository. To add this, simply 
take out the files, or add 'src/test/config/hadoop_bin_windows/bin' to PATH. 
Just like for JAVA_HOME set a HADOOP_HOME to the environment variable without 
the bin part, and add the `%HADOOP_HOME%\bin` to path.
 
-Finally if you want to run systemds from command line, add a SYSTEMDS_ROOT 
that points to the repository root, and add the bin folder to the path.
+On Windows, cloning large repositories via GitHub Desktop may stall in some 
environments. If this happens, cloning via the Git command line is a reliable 
alternative.
+Example:
+```bash
+git clone https://github.com/apache/systemds.git 
+```
 
-To make the build go faster set the IDE or environment variables for java: 
'-Xmx16g -Xms16g -Xmn1600m'. Here set the memory to something close to max 
memory of the device you are using.
+To make the build go faster set the IDE or environment variables for Java: 
'-Xmx16g -Xms16g -Xmn1600m'. Here set the memory to something close to max 
memory of the device you are using.
 
 To start editing the files remember to import the code style formatting into 
the IDE, to keep the changes of the files consistent.
 
 A suggested starting point would be to run some of the component tests from 
your IDE.
 
----
+# 2. Install on Ubuntu (22.04 / 24.04)
 
-### Ubuntu 22.04
+### 2.1 Install Java 17 and Maven
 
-First setup java and maven to compile the system note that the java version is 
17.
+First setup Java, maven and git to compile the system note that the Java 
version is 17.
 
 ```bash
-sudo apt install openjdk-17-jdk
-sudo apt install maven
+sudo apt update
+sudo apt install openjdk-17-jdk maven
+sudo apt install -y git
 ```
 
 Verify the install with:
-
 ```bash
 java -version
 mvn -version
+git --version
 ```
 
 This should return something like:
+```bash
+openjdk 17.x.x
+Apache Maven 3.x.x
+git version 2.x.x
+```
+
+### 2.2 Set JAVA_HOME for Javadocs
 
+Set `JAVA_HOME` (required for generating Javadocs during the Maven build):
 ```bash
-openjdk 17.0.11 2024-04-16
-OpenJDK Runtime Environment Temurin-17.0.11+9 (build 17.0.11+9)
-OpenJDK 64-Bit Server VM Temurin-17.0.11+9 (build 17.0.11+9, mixed mode, 
sharing)
+export JAVA_HOME=$(dirname $(dirname $(readlink -f $(which javac))))
+export PATH="$JAVA_HOME/bin:$PATH"
+```
 
-Apache Maven 3.9.9 (8e8579a9e76f7d015ee5ec7bfcdc97d260186937)
-Maven home: /home/usr/Programs/maven
-Java version: 17.0.11, vendor: Eclipse Adoptium, runtime: 
/home/usr/Programs/jdk-17.0.11+9
-Default locale: en_US, platform encoding: UTF-8
-OS name: "linux", version: "6.8.0-59-generic", arch: "amd64", family: "unix"
+### 2.3 Clone Source Code
+
+Clone the source code:
+```bash
+cd /opt
+git clone https://github.com/apache/systemds.git
+cd systemds
 ```
 
-#### Testing
+### 2.4 Testing
 
-R is required to be install to run the test suite, since many tests are 
constructed to compare output with common R packages.
-One option to install this is to follow the guide on the following link: 
<https://linuxize.com/post/how-to-install-r-on-ubuntu-20-04/>
+R should be installed to run the test suite, since many tests are constructed 
to compare output with common R packages. One option to install this is to 
follow the guide on the following link: 
<https://linuxize.com/post/how-to-install-r-on-ubuntu-20-04/>
 
-At the time of writing the commands to install R 4.0.2 are:
+R can be installed using the CRAN repository.
+
+**Ubuntu 22.04** 

Review Comment:
   both 



-- 
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]

Reply via email to