Closed
Bug 1140507
Opened 10 years ago
Closed 10 years ago
Release a new mozprocess to pick up recent changes
Categories
(Testing :: Mozbase, defect)
Tracking
(firefox39 fixed)
RESOLVED
FIXED
mozilla39
| Tracking | Status | |
|---|---|---|
| firefox39 | --- | fixed |
People
(Reporter: chmanchester, Assigned: chmanchester)
References
Details
Attachments
(1 file, 1 obsolete file)
No description provided.
| Assignee | ||
Comment 1•10 years ago
|
||
* Bug 1139722 - Allow use of nested job objects in mozprocess on windows versions that support them.;r=ahal
This patch detects when breaking away from the parent job isn't strictly necessary to manage processses in a new job to allow using nested job objects to manage processes and their children. Loss of job object functionality is handled in mozprocess as non-fatal, however mozrunner and others doing things like restarting firefox require it.
* bug 1137228 - use readline instead of iterator for reading mozprocess output. r=ahal
* Bug 794984 - [mozprocess] Add ability to separate stderr from stdout. r=ahal
A new parameter called 'processStderrLine' is added. When specified, stdout gets processed by the
'processOutputLine' callbacks and stderr is processed by the 'processStderrLine' callbacks. When
not specified, stderr is redirected to stdout which is the same default behaviour.
A side effect of this is that mozprocess now uses three threads to process output. One thread each
for stdout and stderr that reads output lines and stores them in a Queue as fast as possible, this
makes sure there is no blocking in stdout.write(). A third thread executes the callbacks.
* Bug 1060527 - [mozprocess] signal finished if IO completion port unexpectedly closed on windows, r=jgriffin
* Bug 1064960 - [mozprocess] Guard against the output thread trying to join with itself. r=ahal
* Bug 705864 - [mozprocess] mozprocess tests should use mozprocess.pid. r=ahal
* Bug 1067008 - [mozprocess] mozprocess.pid running_process should look at the entire command not just the executable name. r=ahal
| Assignee | ||
Comment 2•10 years ago
|
||
/r/4989 - Bug 1140507 - Bump mozprocess version and release to pypi to pick up recent changes.;r=ahal
Pull down this commit:
hg pull review -r 4ab3e302e082a2c36f6a3f8408cb922879670a59
Attachment #8574137 -
Flags: review?(ahalberstadt)
Updated•10 years ago
|
Attachment #8574137 -
Flags: review?(ahalberstadt) → review+
Comment 3•10 years ago
|
||
Comment on attachment 8574137 [details]
MozReview Request: bz://1140507/chmanchester
https://reviewboard.mozilla.org/r/4987/#review3995
Ship It!
| Assignee | ||
Comment 4•10 years ago
|
||
Comment 5•10 years ago
|
||
Assignee: nobody → cmanchester
Status: NEW → RESOLVED
Closed: 10 years ago
status-firefox39:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla39
| Assignee | ||
Comment 6•10 years ago
|
||
Attachment #8574137 -
Attachment is obsolete: true
Attachment #8619695 -
Flags: review+
| Assignee | ||
Comment 7•10 years ago
|
||
You need to log in
before you can comment on or make changes to this bug.
Description
•