Opened 4 years ago

Closed 4 years ago

#20885 closed defect (fixed)

Portions of the tor.1 man page and html doc formatted incorrectly

Reported by: jryans Owned by: jryans
Priority: Medium Milestone: Tor: 0.3.0.x-final
Component: Core Tor/Tor Version:
Severity: Normal Keywords: man, doc, review-group-14
Cc: Actual Points:
Parent ID: Points:
Reviewer: Sponsor:


Some portions of the tor.1 doc don't join paragraphs correctly, leading to strange output styles, especially in html view, which appears to default to <pre> blocks when asciidoc is confused by the syntax.

Child Tickets

Change History (7)

comment:1 Changed 4 years ago by jryans

Owner: set to jryans
Status: newassigned

comment:2 Changed 4 years ago by jryans

Status: assignedneeds_review

comment:3 Changed 4 years ago by dgoulet

Keywords: man doc added
Milestone: Tor: 0.3.???Tor: 0.3.0.x-final
Status: needs_reviewneeds_information

Thanks jryans! Question:

Why SocksPortFlagsMisc and ExitPolicyDefault and ORPortFlagsExclusive don't show in the generated man page or HTML? So what are they for?

I'm asking because those could be confused with torrc option so I would be careful not to introduce section title that looks like such :S.

comment:4 Changed 4 years ago by jryans

Status: needs_informationneeds_review

Sorry, I should have explained that up front!

These "fake" sections were the only way I could see to have a paragraph be indented in the style of a first-level list item when it comes after a second-level / nested list item. All the other approaches I attempted led to the paragraph glued onto the content of the last second-level list item or formatted as a code sample, both of which are not what's intended. (There seems to be no explicit way to "end" the second-level list, so you have to find some way to trigger that implicitly. asciidoc is strange.)

I noticed there is a pre-existing usage of this technique with OtherSocksPortFlags. So anyway, these sections could really be named anything, since they don't appear in the output. Any suggestions for less confusing names, or do they seem okay?

comment:5 Changed 4 years ago by jryans

Maybe <X>Footer is a good name for these "hidden" sections?

comment:6 Changed 4 years ago by nickm

Keywords: review-group-14 added

comment:7 Changed 4 years ago by nickm

Resolution: fixed
Status: needs_reviewclosed

This looks good to me. Merging; thanks!

Note: See TracTickets for help on using tickets.