Re: Bug#980903: debhelper: doc-base doc-id deduplication does not work as documented with multiple "dh_installdocs -p<pkg>" calls; causes /usr/share/doc-base/<doc-id> to be installed multiple times

Hi Justin,

thanks for your prompt reply,

Justin B Rye wrote:
> > I've now added one paragraph to the POD of dh_installdocs:
> > 
> >   Please be aware that this deduplication currently only works if
> >   B<dh_installdocs> is called only once during the package build as
> >   deduplication is done in memory only. Especially calling
> >   B<dh_installdocs -p>I<package> in combination with using
> >   F<debian/>I<package>F<.doc-base.*> files can lead to uninstallable
> >   packages. See L<https://bugs.debian.org/980903> for details.
[…]
> It's grammatical and I believe it says what you need it to say,

Thanks!

> but I'd be happier with it if it had fewer uses of "only" per
> sentence...

Hehe, good point.

>     Please be aware that this deduplication is currently done in memory
>     only, so it requires B<dh_installdocs> to be called no more than
>     once during the package build. Calling B<dh_installdocs -p>I<package>
>     in combination with using F<debian/>I<package>F<.doc-base.*> files
>     can lead to uninstallable packages. See
>     L<https://bugs.debian.org/980903> for details.
> 
> (Is it still accurate with the "currently" relocated like this, or
> does it need to be before "requires"?)

My gut feeling says it's needed in both places. Would that be too
much? Would "so for now it requires" be an acceptable alternative to
using "currently" twice?

Context: I've prepared a patch which will hopefully fix this in the
future, but it's likely too invasive for bullseye. So this text will
be in there only until #980903 will be fixed properly.

		Regards, Axel