From mboxrd@z Thu Jan  1 00:00:00 1970
Return-Path: <e@80x24.org>
X-Spam-Checker-Version: SpamAssassin 3.4.2 (2018-09-13) on dcvr.yhbt.net
X-Spam-Level: 
X-Spam-Status: No, score=-4.0 required=3.0 tests=ALL_TRUSTED,AWL,BAYES_00
	shortcircuit=no autolearn=ham autolearn_force=no version=3.4.2
Received: from localhost (dcvr.yhbt.net [127.0.0.1])
	by dcvr.yhbt.net (Postfix) with ESMTP id E9D761F4B4;
	Fri, 18 Dec 2020 12:17:21 +0000 (UTC)
Date: Fri, 18 Dec 2020 12:17:21 +0000
From: Eric Wong <e@80x24.org>
To: Uwe =?utf-8?Q?Kleine-K=C3=B6nig?= <uwe@kleine-koenig.org>
Cc: meta@public-inbox.org
Subject: Re: [PATCH] Fix manpage section of perl module documentation
Message-ID: <20201218121721.GA23386@dcvr>
References: <20201218115614.613227-1-uwe@kleine-koenig.org>
MIME-Version: 1.0
Content-Type: text/plain; charset=utf-8
Content-Disposition: inline
Content-Transfer-Encoding: 8bit
In-Reply-To: <20201218115614.613227-1-uwe@kleine-koenig.org>
List-Id: <meta.public-inbox.org>

Uwe Kleine-König <uwe@kleine-koenig.org> wrote:
> On Debian (at least) perl documentation is supposed to be installed in
> section 3pm. With the build system hardcoding this to 3 instead this
> results in a warning by the Debian package linter:
> 
> 	W: public-inbox: wrong-manual-section usr/share/man/man3/PublicInbox::Git.3.gz:74 3 != 3pm
> 	W: public-inbox: wrong-manual-section usr/share/man/man3/PublicInbox::Import.3.gz:74 3 != 3pm
> 	W: public-inbox: wrong-manual-section usr/share/man/man3/PublicInbox::SaPlugin::ListMirror.3.gz:74 3 != 3pm
> 	W: public-inbox: wrong-manual-section ... use --no-tag-display-limit to see all (or pipe to a file/program)
> 
> So use "$(MAN3EXT)" instead of "3".

Thank you, I was wondering about 3pm a few years ago but
forgot about it.  I just checked my FreeBSD system and
that uses 3, so it's definitely system-dependent.
CentOS 7 also uses 3pm.

> Hello,
> 
> I'm continuing to package public-inbox for Debian now. I'm not a perl
> pro, so take this patch with a grain of salt. The only indication I have
> that this patch is fine is that it makes the lintian warning go away :-)

Thanks again, and Debian is still my favorite distro.  I'm not a
build system/packaging pro but the patch looks good to me.