#1420 closed enhancement (fixed)
adopt man page that bertagaz at ptitcanardnoir.org wrote
Reported by: | zooko | Owned by: | zooko |
---|---|---|---|
Priority: | minor | Milestone: | 1.9.0 |
Component: | documentation | Version: | 1.8.2 |
Keywords: | debian docs news | Cc: | |
Launchpad Bug: |
Attachments (3)
Change History (12)
Changed at 2011-06-17T03:35:19Z by zooko
comment:1 Changed at 2011-06-17T03:35:37Z by zooko
- Owner changed from somebody to warner
comment:2 Changed at 2011-07-22T20:41:42Z by davidsarah
comment:3 follow-up: ↓ 4 Changed at 2011-07-24T16:21:31Z by davidsarah
- Owner changed from warner to davidsarah
- Status changed from new to assigned
I asked on the tahoe-debian list where this should go in the source tree, and Julian Taylor confirmed that docs/man/tahoe.1 would be reasonable. He also gave a patch to the debian/rules file (not in our source tree) to install it to the right place.
The "tahoe [general options] [command] [command options]" synopsis is not correct: all options must currently go after the command. (See #166 about fixing that, but it's not fixed yet.) Also the command is not optional, except for the special case of tahoe --help.
The "runner command" feature is not supported by the /usr/bin/tahoe script installed by the Debian/Ubuntu packages. It isn't actually needed, since that script bypasses the support script, so you can just do
coverage run --branch /usr/bin/tahoe debug trial
or
python -3 /usr/bin/tahoe COMMAND [OPTION]...
for example.
I will attach a new patch with these and a couple of other formatting changes.
comment:4 in reply to: ↑ 3 Changed at 2011-07-24T17:19:07Z by davidsarah
Replying to davidsarah:
coverage run --branch /usr/bin/tahoe debug trial
Well, except that tahoe debug trial was added in v1.8.2, which isn't correctly packaged yet. But if you kludge around that (I did so by copying src/allmydata/scripts/debug.py over the installed version of Tahoe v1.7.1) and install the python-coverage package, then:
python-coverage run --branch /usr/bin/tahoe debug trial
does work.
comment:5 Changed at 2011-07-24T18:17:59Z by davidsarah
- Keywords review-needed added
- Owner davidsarah deleted
- Status changed from assigned to new
comment:6 Changed at 2011-07-25T01:24:49Z by marlowe
- Keywords reviewed added; review-needed removed
- Owner set to zooko
Reviewed patch and diff. Both look good to me.
comment:7 Changed at 2011-07-25T01:31:59Z by david-sarah@…
- Resolution set to fixed
- Status changed from new to closed
In da58fefd50c8cd85:
comment:8 Changed at 2011-07-25T01:32:19Z by davidsarah
- Keywords reviewed removed
comment:9 Changed at 2011-08-02T00:23:58Z by davidsarah
- Keywords news added
Proposed NEWS:
- A man page has been added in `<docs/man/tahoe.1>`_. (`#1420`_)
#984 was a duplicate.