Opened 8 years ago

Closed 8 years ago

#6356 closed task (implemented)

LOADCONF wrapper method for Stem

Reported by: neena Owned by: neena
Priority: Medium Milestone:
Component: Archived/Stem Version:
Severity: Keywords:
Cc: atagar Actual Points:
Parent ID: Points:
Reviewer: Sponsor:


Stem must implement a wrapper method around LOADCONF control messages.

My branch implementing this is here. (It has has a few minor aesthetic changes to the code).

Child Tickets

Change History (3)

comment:1 Changed 8 years ago by neena

Owner: changed from atagar to neena
Status: newassigned

comment:2 Changed 8 years ago by neena

Status: assignedneeds_review

comment:3 Changed 8 years ago by atagar

Resolution: implemented
Status: needs_reviewclosed

Changes pushed, thanks!

Minor thing, but when you have a long list of things like...

+all = [
+ "descriptor",
+ "response",
+ "util",
+ "connection",
+ "control",
+ "prereq",
+ "process",
+ "socket",
+ "version"

... please include a comma on the last element. This makes reordering the elements a little less error prone.

[LOADCONF was added in Tor]

When a feature has a version requirement please add it to 'stem.version.Requirement' and add a version requirement for the test.

Here's the revisions that I made to your changes...

Cheers! -Damian

Note: See TracTickets for help on using tickets.