-
Notifications
You must be signed in to change notification settings - Fork 75
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
bundle jre with windows #289
Conversation
@jcharlet Sounds reasonable to me. The thing will be making sure that when you have produced runs with the Bundled JRE and not a JRE which is already installed on the system, getting that right can be a bit tricky sometimes ;-) |
I download the jre from adoptopenjdk with ant maven plugin, then include it with the assembly plugin, no risk using the local jre! Ok then, solution agreed, I'll work on fixing the travis build. |
I don't understand why travis job can't download the jdk, and the logs do not say anything helpful, I want to ssh into it and try running things manually to see why it's stuck. @adamretter do we have debug mode available on travis job, is it something you could grant me access? https://docs.travis-ci.com/user/running-build-in-debug-mode/ Otherwise can I send a request to travis support team to enable it for our repository? |
@jcharlet I would suggest not using the Maven Ant plugin for the download, but rather to use maven-download-plugin - it should give you clearer error messages. It will also allow you to cache the downloads to speed up repetitive builds. |
that worked, thanks @adamretter ! |
…script to use that jre, packaged in droid-binary zip file
…at script to use that jre, packaged in dedicated droid-binary zip file
177bfd1
to
4e729d8
Compare
Provide now 2 packages:
Reasoning:
What do you think @adamretter ? Are you happy with that solution? I can otherwise do a few more commits to include a linux and an osx packages. Will need some help testing the mac version too :).
I did win64 btw, do we need win32? A bit concerned about creating loads of packages.