Hello.
This patch removes the case sensitivity from a bunch of the regular expressions
used to handle docbook tags within the mandocs scripts. It also tweaks the
scripts to use the value of the $TMPDIR environment variable for the location of
temporary files (if set). The default is to still use /tmp.
The patch results in significantly fewer warnings being generated by a "make
mandocs". It's part of an ongoing warning squelching effort.
The patch can also be downloaded from
http://www.stillhq.com/linux/patches/2.6.0-test4-bk5/mandocs_ignorecase, sample
output at http://www.stillhq.com/linux/mandocs/2.6.0-test4-bk5/ although there
should be no visible changes...
Please apply.
Thanks,
Mikal
PS: Oh, and Gordon says hello.
--
Michael Still ([email protected]) | "All my life I've had one dream,
http://www.stillhq.com | to achieve my many goals"
UTC + 10 | -- Homer Simpson
-------------------------------------------------
This mail sent through IMP: http://horde.org/imp/
Looks fine. Technically xml docbook tags must be lower case but older
SGML docbook is undefined. The /tmp handling still does want fixing
though - it does several things that don't look remotely /tmp safe.
Quoting Alan Cox <[email protected]>:
Alan, thanks for your prompt comments.
> Looks fine.
Ta.
> Technically xml docbook tags must be lower case but older
> SGML docbook is undefined.
For sure -- there are examples in the current kernel documentation which used
mixed case, which I personally feel is not as nice as lower case. I wanted to
tend on the side of working with whatever documentation we have though, and deal
with making the entire world pretty at some later date.
Note also that the patch only makes the internal handling of the tags case
insensitive -- it doesn't change the case of the tags in the kernel-doc, and it
respects the case the author chose when it emits the tags to the intermediate
SGML files used by mandocs.
> The /tmp handling still does want fixing
> though - it does several things that don't look remotely /tmp safe.
This patch was more a case of putting the temporary files where the user
expected them to be. To me, the risk associated with a poorly intentioned person
substituting incorrect manual pages for the internal kernel APIs is sufficiently
low that I am uncertain if it is worthwhile going with a patch to make this
safer. I'm quite content to be wrong on this point however -- do people feel
that making the mandocs scripts temporary file safe is an important goal?
Cheers,
Mikal
--
Michael Still ([email protected]) | "All my life I've had one dream,
http://www.stillhq.com | to achieve my many goals"
UTC + 10 | -- Homer Simpson
-------------------------------------------------
This mail sent through IMP: http://horde.org/imp/