Mark Hammond wrote:

> My problem would be that SetEnv() wont work as implied.  Eg:


You're the one making the implication. =)

> starter = Components[...].createInstance()
> starter.run(...) // create child process.
> starter.SetEnv(...) // Eeek - can't set in child after started.
> starter.GetEnv(...) // Eeek - child environment probably different now


Those sound like wonderful times for exceptions, to me.

You can't call init after execution starts either, I expect, and you 
can't get pid before execution starts.  People will just have to read 
the documentation (assuming that people write it =) ).

> Thus, I believe:
> * nsIProcess should grow a technique for specifying the child process's 
> startup environment - probably as a param to a "run" method.  No need 
> for a "get" at all.


"get" is kinda handy where you don't know what you're inheriting from 
the parent.  Imagine: I want to add "/opt/mozilla-apps/bin" to the $PATH 
before executing another process.  get and set.

Mike


Reply via email to