Opened 4 years ago

Closed 3 years ago

#4619 defect closed fixed (fixed)

doc/core/man/twistd.1 seems outdated

Reported by: jafd Owned by: thijs
Priority: normal Milestone:
Component: core Keywords: documentation, manpage
Cc: thijs Branch:
Author: Launchpad Bug:

Description

There is a couple of things in man 1 twistd which can be seen as typos, and some describe things which apparently are not there anymore.

First, there is some tautology. A fragment near --rundir is as follows:

       -d, --rundir <directory>
              Change to a supplied directory before running (default: .).
       -u, --uid <uid>
              The uid to run as (default: don't change).
       -g, --gid <gid>
              The gid to run as (default: don't change).

and then a bit further:

       -g, --plugin <plugin name>
              Read config.tac from a plugin package, as with -y.
       --syslog
              Log to syslog instead of a file.
       -u, --uid <uid>
              The uid to run as.
       -g, --gid <gid>
              The gid to run as.

First, I believe the --plugin parameter went away at some point, as twistd doesn't display it altogether. Second, -g --plugin conflicts with -g --gid. And third, --uid and --gid are described in two places.

Attachments (1)

twistd-man.patch (1.5 KB) - added by thijs 3 years ago.

Download all attachments as: .zip

Change History (8)

comment:1 Changed 4 years ago by jafd

  • Keywords manpage added

comment:2 Changed 4 years ago by <automation>

  • Owner glyph deleted

Changed 3 years ago by thijs

comment:3 Changed 3 years ago by thijs

  • Cc thijs added
  • Keywords documentation review added; docs removed

comment:4 Changed 3 years ago by jesstess

  • Owner set to thijs

Thanks for reporting this, jafd, and thanks for working up a patch, thijs.

Looks good, please merge.

comment:5 Changed 3 years ago by jesstess

  • Keywords review removed

comment:6 Changed 3 years ago by thijs

  • Status changed from new to assigned

#5125 was a duplicate but merged, so I'll just apply the rest of the patch which removes the --plugin option.

comment:7 Changed 3 years ago by thijs

  • Resolution set to fixed
  • Status changed from assigned to closed

(In [32006]) Apply twistd-man.patch: Remove old --plugin option and update dates.

Author: thijs
Reviewer: jesstess
Fixes: #4619

Note: See TracTickets for help on using tickets.