You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
From the maintainer Li Haoyi: I'm putting a 500USD bounty on this issue, payable by bank transfer on a merged PR implementing this.
The various Jvm.* methods are a mess. #3770 makes them marginally more messy, but they've been messy for years and could use a cleanup.
The goal of this issue is to consolidate all subprocess and classloader spawning operations in Jvm.scala into (approximately) the following four signatures
Jvm.spawn and Jvm.call intentionally follow the signatures of os.spawn and os.call. Jvm.spawnClassloader and Jvm.callClassloader are in-memory variations of the theme, with no subprocess-related parameters and an API tweaked to work in-memory.
All existing subprocess/classloader APIs in Jvm.scala should be refactored to go through the four APIs above. The other APIs should all be deprecated.
All code outside Jvm.scala spawning subprocesses or classloaders should go through those four APIs,
We should ensure all tests pass both with and without change (2) above to ensure that the behavior of the existing methods is maintained (at least as far as they are covered by Mill's existing test suite)
The text was updated successfully, but these errors were encountered:
From the maintainer Li Haoyi: I'm putting a 500USD bounty on this issue, payable by bank transfer on a merged PR implementing this.
The various
Jvm.*
methods are a mess. #3770 makes them marginally more messy, but they've been messy for years and could use a cleanup.The goal of this issue is to consolidate all subprocess and classloader spawning operations in
Jvm.scala
into (approximately) the following four signaturesJvm.spawn
andJvm.call
intentionally follow the signatures ofos.spawn
andos.call
.Jvm.spawnClassloader
andJvm.callClassloader
are in-memory variations of the theme, with no subprocess-related parameters and an API tweaked to work in-memory.Jvm.scala
should be refactored to go through the four APIs above. The other APIs should all be deprecated.Jvm.scala
spawning subprocesses or classloaders should go through those four APIs,The text was updated successfully, but these errors were encountered: