Hi,

Here is a quick review of the package:

debian/rules:

> .PHONY: override_dh_strip

Seems a bit inconsistent to only add dh_strip.

>     dh $@

Please enable parallel building.

> override_dh_auto_test:
>     # Do not perform test suite as it requires X11 running

You can run the tests by build-depending on xvfb and calling
xvfb-run dh_auto_test


debian/patches/debian-multiarch.patch:

This one is not really needed since you can't co-install gammaray for
multiple architectures anyway.


debian/patches/gammaray-probe-versioned-soname.patch:

Why do you set a SONAME for gammaray_probe?
When gammaray changes the ABI custom plugins will break anyway
since you can't install different versions of gammaray.


debian/patches/harfbuzz-search-path.patch:

What does this patch do?
libqt4-private-dev doesn't contain the harfbuzz headers and that won't
change at least until wheezy is released.

Felix


-- 
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of "unsubscribe". Trouble? Contact listmas...@lists.debian.org

Reply via email to