tags 451187 fixed-upstream thanks On Sat, Nov 24, 2007 at 03:28:45AM +0600, Ivan Shmakov wrote: > It was suggested that man-db gets one more command as well, to > do the checking in a ``standard way''. So, I suggest the > following: > > * add support for an MANROFFOPT environment variable to specify > extra arguments to groff (analogous to MANROFFSEQ and MANOPT);
I like this, thanks; I've applied your patch with a few small modifications, and also added documentation of it to man(1). If you want to check this out, install bzr and run: bzr get http://www.chiark.greenend.org.uk/~cjwatson/bzr/man-db/trunk/ > * add a command, `manwe' (for ``MAN with Warnings Enabled''), > which will behave just like `man', but with (some) Groff > warnings enabled; While I appreciate the Silmarillion pun :-), I'm not very keen on another command just for this. I would rather have something like 'man --warnings'. Once I finish converting man to argp, it will be easier to add options without trashing translations every time. > Please note that `manwe' would only work as intended if `man' is > to call `groff'. So, `-t' option may be necessary, e. g.: > > $ manwe -t ifconfig > /dev/null > <standard input>:63: warning: `Note:' not defined > $ I just sent a patch to the groff mailing list to teach its nroff shell script about -w and -W options. If this is accepted upstream, I'll also apply it to the Debian groff package. > --- man-db-2.5.0-debian-3/src/man.c 2007-11-24 01:54:12.000000000 +0600 > +++ man-db-2.5.0-debian-3-manwe/src/man.c 2007-11-24 02:15:52.000000000 > +0600 > @@ -1522,6 +1529,7 @@ > cmd = command_new_args (fmt_prog, file, pp_string, NULL); > if (roff_device) > command_arg (cmd, roff_device); > + command_argstr (cmd, roff_opt); > pipeline_command (p, cmd); > } > Note that this part is incorrect because, for better or worse, the interface to custom format programs doesn't support adding *roff options (see e.g. tools/mandb_fmt-script). Fortunately, as far as I know hardly anyone uses this feature. Cheers, -- Colin Watson [EMAIL PROTECTED] -- To UNSUBSCRIBE, email to [EMAIL PROTECTED] with a subject of "unsubscribe". Trouble? Contact [EMAIL PROTECTED]
