-
-
Notifications
You must be signed in to change notification settings - Fork 32
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
CI: Extend Github Release Target #45
Comments
I am already trying stuff out on the |
It seems like Launch4j enables using an JRE located relative to the generated I will give it a try because I think that the Client would be more accessible to others if installing Java 8 is not a requirement. |
|
For the version consistency, I think we could migrate to version format like 2.1.0.0, which should be usable in all settings. |
Yes, separate issue please.
Unfortunately not. It requires the jre to be placed relatively to the .exe. For this reason installers are recommended.
I agree, Homebrew makes it very easy to install Java 8. However bundling the JRE is extremely easy (download JRE directory to a temporary directory and adding a single line to the |
I already wanted to mention this when I opened the issue bit somehow forgot to write this down. Currently 1.0.0 is hardcoded at some places and yes, your mentioned format should be used. |
We can do this manually before release with |
Right now it generates DMG (and github actions also generates it) but I do now know if it works as I do not have a mac. I would very much like to avoid having a specific distribution branch with specific scripts as it creates a bunch of problems, so in my opinion, if we can do it all through maven it might be alright to bundle JRE with it, but we can't I would leave it as it is.
The same problem, if we can somehow do it all through maven I am ok with it, otherwise, I would swallow the fact that the .exe will require java if we the error they get clearly states that they need java 1.8. Since most of the potential players already played minigolf on Playforia, they already know about java a most likely have it installed anyway, so it shouldn't be that big of a deal. If we somehow manage to download and bundle JRE with maven, however, we should think about binding these bundling actions to a different phase (maybe deploy?) so we don't download JRE with every |
One additional thing we could do is to create pre-release (2.1.0.0-BETA) to test if the release action works, to actually test it and to compare with this updated one. |
The changelog in the Release notes are intended to help drafting the actual release notes by having the commit messages as an reference. |
Features to be implemented:
The text was updated successfully, but these errors were encountered: