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

New Updated version of JavaAppLauncher i.e 3.1.0 #51

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

833M0L3
Copy link

@833M0L3 833M0L3 commented Jan 24, 2021

The JavaAppLauncher that was currently in the Jar2app is outdated and was having a problem with a newer version of the Java runtime.

This new JavaAppLauncher from https://github.com/tofi86/universalJavaApplicationStub/tree/master/src , was recently updated to work with the newest Java runtime and fixes the few issues that the people were having while passing the arguments in -j.

@Lucina
Copy link
Contributor

Lucina commented Jan 25, 2021

This seems redundant as it is. You already have an option to use a custom stub with -e, and the default behaviour of having a binary stub is preferable over something that can't be signed (as per ddyer0 in #50).
Also, unfortunately this repository hasn't been actively maintained for years, even #38 which should be a huge boon hasn't been merged.

@833M0L3
Copy link
Author

833M0L3 commented Jan 25, 2021

Yeah, I just read the whole conversation in the comments. I guess I will delete this pull request then.

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