Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

INSTALLING.MD misses some dependencies #1616

Closed
U65535F opened this issue Feb 27, 2025 · 1 comment · Fixed by #1618
Closed

INSTALLING.MD misses some dependencies #1616

U65535F opened this issue Feb 27, 2025 · 1 comment · Fixed by #1618

Comments

@U65535F
Copy link

U65535F commented Feb 27, 2025

To install the dependencies, the guide says:

Install dependencies. During installation, use default=all by leaving the input blank and pressing enter.
64-bit: pacman -S mingw-w64-x86_64-toolchain make mingw-w64-x86_64-cmake git
32-bit: pacman -S mingw-w64-i686-toolchain make mingw-w64-i686-cmake git

And then it says to install JDK 21,

On all platforms, install Java JDK 21:

curl -s "https://get.sdkman.io" | bash
sdk install java 21.0.2.fx-librca

But it seems like that script requires unzip and zip packages to be installed on the system. So it would be better if we just add those packages to the dependency installation section in the installing.md.

How to reproduce:

  1. Install MSYS2 (Windows) freshly
  2. Do not install any packages, only execute the commands listed in the installing.md.
  3. When executing the SDK installer script, it will fail saying that Please install unzip on your system ...
@U65535F
Copy link
Author

U65535F commented Feb 27, 2025

Additionally, it is also not written in the guide that you have to restart your MSYS2 after you install sdkman. It will display that the command sdk is not found if you directly try to install. Either you have to resource the bash_profile using source ~/.bash_profile or you have to restart the MSYS2 to get it working.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant