On Wed, Apr 13, 2011 at 02:01:58AM +0200, Ingo Schwarze wrote: > Hi, > > Gleydson Soares wrote on Tue, Apr 12, 2011 at 03:39:26PM -0300: > > On Tue, Apr 12, 2011 at 10:42:40PM +0800, wen heping wrote: > > >> Here is a new port of devel/p5-ExtUtils-CChecker, it is required by > >> net/p5-Socket-GetAddrInfo which is required by the update of devel/p5-POE. > >> Tested OK on Loongson and regress OK too. > >> Comments ? > > > looks fine to me, but please, > > set USE_GROFF(mandoc is unhappy with those manpages) > > > > $ mandoc -T lint -W error * > > ExtUtils::CChecker.3p:210:8: ERROR: macro requires body argument(s) > > ExtUtils::CChecker.3p:309:8: ERROR: macro requires body argument(s) > > $ > > Oh, you have found an error reporting bug in mandoc(1). > Thanks for paying attention and bringing this up.
by chance. > The complaint from mandoc is bogus, and if all parts > of the manual look good, USE_GROFF is pointless. in this point, i agreed with you. as you said, this was a bogus. > > In particular, the vicinity of this particular complaint does > look good with mandoc, even though mandoc complains. agreed. > > > Thus, i have added an entry to the mandoc TODO: > > - In man(7), an empty subsection (.SS) should not be an error, probably > not even a warning. For example, in ExtUtils::CChecker(3p), this is > used in a way similar to empty .It -tag in mdoc(7), that is, explaining > two commands (both presented with .SS markup) in one paragraph of text. > Reported by Gleydson Soares <gsoares at trusted dot com dot br> > on ports@ Tue, 12 Apr 2011 15:39:26 -0300 nice. thanks. > > Thanks, > Ingo