Opened 12 years ago
Closed 11 years ago
#4585 enhancement closed fixed (fixed)
Mark `t.i.process.Process` and `t.i.process.PTYProcess` as implementing IProcessTransport
| Reported by: | Screwtape | Owned by: | |
|---|---|---|---|
| Priority: | normal | Milestone: | |
| Component: | core | Keywords: | documentation |
| Cc: | Branch: |
branches/process-interface-4585
branch-diff, diff-cov, branch-cov, buildbot |
|
| Author: | exarkun |
Description
Currently the IProcessTransport documentation claims that the only process transport in Twisted is for Win32, which is a total lie. t.i.process.Process and t.i.process.PTYProcess also implement it, and are very likely much more solid implementations.
We should mark both those classes as implementing that interface.
(this was filed once before as #4479, but that ticket got derailed into a different documentation issue)
Change History (9)
comment:1 Changed 11 years ago by
comment:2 Changed 11 years ago by
| Author: | → exarkun |
|---|---|
| Branch: | → branches/process-interface-4585 |
(In [30045]) Branching to 'process-interface-4585'
comment:3 Changed 11 years ago by
comment:4 Changed 11 years ago by
comment:6 Changed 11 years ago by
| Keywords: | review removed |
|---|---|
| Owner: | set to Jean-Paul Calderone |
test_process imports verifyObject and then doesn't use it.
I think "# Oh, the transport is available synchronously, how convenient." is a bit, um, sort of passive-aggressive or something. Can you make it a bit less snarky?
Fix these and land, please.
comment:7 Changed 11 years ago by
comment:8 Changed 11 years ago by
| Resolution: | → fixed |
|---|---|
| Status: | new → closed |
comment:9 Changed 11 years ago by
| Owner: | Jean-Paul Calderone deleted |
|---|
#4648 was a duplicate of this.