VirtusLab/besom

Set assumedEvictionErrorLevel to Level.Error

jchapuis opened this issue · 9 comments

jchapuis commented

There is an intrinsic eviction warning in the latest build of besom-core with com.lihaoyi:sourcecode . pprint depends on 0.2.7 while besom-core depends on 0.3.1.

Setting assumedEvictionErrorLevel := Level.Error should help in surfacing these issues.

lbialy commented

Interesting. We use scala-cli to build the whole thing and it didn't catch it, apparently.

jchapuis commented

Yes so this is actually quite problematic for us as we are using sbt with strict eviction avoidance. Because of this intrinsic compatibility issue in the library, in all projects using besom we have to add libraryDependencySchemes += "com.lihaoyi" %% "sourcecode" % "always"

lbialy commented

Would a quickfix release of 0.3.2 with correct dependency tree help?

  • 👍1
lbialy commented

@Gedochao are you aware if scala-cli has any way to prevent eviction-related issues like this? Docs are silent on this topic.

jchapuis commented

@lbialy maybe I should have answered directly, yes a quickfix release would help as right now this libraryDependencySchemes is propagating throughout our repos 🙂

  • 👍1
  • 🚀1
lbialy commented

For reference: the only way to validate this, for now, is to use scala-cli export . --sbt in core and then sbt evicted in dest directory. This shows possible versioning problems. I have fixed them manually in #500.

  • 👍1
lbialy commented

fixed in #500, will release 0.3.2 momentarily

  • 🎉1
lbialy commented

@jchapuis I have released 0.3.2-SNAPSHOT (sonatype snapshot repo) today, could you please verify that it solves this issue for you? I will release complete 0.3.2 version once you confirm it does indeed solve your issues.

jchapuis commented

@lbialy thanks! I can confirm it works 🎉