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
[docs] Installation for Windows docs improved #413
[docs] Installation for Windows docs improved #413
Conversation
docs/source/installation.rst
Outdated
cd %USERPROFILE%\MyProjects\TornadoVM | ||
etc/setvars.cmd | ||
nmake /f Makefile.mak graal-jdk-21 BACKEND=opencl,ptx | ||
cd %USERPROFILE%\MyProjects |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this path may not exist for every user.
python -m venv .venv | ||
.venv\Scripts\activate.bat | ||
.\bin\windowsMicrosoftStudioTools2022.cmd | ||
python bin\tornadovm-installer --jdk jdk21 --backend=opencl |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think this command should change to be:
nmake /f Makefile.mak jdk21 BACKEND=opencl,ptx
The reason is that we are in the section of the manual installation, so we should point to the manual build command.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think in the feature we should only allow to install through the installer script. For now, let's maintain the nmake
approach as well. I will change this.
Co-authored-by: Thanos Stratikopoulos <34061419+stratika@users.noreply.github.com>
Co-authored-by: Thanos Stratikopoulos <34061419+stratika@users.noreply.github.com>
Co-authored-by: Thanos Stratikopoulos <34061419+stratika@users.noreply.github.com>
Co-authored-by: Thanos Stratikopoulos <34061419+stratika@users.noreply.github.com>
Co-authored-by: Thanos Stratikopoulos <34061419+stratika@users.noreply.github.com>
Co-authored-by: Thanos Stratikopoulos <34061419+stratika@users.noreply.github.com>
All comments resolved. |
Description
Improve the documentation for the installation of TornadoVM on Windows 10/11
Problem description
n/ a.
Backend/s tested
Mark the backends affected by this PR.
OS tested
Mark the OS where this PR is tested.
Did you check on FPGAs?
If it is applicable, check your changes on FPGAs.
How to test the new patch?
Link to the documentation.