[tor-bugs] #4257 [Torctl]: Stem / TorCtl Integration
Tor Bug Tracker & Wiki
torproject-admin at torproject.org
Tue Oct 18 16:17:35 UTC 2011
#4257: Stem / TorCtl Integration
-------------------------+--------------------------------------------------
Reporter: atagar | Owner:
Type: enhancement | Status: new
Priority: normal | Milestone:
Component: Torctl | Version:
Keywords: | Parent:
Points: | Actualpoints:
-------------------------+--------------------------------------------------
As the saying goes: commit early, commit often. Rather than completing
stem then tossing it over the wall, the stem-integration branch of my
TorCtl repo [1] has a bite sized commit to start making use of it:
https://gitweb.torproject.org/atagar/pytorctl.git/commitdiff/d3a12d5724e44dac9096a56d0706bbc342efa304
This replaces TorCtl's _read_reply and related util functionality with
stem's ControlMessage class and parsing [2], which has unit [3] and
integration [4] tests. Doing a series of small commits like this will let
TorCtl users take advantage of the tests I'm writing for stem without a
need for changes on their part.
Cheers! -Damian
PS. Historically my TorCtl patches collect a lot of dust, so I'm testing
the waters a bit with this ticket. If there isn't time or interest on your
part to make this happen then it ain't gonna happen and I'll focus on stem
solely rather than using it as a means to improve TorCtl.
[1] https://gitweb.torproject.org/atagar/pytorctl.git/shortlog/refs/heads
/stem-integration
[2] https://gitweb.torproject.org/stem.git/blob/HEAD:/stem/types.py
[3] https://gitweb.torproject.org/stem.git/blob/HEAD:/test/unit/message.py
[4]
https://gitweb.torproject.org/stem.git/blob/HEAD:/test/integ/message.py
--
Ticket URL: <https://trac.torproject.org/projects/tor/ticket/4257>
Tor Bug Tracker & Wiki <https://trac.torproject.org/>
The Tor Project: anonymity online
More information about the tor-bugs
mailing list