-
-
Notifications
You must be signed in to change notification settings - Fork 925
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
AppVeyor: set MAVEN_OPTS=-Xmx3g #4345
Conversation
@olleolleolle sure let's try it. |
Are the m2 caches really working? Seems like a lot of downloading. (I'm not used to Maven work, so I may be way off.) |
Seen a few other |
@olleolleolle our build does download lots of crap once a day so it might be that build firing on appveyor...or not? |
the maven cache is emptied on AppVeyor when the pom file is updated: cache:
- C:\Users\appveyor\.m2\repository -> pom.xml Note; AppVeyor lacks a way to purges the cache manually unlike Travis-CI |
@mprins would it be possible to associate the cache with |
Just change that line in the appveyor.yml to point to pom.rb instead |
What is the needed amount of RAM to run the test suite in AppVeyor? |
yaaay - finally a 💚 ci :) ... @olleolleolle maybe do the OPTS similar to how its done on travis-ci (... but if its green already than this is better than the current setting -> can be tuned later) |
@kares An issue that I suspect is: AppVeyor is not run on all pushes? I can only see it run on my first commit to this PR. And, thank you for the values tips - I changed the YAML config for AppVeyor to that. Ugh: https://github.com/jruby/jruby/blob/master/.travis.yml#L12-L13 Should I recreate that logic? |
aaah, bumper - do not know much about AppVeyor setup ... guess @enebo will have to approve the merge. |
@olleolleolle that referenced logic in the |
@mprins For Java8 on AppVeyor, should I add Also: Do I need to have the data from JAVA_OPTS also in the MAVEN_OPTS? |
This fiddle-with-the-appveyor-build PR tries to get around the issue in AppVeyor that Maven options are handed in a way that generates warnings on Java 8.
Example of build with warnings: https://ci.appveyor.com/project/jnr/jruby/build/2880/job/shniu1cn7yluu99o#L1692
The changes are:
pom.rb
changesSee #4075.