eclipse-equinox / p2

Eclipse Public License 2.0
14 stars 40 forks source link

Workaround for Jenkins builds failing with "Java heap space" errors #464

Closed iloveeclipse closed 7 months ago

iloveeclipse commented 7 months ago

Fixes https://github.com/eclipse-equinox/p2/issues/463

github-actions[bot] commented 7 months ago

Test Results

    9 files  ±0      9 suites  ±0   34m 33s :stopwatch: -16s 2 183 tests ±0  2 179 :white_check_mark: ±0   4 :zzz: ±0  0 :x: ±0  6 639 runs  ±0  6 628 :white_check_mark: ±0  11 :zzz: ±0  0 :x: ±0 

Results for commit aab09715. ± Comparison against base commit 4327aa9c.

:recycle: This comment has been updated with latest results.

iloveeclipse commented 7 months ago

@laeubi : no idea what is wrong, but maven seem to read JVM argument as maven option:

Error:  Error executing Maven.
org.apache.commons.cli.ParseException: Unrecognized maven.config file entries: [mx2g]
mickaelistria commented 7 months ago

I think you mean Jenkins, not Gerrit.

iloveeclipse commented 7 months ago

I think you mean Jenkins, not Gerrit.

Doesn't work either.

akurtakov commented 7 months ago

"Please add https://github.com/eclipse-platform/eclipse.platform.releng.aggregator/blob/master/.mvn/jvm.config to this repository as well, it should help with that OOM errors as currently only 1GB of 4G is used by default." but you put it in maven.config. It's well documented and explained in https://maven.apache.org/configure.html .

iloveeclipse commented 7 months ago

"Please add https://github.com/eclipse-platform/eclipse.platform.releng.aggregator/blob/master/.mvn/jvm.config to this repository as well, it should help with that OOM errors as currently only 1GB of 4G is used by default." but you put it in maven.config. It's well documented and explained in https://maven.apache.org/configure.html .

I'm allowed to make mistakes, OK?

akurtakov commented 7 months ago

"Please add https://github.com/eclipse-platform/eclipse.platform.releng.aggregator/blob/master/.mvn/jvm.config to this repository as well, it should help with that OOM errors as currently only 1GB of 4G is used by default." but you put it in maven.config. It's well documented and explained in https://maven.apache.org/configure.html .

I'm allowed to make mistakes, OK?

Sure, that's why I point you to the detailed documentation for reference in the future.