Change os.proc.env to os.SubProcess.env

build.sc

@@ -55,8 +55,7 @@ trait MiMaChecks extends Mima {
   override def mimaBinaryIssueFilters: T[Seq[ProblemFilter]] = Seq(
     ProblemFilter.exclude[ReversedMissingMethodProblem]("os.PathConvertible.isCustomFs"),
     // this is fine, because ProcessLike is sealed (and its subclasses should be final)
-    ProblemFilter.exclude[ReversedMissingMethodProblem]("os.ProcessLike.joinPumperThreadsHook"),
-    ProblemFilter.exclude[MissingTypesProblem]("os.proc$")
+    ProblemFilter.exclude[ReversedMissingMethodProblem]("os.ProcessLike.joinPumperThreadsHook")
   )
 }
 

os/src/ProcessOps.scala

@@ -251,19 +251,6 @@ case class proc(command: Shellable*) {
   def pipeTo(next: proc): ProcGroup = ProcGroup(Seq(this, next))
 }
 
-object proc {
-
-  /**
-   * The env passed by default to child processes
-   */
-  val env = new scala.util.DynamicVariable[Map[String, String]](sys.env)
-
-  // TODO: Delete when in binary compatibity breaking window
-  def andThen[T](f: proc => T): Seq[Shellable] => T = s => f(apply(s))
-  // TODO: Delete when in binary compatibity breaking window
-  def compose[T](f: T => Seq[Shellable]): T => proc = t => apply(f(t))
-}
-
 /**
  * A group of processes that are piped together, corresponding to e.g. `ls -l | grep .scala`.
  * You can create a `ProcGroup` by calling `.pipeTo` on a [[proc]] multiple times.
@@ -509,7 +496,7 @@ private[os] object ProcessOps {
       }
 
     if (propagateEnv) {
-      addToProcessEnv(os.proc.env.value)
+      addToProcessEnv(os.SubProcess.env.value)
     }
 
     addToProcessEnv(env)

os/src/SubProcess.scala

@@ -162,6 +162,11 @@ class SubProcess(
 
 object SubProcess {
 
+  /**
+   * The env passed by default to child processes
+   */
+  val env = new scala.util.DynamicVariable[Map[String, String]](sys.env)
+
   /**
    * A [[BufferedWriter]] with the underlying [[java.io.OutputStream]] exposed
    *

os/test/src/SubprocessTests.scala

@@ -153,7 +153,7 @@ object SubprocessTests extends TestSuite {
         val before = envValue()
         assert(before == "")
 
-        os.proc.env.withValue(Map("TEST_ENV_FOO" -> "bar")) {
+        os.SubProcess.env.withValue(Map("TEST_ENV_FOO" -> "bar")) {
           val res = envValue()
           assert(res == "bar")
         }