You are viewing a plain text version of this content. The canonical link for it is here.
Posted to pr@cassandra.apache.org by GitBox <gi...@apache.org> on 2021/10/24 10:59:28 UTC
[GitHub] [cassandra-website] michaelsembwever commented on a change in pull request #79: build and test documentation improvements
michaelsembwever commented on a change in pull request #79:
URL: https://github.com/apache/cassandra-website/pull/79#discussion_r735100589
##########
File path: site-content/source/modules/ROOT/pages/development/ide.adoc
##########
@@ -2,68 +2,117 @@
== Building and IDE Integration
-=== Building From Source
+=== Prerequisites
-Building Cassandra from source is the first important step in contributing
-to the Apache Cassandra project.
-You'll need to install http://www.oracle.com/technetwork/java/javase/downloads/index.html[Java 8], https://git-scm.com/[Git], and http://ant.apache.org/[Ant] first.
+Building Cassandra from source is the first important step in contributing to the Apache Cassandra project.
+You'll need to install:
-The source code for Cassandra is shared on the central Apache Git
-repository and organized by branch, one branch for each major version.
-You can access the code for the current development branch using:
+- Java 8, also optionally Java 11 - the most convenient and versatile way of installing Java is using https://sdkman.io/[The Software Development Kit Manager]
+- it lets you install multiple Java versions and easily switch between them
-// 01.15.21 - polandll - should this step be a fork rather than a clone?
+- GIT - you can use your package manager to install it
-[source, plaintext]
+- Ant - you can use your package manager to install it; note that you also need to install `ant-optional` module so for example on Debian-like operating systems you should install do that with the following command:
+
+[source,shell script]
Review comment:
I'm not convinced the expanded "Prerequisites" section here is adding much value or clarity.
Previously this was (basically) the `You’ll need to install Java 8, Git, and Ant first.` sentence.
My suggestion is just to change that sentence to
```
You will need to install Java (JDK 8 or 11), Git, and Ant (including ant-optional).
```
--
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: pr-unsubscribe@cassandra.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: pr-unsubscribe@cassandra.apache.org
For additional commands, e-mail: pr-help@cassandra.apache.org