Alex Muntada wrote...

> I made some improvements in the documentation based on your
> comments. Please find the changes in http://deb.li/30t7A

Nice. You plan to leave the description in debian/control unchanged?
Then I suspect people will still follow just the link there and might
get stuck the way I did. It's so clickable in
    https://packages.debian.org/sid/libdbd-oracle-perl

Also README.Debian is not accessible before the package is installed,
but it's not installable before the procedures described there to
install the instant client have been followed %-)

So I'd rather re-phrase the last sentence, use an URL to your README,
and also mention the arch restriction.

| To install the Instant Client, available for the i386 and amd64 archs
| only, download the RPM package from the Oracle web page and install it
| using Alien. Detailled instructions for this are available at
| 
<https://anonscm.debian.org/cgit/pkg-perl/packages/libdbd-oracle-perl.git/tree/debian/README>

This might be a bit unusual, but quite a lot about this package is
unusual anyway ...

Just my suggestion, use it, drop it, whatever you think is right.

> alien and fakeroot are both build dependencies for the instant
> client packages, but not for libdbd-oracle-perl itself.

ACK

    Christoph

Attachment: signature.asc
Description: Digital signature

Reply via email to