Skip to content

Commit 06c7513

Browse files
committed
Stop crashes because we're out of memory by disabling t7880
All of our recent memory-related tests failures since #1030 was merged seem to be caused by t7880.scala. It tries to intentionally trigger an OutOfMemoryError, however since we don't pass -Xmx to our run tests it's possible that this we fill up the memory of our host before we reach the maximum heap size of the JVM. Ideally, we would specify a -Xmx for run tests (scalac uses 1 GB), unfortunately in the version of partest we use this is tricky because we need to set the system property "partest.java_opts". If we upgrade our partest to the latest release, we can instead specify it by setting the argument `javaOpts` of the constructor of `SuiteRunner`, see scala/scala-partest@7c4659e
1 parent 0e32220 commit 06c7513

File tree

1 file changed

+1
-0
lines changed

1 file changed

+1
-0
lines changed

tests/run/t7880.scala renamed to tests/pending/run/t7880.scala

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,4 @@
1+
// Do "git log" on this file to know why it's been moved to pending
12
object Test extends dotty.runtime.LegacyApp {
23
// This should terminate in one way or another, but it shouldn't loop forever.
34
try {

0 commit comments

Comments
 (0)