[Pkg-openldap-devel] Bug#794998: Bug#794998: openldap: no manual page for smbk5pwd module

[Peter Marschall]

Hi,

On Friday, 14. August 2015 17:46:41 Ryan Tandy wrote:
> On Sun, Aug 09, 2015 at 09:56:03AM +0200, Peter Marschall wrote:
> >++.\" Copyright 2015 The OpenLDAP Foundation All Rights Reserved.
> 
> I assume this is intentional, and is because the content is adapted from
> the existing README?
Yes to both.
My hope is to get the man page upstreamed (see upstream ITS#8205)

> >++.SH SEE ALSO
> >++.BR slapd.conf (5),
> >++.BR ldappasswd (1),
> >++.BR ldap (3),
> 
> Looks like a trailing comma?
You're right: copy&paste artifact from the slapo-lastbind manpage ;-)

English is not my mother tongue, so please feel free to change what you 
consider in need of changing.

> >diff --git a/debian/slapd.manpages b/debian/slapd.manpages
> [...]
> 
> Thinking out loud... I'd be happier if newly added man pages could be
> installed automatically, or at least if we could be notified about
> needing to update the file.
Completely understandable. I am also not happy about that static list of 
manual pages.
I was looking for a syntax in the .manpages file that allows excluding a 
specific file, but in vain.

But I found a solution:  executable debhelper files.
See the attached patch that converts slapd.manpages into a script
which 
- first expands the manpage patterns it feels generally responsible for
- then excludes all the manual pages listed in other slapd-*.manpages files
- finally outputs all the remaining ones
I checked it with my private openldap packaging: it works!

> I tried to use dh_install --fail-missing, but it looks like it doesn't
> (yet) cooperate with dh_installman and others, so files are spuriously
> reported missing (#436240#c15). The solution used for the module itself,
> letting it be installed into slapd and then rm'ing it, is a bit of a
> hack, but does achieve that. On the other hand, the only new slapo-* man
> pages we're likely to get in 2.4 are contrib ones, so maybe this is
> fine.
The attacked patch avoids the need for a hack.

Best
Peter

-- 
Peter Marschall
peter at adpm.de
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-debian-slapd.manpages-use-script-to-automatically-in.patch
Type: text/x-patch
Size: 3195 bytes
Desc: not available
URL: <http://lists.alioth.debian.org/pipermail/pkg-openldap-devel/attachments/20150815/b03550a0/attachment.bin>