bazelbuild / rules_scala

Scala rules for Bazel
Apache License 2.0
360 stars 273 forks source link

Fix handling of cycles in AST traversal in Dependency Analyzer #1519

Closed liucijus closed 10 months ago

liucijus commented 10 months ago

Unfortunately I can't quickly figure out how to create a public repro for this one (any help would be appreciated). During the last few years I faced this issue 4 times, where scalac worker got into the infinite loop.

I've added explicit tracking of cycles using mutable.Set[Tree] of visited tree notes, which seems to be fixing the infinite loop problem.