-
Notifications
You must be signed in to change notification settings - Fork 766
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
Add output parameter with path to installed JDK #65
Comments
This is actually worse than that: As a workaround, I'm using (with a previous call with # Workaround for https://github.com/actions/setup-java/issues/65
- name: Backup JAVA_HOME
id: java-home
run: echo "::set-output name=path::$JAVA_HOME" I could have used a |
This allows calling the action multiple times in the same job and retrieving the path and/or version in other steps. Fixes actions#65
This allows calling the action multiple times in the same job and retrieving the path and/or version in other steps. Fixes actions#65
This allows calling the action multiple times in the same job and retrieving the path and/or version in other steps. Fixes actions#65
This allows calling the action multiple times in the same job and retrieving the path and/or version in other steps. Fixes actions#65
This allows calling the action multiple times in the same job and retrieving the path and/or version in other steps. Fixes actions#65
Nmnbbb |
Bjjk |
Bnnk no nn
|
When calling the action several times, it will overwrite the
JAVA_HOME
environment variable and prepend it to thePATH
, effectively shadowing a previous run of the action.Fortunately, it will also set a
JAVA_HOME_<version>_<architecture>
environment variable, but this could be more idiomatic as an output parameter (in addition to the environment variable, which can also be useful on its own: #44 (comment))This would be a great addition to #21
That way, one could write something like:
(here, the build is still running with JDK 8, but it coud configure its tests to actually use another JDK)
The text was updated successfully, but these errors were encountered: