-
Notifications
You must be signed in to change notification settings - Fork 73
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
Adds minor corrections to the open source guide (#1005)
* Clarified the steps required to install Java 17 on MacOS when using Homebrew. * Corrected the Java version to check post-install. * Fixed extra triple brace near database start step.
- Loading branch information
Showing
1 changed file
with
10 additions
and
4 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -26,10 +26,16 @@ You can also get started by using a [docker image](#docker-image) that has the b | |
|
||
Install `brew` following the website instructions: https://brew.sh/ | ||
|
||
Ensure that you have added the `cask-versions` [tap](https://github.com/Homebrew/homebrew-cask-versions). | ||
|
||
```sh | ||
brew tap homebrew/cask-versions | ||
``` | ||
|
||
Install `java 17` from `Temurin`: | ||
|
||
```sh | ||
brew install temurin17 | ||
brew install --cask temurin17 | ||
``` | ||
|
||
Check that `java 17` is now in use with `java -version`. If you need multiple versions of `java` consider using | ||
|
@@ -44,7 +50,7 @@ brew install maven | |
Optionally, install `Mysql 8` | ||
|
||
```sh | ||
brew install mysql@8 | ||
brew install mysql@8.0 | ||
``` | ||
|
||
Note, to install the exact same `maven` version as the wrapper: `brew install [email protected]` (check the instructions since it is key-only) . | ||
|
@@ -57,7 +63,7 @@ Install `java 17` from `OpenJDK`: | |
```sh | ||
sudo apt-get install openjdk-17-jdk | ||
``` | ||
Check that `java 8` is now in use with `java -version`. If not you, can set it as default with | ||
Check that `java 17` is now in use with `java -version`. If not you, can set it as default with | ||
`sudo update-java-alternatives -s /usr/lib/jvm/java-1.17.0-openjdk-amd64` (To check the list | ||
`sudo update-java-alternatives -l`). If you need multiple versions of `java` | ||
consider using something like `jenv` or `sdkman`. | ||
|
@@ -69,6 +75,7 @@ sudo apt-get install maven | |
``` | ||
|
||
Install `git` | ||
|
||
```sh | ||
sudo apt-get install git | ||
``` | ||
|
@@ -100,7 +107,6 @@ max_allowed_packet = 256M | |
[mysqld] | ||
default-time-zone = '+00:00' | ||
``` | ||
``` | ||
|
||
The server needs to be started/restarted. | ||
```sh | ||
|