* Dmitry Smirnov <[email protected]>, 2012-11-17, 19:16:
Source package is available:
https://mentors.debian.net/package/python-webm
I don't intend to sponsor this, but here's my quick review:
Support for .pyinstall files was added in python-defaults 2.6.6-6. This
is newer than the version in squeeze, so please add version constraint
to build-dependency on python-all.
s/python/Python/ is the package synopsis.
The extended description makes sense only when displayed together with
the short one. You shouldn't assume that this will always be the case;
long description should be stand-alone. Also, lintian emits:
I: python-webm: extended-description-is-probably-too-short
I would advise you against using a more restrictive license than
upstream uses.
Short license names in DEP-5 copyright file cannot contain spaces.
Regarding with-libwebp2.patch: you must not use the .so symlink at
runtime, because it could point a library that is not ABI-compatible
with python-webm. (See #651865 for a practical example of what could
happen when one uses .so symlink.)
--
Jakub Wilk
--
To UNSUBSCRIBE, email to [email protected]
with a subject of "unsubscribe". Trouble? Contact [email protected]
Archive: http://lists.debian.org/[email protected]
