Ticket #2940 enhancement closed fixed

Opened 5 years ago

Last modified 5 years ago

Need first-class man .xhtml generation code

Reported by: radix Owned by:
Priority: highest Milestone: twisted-8.0
Component: release management Keywords:
Cc: therve Branch: branches/man-builder-2940
Author: therve Launchpad Bug:

Description

admin/process-docs generates .xhtml and then .html files from our man pages. We need to do this in a way that's well -tested and -documented with python code. Then we should distribute the output both on the web site and with the release tarball.

Unfortunately this requires fixing a bug in lore in its man page processor (ticket to be linked here soon).

Change History

1

Changed 5 years ago by therve

  • cc therve added

What's the problem with lore exactly? Is it related to #1256?

2

Changed 5 years ago by radix

This ticket isn't actually blocked on that other issue. It can be done independently.

I think therve is investigating the other issue now which will hopefully result in another ticket.

3

Changed 5 years ago by therve

  • branch set to branches/man-builder-2940
  • author set to therve

(In [22423]) Branching to 'man-builder-2940'

4

Changed 5 years ago by therve

(In [22424]) Basic implementation of ManBuilder.

Refs #2940

5

Changed 5 years ago by therve

  • owner changed from radix to therve

6

Changed 5 years ago by therve

The lore issue is #3017.

7

Changed 5 years ago by therve

  • owner changed from therve to radix
  • keywords review added
  • priority changed from normal to highest

Alright, this deserves a first look to check if it's in the right direction.

8

Changed 5 years ago by radix

  • keywords review removed
  • owner changed from radix to therve
  1. I don't think we need a deleteInput option. We don't delete man files after converting them to lore.
  2. The docstring of ManBuilder.build could be improved. The summary should probably be "Generate Lore input files from man files."

Looks good! Thanks for working on this.

9

Changed 5 years ago by therve

(In [22580]) Process review comments.

Refs #2940

10

Changed 5 years ago by therve

  • keywords review added
  • owner changed from therve to radix

Done!

11

Changed 5 years ago by radix

  • keywords review removed
  • owner changed from radix to therve

Please fix the >80 column lines in test_release.py. After that you can merge. Thanks!

12

Changed 5 years ago by therve

(In [22583]) Fit into 80 columns.

Refs #2940

13

Changed 5 years ago by therve

  • status changed from new to closed
  • resolution set to fixed

(In [22584]) Merge man-builder-2940

Author: therve Reviewer: radix Fixes #2940

Add a tested object to generate Lore files from man pages.

14

Changed 5 years ago by therve

(In [22585]) Revert r22584: windows slaves failures.

Refs #2940

15

Changed 5 years ago by therve

  • status changed from closed to reopened
  • resolution fixed deleted 
[FAIL]: twisted.python.test.test_release.ManBuilderTestCase.test_build

Traceback (most recent call last):
  File "c:\twistedbuildbot24\WXP32-full2.5-scmikes-select\Twisted\twisted\python\test\test_release.py", line 535, in test_build
    self.assertEquals(output, "<html><head>\n<title>MANHOLE.1</title>"
twisted.trial.unittest.FailTest: not equal:
a = '<html><head>\r\n<title>MANHOLE.1</title></head>\r\n<body>\r\n\r\n<h1>MANHOLE.1</h1>\r\n\r\n<h2>NAME</h2>\r\n\r\n<p>manhole - Connect to a Twisted Manhole service\r\n</p>\r\n\r\n<h2>SYNOPSIS</h2>\r\n\r\n<p><strong>manhole</strong> </p>\r\n\r\n<h2>DESCRIPTION</h2>\r\n\r\n<p>manhole is a GTK interface to Twisted Manhole services. You can execute python\r\ncode as if at an interactive Python console inside a running Twisted process\r\nwith this.</p>\r\n\r\n</body>\r\n</html>\r\n'
b = '<html><head>\n<title>MANHOLE.1</title></head>\n<body>\n\n<h1>MANHOLE.1</h1>\n\n<h2>NAME</h2>\n\n<p>manhole - Connect to a Twisted Manhole service\n</p>\n\n<h2>SYNOPSIS</h2>\n\n<p><strong>manhole</strong> </p>\n\n<h2>DESCRIPTION</h2>\n\n<p>manhole is a GTK interface to Twisted Manhole services. You can execute python\ncode as if at an interactive Python console inside a running Twisted process\nwith this.</p>\n\n</body>\n</html>\n'

16

Changed 5 years ago by therve

  • owner changed from therve to radix
  • keywords review added
  • status changed from reopened to new

Just in case of.

17

Changed 5 years ago by radix

  • owner changed from radix to therve
  • keywords review removed

+1

18

Changed 5 years ago by therve

  • status changed from new to closed
  • resolution set to fixed

(In [22591]) Merge man-builder-2940

Author: therve Reviewer: radix Fixes #2940

Add a tested object to generate Lore files from man pages.

19

Changed 2 years ago by <automation>

  • owner therve deleted
Note: See TracTickets for help on using tickets.