Opened 17 years ago

Closed 17 years ago

#5376 closed (fixed)

Add --addrport option to testserver command

Reported by: Todd O'Bryan Owned by: Todd O'Bryan
Component: Core (Management commands) Version: dev
Severity: Keywords:
Cc: Triage Stage: Ready for checkin
Has patch: yes Needs documentation: no
Needs tests: no Patch needs improvement: no
Easy pickings: no UI/UX: no

Description

Because the testserver command takes fixtures as arguments, you can't currently set the IP address or port that it should use when it runs the server.

This ticket adds the option --addrport to testserver and just passes it to the runserver command.

Attachments (2)

testserver-addrport.patch (3.1 KB ) - added by Todd O'Bryan 17 years ago.
docs.patch (811 bytes ) - added by Todd O'Bryan 17 years ago.

Download all attachments as: .zip

Change History (12)

comment:1 by Todd O'Bryan, 17 years ago

Owner: changed from nobody to Todd O'Bryan
Status: newassigned

comment:2 by Adrian Holovaty, 17 years ago

Patch looks good, other than the fact that it should be made relative to your trunk directory -- i.e., the diff shouldn't contain "/home/tobryan1/workspace". Also, could you include a patch to docs/django-admin.txt? Here's all you need to know about documentation changes: http://www.djangoproject.com/documentation/contributing/#documentation-style

comment:3 by Adrian Holovaty, 17 years ago

Triage Stage: UnreviewedAccepted

comment:4 by Adrian Holovaty, 17 years ago

Needs documentation: set

by Todd O'Bryan, 17 years ago

Attachment: testserver-addrport.patch added

comment:5 by Todd O'Bryan, 17 years ago

Needs documentation: unset

comment:6 by Simon G. <dev@…>, 17 years ago

Triage Stage: AcceptedReady for checkin

comment:7 by Adrian Holovaty, 17 years ago

I'm working on reviewing this and checking it in.

comment:8 by Adrian Holovaty, 17 years ago

Resolution: fixed
Status: assignedclosed

(In [6204]) Fixed #5376 -- Added --addrport option to the 'testserver' command. Thanks, toddobryan

by Todd O'Bryan, 17 years ago

Attachment: docs.patch added

comment:9 by Todd O'Bryan, 17 years ago

Resolution: fixed
Status: closedreopened

There are two typos that are misleading. They're fixed in docs.patch.

comment:10 by Adrian Holovaty, 17 years ago

Resolution: fixed
Status: reopenedclosed

(In [6363]) FIxed #5376 -- Fixed typo in testserver docs

Note: See TracTickets for help on using tickets.
Back to Top