Subject: Re: [PATCH] Add manpage syntax checker to 'check' target

Re: [PATCH] Add manpage syntax checker to 'check' target

From: Daniel Stenberg <daniel_at_haxx.se>
Date: Tue, 6 Sep 2011 22:39:25 +0200 (CEST)

On Tue, 6 Sep 2011, Mikhail Gusarov wrote:

> In virtually every libssh2 release Debian's lintian catches syntax errors in
> manpages. Prevent it by checking manpages as a part of testsuite.

Sounds like a useful feature!

One litte nit though:

> + warnings=$(LANG=en_US.UTF-8 MANWIDTH=80 man -M "$srcdir" --warnings \
> + -E UTF-8 -l "$manpage" 2>&1 >/dev/null)

... this is not POSIX compliant so this command will cause grand error outputs
on lots of platforms. I think you need to do some basic checks first to see if
-M and --warnings etc work.

Or am I wrong?

-- 
  / daniel.haxx.se
_______________________________________________
libssh2-devel http://cool.haxx.se/cgi-bin/mailman/listinfo/libssh2-devel
Received on 2011-09-06