Axel Beckert 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.
> 
> Cc'ing [email protected] for input on this phrasing as per
> https://www.debian.org/doc/manuals/developers-reference/best-pkging-practices.en.html#write-correct-english

It's grammatical and I believe it says what you need it to say, but
I'd be happier with it if it had fewer uses of "only" per sentence...

    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"?)
-- 
JBR     with qualifications in linguistics, experience as a Debian
        sysadmin, and probably no clue about this particular package

Reply via email to