twistd should print the subcommand name in its usage error messages

[twisted-trac]

	@wallrj reported
Trac ID	trac#6460
Type	enhancement
Created	2013-04-22 18:24:20Z

Certain twistd usage errors can cause confusion because they don't print the name of the subcommand. eg

[richard@zorin ~]$ twistd -n procmon -n

/home/richard/projects/Twisted/trunk/bin/twistd: option -n not recognized

In this case -n is a valid twistd option but an invalid twistd procmon option.

Another good reason for printing the subcommand sequence is that the error line is often preceded by a full usage message which pushes the original command line off the page. Having the subcommand in the error message will make it obvious in cases where you typed an unintended (but valid) subcommand.

eg twistd mail_pop instead of twistd mail_imap (assuming for a moment that those two plugins existed).

Attachments:

• twistd-subcommand-errors-6460.patch (1266 bytes) - added by rwall on 2013-04-22 18:27:58Z - An untested idea for a fix

Searchable metadata

trac-id__6460 6460
type__enhancement enhancement
reporter__rwall rwall
priority__normal normal
milestone__ 
branch__ 
branch_author__ 
status__closed closed
resolution__fixed fixed
component__core core
keywords__easy easy
time__1366655060000000 1366655060000000
changetime__1613438585267958 1613438585267958
version__None None
owner__eevelweezel eevelweezel

[twisted-trac]

	@wallrj commented

It should be quite easy to write a unit test for this and (if necessary) review and fix any existing unit tests that are broken by it.

[twisted-trac]

	Infinity set owner to Infinity Infinity set status to assigned

[twisted-trac]

	eevelweezel set owner to eevelweezel eevelweezel set status to new

PR located at: #1504

[twisted-trac]

	@glyph removed owner

[twisted-trac]

	@wsanchez set owner to @wsanchez @wsanchez set status to assigned

[twisted-trac]

	@wsanchez set owner to eevelweezel @wsanchez set status to new

[twisted-trac]

	@wsanchez set status to closed

[twisted-trac]

	@glyph commented

(Fixed in b1897c9 which didn't follow the commit message format)

[twisted-trac]

	@glyph commented

(It's not in review now that it's merged.)