#1693 closed defect (fixed)

flogtool doesn't get automatically provided

Reported by: zooko Owned by: warner
Priority: normal Milestone: 1.10.0
Component: packaging Version: 1.9.1
Keywords: doc packaging usability logging review-needed Cc:
Launchpad Bug:

Description

Wolfgang Strobl built tahoe-lafs per source:docs/quickstart.rst, and later, because source:docs/logging.rst had told him to, wanted to run flogtool. He was frustrated that there was no documentation explaining how to get a working flogtool script, and that the thing named ./support/bin/flogtool that he found gave an exception when executed:

https://tahoe-lafs.org/pipermail/tahoe-dev/2012-March/007159.html

To close this ticket, extend source:docs/logging.rst to explain how to get flogtool running, and possibly also extend the build system somehow so that it makes it easier to get a working flogtool script.

Attachments (2)

tahoe-debug-flogtool.darcs.patch (88.6 KB) - added by davidsarah at 2012-03-25T14:13:36Z.
tahoe-debug-flogtool-v2.darcs.patch (92.5 KB) - added by davidsarah at 2012-03-31T22:42:44Z.
Add 'tahoe debug flogtool' command, test for --help, and docs. This version gets the help synopses more correct, and changes the doc to say that this command is added in 1.10.0 rather than 1.9.2. fixes #1693

Download all attachments as: .zip

Change History (14)

comment:2 Changed at 2012-03-25T14:11:54Z by davidsarah

  • Keywords review-needed added
  • Owner changed from somebody to zooko

Changed at 2012-03-25T14:13:36Z by davidsarah

comment:3 Changed at 2012-03-31T21:29:11Z by davidsarah

  • Milestone changed from undecided to 1.10.0
  • Priority changed from minor to normal

comment:4 Changed at 2012-03-31T22:40:44Z by david-sarah@…

In 922ed08c25a1a142:

Document PYTHONPATH problem when running flogtool. refs #1693

Changed at 2012-03-31T22:42:44Z by davidsarah

Add 'tahoe debug flogtool' command, test for --help, and docs. This version gets the help synopses more correct, and changes the doc to say that this command is added in 1.10.0 rather than 1.9.2. fixes #1693

comment:5 Changed at 2012-04-01T00:22:36Z by davidsarah

  • Milestone changed from 1.11.0 to 1.10.0

comment:6 Changed at 2012-05-13T04:20:47Z by warner

  • Keywords review-needed removed

The patch looks good, and I'm landing it. One quibble about the docs it's modifying, though: the specific PYTHONPATH issue is when Foolscap was only installed as part of Tahoe, and therefore bin/foolscap wasn't available from outside the tahoe tree's copy. The exception cited in the original report looks like it was a pkg_resources problem (probably because pkg_resources was only installed as part of Tahoe). My quibble is that I want to avoid giving the impression that tahoe itself has to be on PYTHONPATH to use 'flogtool' against a Tahoe process. That's very much not the case: nothing in the logfiles or event stream requires the original application's source code to display. (there have been bugs in tahoe and/or foolscap which have made it seem like this must be necessary, but really, flogfiles are app-independent).

I may update the docs at some point to reflect this better.

comment:7 Changed at 2012-05-13T04:29:32Z by david-sarah <david-sarah@…>

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

In 379901bf8f2b4c17:

Add 'tahoe debug flogtool' command, test for --help, and docs. This version gets the help synopses more correct, and changes the doc to say that this command is added in 1.10.0 rather than 1.9.2. fixes #1693

comment:8 Changed at 2012-05-16T22:16:15Z by david-sarah@…

In 959248cd4917944f:

Change logging.rst to address warner's review comment. refs #1693

comment:9 Changed at 2012-05-16T22:17:04Z by davidsarah

  • Keywords review-needed added
  • Resolution fixed deleted
  • Status changed from closed to reopened

Post-hoc review needed for 959248cd4917944f.

comment:10 Changed at 2012-05-16T22:17:21Z by davidsarah

  • Owner changed from zooko to warner
  • Status changed from reopened to new

comment:11 Changed at 2012-05-16T22:20:48Z by davidsarah

Zooko was unclear on the advantage of tahoe debug flogtool over tahoe @flogtool. It is this:

tahoe @flogtool depends on there being a flogtool command on the PATH, which in turn depends on foolscap being installed. On Windows that command isn't installed correctly IIRC.

tahoe debug flogtool, OTOH, will work whenever Tahoe does, and will use the same foolscap, Twisted, and zope.interface that Tahoe uses (which is not strictly necessary but is a useful simplification).

comment:12 Changed at 2012-09-04T23:04:18Z by davidsarah

  • Resolution set to fixed
  • Status changed from new to closed
Note: See TracTickets for help on using tickets.