Albert Meltzer
Albert Meltzer
> > MUnitRunNotifier: use concurrent for Native munit#946 > > A shot in the dark until we get this resolved. What happens if you build with a scratch munit which...
> If I clone the kitbellew:4913 branch, any special tricks for building on macOS (or Ubuntu)? i build on my Mac, i don't remember any tricks.
> There have been a number of changes to `java.lang.Process*` in SN 0.5.7 and 0.5.8-SNAPSHOT. There are also some unresolved SN Issues. In particular #4264 mentions a deadlock in scalafmt....
@LeeTibbert i updated native dependency to 0.5.8-SNAPSHOT and modified munit-native (also snapshot) to log errors when a suite cannot be instantiated, and here's an error: ``` ==> X org.scalafmt.FidelityTest java.io.UncheckedIOException:...
> I see a `D:\`, is this being built on a Windows system (makes sense, if the defect in the base note is Windows). Yes, the failures are on Windows...
@LeeTibbert @WojciechMazur I seem to have found the culprit; the hang happens within GitOpsTest which executes a [`sys.process.Process`](https://github.com/scalameta/scalafmt/blob/e87eceb3986513302f49cf7d46ba8309bbdf3b3d/scalafmt-sysops/jvm-native/src/main/scala/org/scalafmt/sysops/PlatformRunOps.scala#L47). Anything you know about running processes under Windows which has changed since...
@tgodzik just fyi: scala-native v0.5.7 seems to have a [bug with `sys.process.Process` under Windows](https://github.com/scalameta/scalafmt/pull/4913#issuecomment-2932538412), we can't upgrade anything to that version in the dependency tree of this repo (or to...
@aairey should we merge this, despite multiple failures?
> > @aairey should we merge this, despite multiple failures? > > I don't think so. > > Someone who understands these test failures should take a look. in that...
> > @tgodzik is this something you could help review? > > I am not an expert in Docker things, but looks ok to me as much as I can...