search

halfgaar / FlashMQ

FlashMQ is a fast light-weight MQTT broker/server, designed to take good advantage of multi-CPU environments
https://www.flashmq.org/
Open Software License 3.0
173 stars 24 forks source link

Man-page improvements, including the ditchting of `docbook2x` #106

Closed bigsmoke closed 1 week ago

bigsmoke commented 2 weeks ago

The only dependency that is now left for building the man-pages is an XSLT processor. (The Makefile assumes xsltproc.)

halfgaar commented 2 weeks ago

I don't quite understand Add missing container el and Document command-line args & cross ref. config stuff. It's the same duplication of sections and stuff we saw before?

bigsmoke commented 1 week ago

My original mistake appears to be that I started working on this branch before the changes from my previous PR were merged into origin/master. (What is now 09cdb096edb05ba338f5ff6696ac113eeaf51166 has been squeezed into 9538541b38ed337017e4f36e1e70456ef984fd97 for origin/master previously.) The same goes for 2febd234f3d97cc9d3d7766c69464974b2e6c675.

I removed these two commits from my PR branch and resolved the new conflicts.

Also rebased on your newest master, so that you don't have to resolve the merge conflict on the documentation of your newest config file options.

halfgaar commented 1 week ago

This is so cool:

image

I would like to frame it...

Too bad Debian turns the colors off by default, so few people will see it.

There is a slight generation problem in the html version though:

image

Both literals name flashmq.conf as stray string.

bigsmoke commented 1 week ago

Good catch! I've changed that template to only <xsl:apply-templates select="text()"/>, instead of node(). Previously, the attribute would be caught by node() and processed by the default template, and this only became noticeable with the extra attribute.

By the way: I have not (yet) implemented syntax highlighting for HTML. I'm inclined to use the same (XSLT-based) method, since all the XSLT is bound to this project anyway and I thus see no need for the use of a more general-purpose syntax-highlighting method.

halfgaar commented 1 week ago

It's merged :+1: