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

Allow JREs #1542

Draft
wants to merge 3 commits into
base: main
Choose a base branch
from
Draft

Allow JREs #1542

wants to merge 3 commits into from

Conversation

quintesse
Copy link
Contributor

@quintesse quintesse commented Jan 17, 2023

This PR allows JREs to be used by Jbang. This should make it possible to use much of Jbang's functionality without having a JDK installed. Except for the most important of course, which is being able to build.

But this will let you for example list the available JDKs and decide which one you want to install, instead of Jbang first installing a default version, like 11, before it will allow you do do anything else.

@codecov
Copy link

codecov bot commented Jan 17, 2023

Codecov Report

Base: 0.00% // Head: 0.00% // No change to project coverage 👍

Coverage data is based on head (cff5b07) compared to base (006959a).
Patch coverage: 0.00% of modified lines in pull request are covered.

Additional details and impacted files
@@          Coverage Diff          @@
##            main   #1542   +/-   ##
=====================================
  Coverage   0.00%   0.00%           
=====================================
  Files        121     121           
  Lines       7409    7428   +19     
  Branches    1199    1203    +4     
=====================================
- Misses      7409    7428   +19     
Flag Coverage Δ
Linux 0.00% <0.00%> (ø)
Windows 0.00% <0.00%> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
src/main/java/dev/jbang/cli/App.java 0.00% <0.00%> (ø)
src/main/java/dev/jbang/cli/Info.java 0.00% <0.00%> (ø)
src/main/java/dev/jbang/cli/Jdk.java 0.00% <0.00%> (ø)
.../java/dev/jbang/dependencies/ModularClassPath.java 0.00% <0.00%> (ø)
src/main/java/dev/jbang/net/JdkManager.java 0.00% <0.00%> (ø)
src/main/java/dev/jbang/net/JdkProvider.java 0.00% <0.00%> (ø)
...a/dev/jbang/net/jdkproviders/JBangJdkProvider.java 0.00% <0.00%> (ø)
...va/dev/jbang/net/jdkproviders/PathJdkProvider.java 0.00% <0.00%> (ø)
src/main/java/dev/jbang/source/AppBuilder.java 0.00% <0.00%> (ø)
...java/dev/jbang/source/buildsteps/JarBuildStep.java 0.00% <0.00%> (ø)
... and 3 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

☔ View full report at Codecov.
📢 Do you have feedback about the report comment? Let us know in this issue.

@quintesse quintesse force-pushed the allow_jres branch 3 times, most recently from 682e137 to cff5b07 Compare February 2, 2023 11:48
@quintesse quintesse force-pushed the allow_jres branch 2 times, most recently from bdeb116 to 3f65088 Compare February 8, 2023 16:17
If it finds both a `java` and `javac` on the path that don't point to
the same JDK it will returns both options.
This can be used to determine which JDKs are actually JREs and which
ones are full JDKs.
@maxandersen
Copy link
Collaborator

rebase?

@quintesse
Copy link
Contributor Author

Still not marked as ready :-)
I'm just not 100% happy with it, even though it's a pretty nice feature.

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 this pull request may close these issues.

2 participants