Toonloop package

Alexandre Quessy alexandre at quessy.net
Sat Oct 16 15:49:58 UTC 2010


-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

Hello Jonas,

On 10-10-16 08:42 AM, Jonas Smedegaard wrote:
> On Sat, Oct 16, 2010 at 11:56:36AM +0200, Jonas Smedegaard wrote:
>> Your packaging overrrides a lintian warning.  Please either add a
>> comment in there explaining why you believe that warning to be wrong,
>> or remove the override: It is bad to silence lintian without a good
>> reason!
> 
> I now commented out that lintian override for now.  Feel free to
> reenable it, but please then add an explanation as noted there.

OK. I figured it out now. I need to escape the minus signs in the
examples in the man page text file (source for help2man) upstream.

I just fixed it upstream:
http://github.com/aalex/toonloop/commit/59b304f50826acf641ece01627b8f77b42fb9eae

> I am preparing an upload now.
> 

Thanks a lot!! I'm quite happy it made it. :)

> I would like the following to be improved soon:
> 
> The packaging depends on a bunch of gstreamer packages and more.  Are
> all of that *always* *crucial* for toonloop?  Dependencies that toonloop
> can survive without for some (maybe exotic) use, should be relaxed to
> recommends.

Yes. The audio-related GStreamer can be removed, that's right. (done)

Also, I will verify that both -bad and -ugly are necessary. That might
take a little time, though, and I might need them quite soon, as I'll be
working on the support for more cameras.

> Similar for build-dependencies: I noticed e.g. mencoder is a
> build-dependency, and that autotools check for it, but if not actually
> used at build time or in any tests, then I suggest improving upstream
> routines to distinguish between buildtime and runtime dependencies -
> e.g. only check for actual build dependencies in autotools and have the
> toonloop application check for runtime dependencies when invoked.

That's right. It's a runtime dependency. I will fix that upstream. I
guess the simplest way to do so would be to simply call g_spawn_async()
and see if it fails.

> 
> Thanks for your great contribution!
> 

:)
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.10 (GNU/Linux)
Comment: Using GnuPG with Mozilla - http://enigmail.mozdev.org/

iQIcBAEBAgAGBQJMucmiAAoJEJQ0pOgl2qx1AtsP/2qyd4VtQB0u3LQ+ObM5d/1E
wNU6vcqMAOGOpTxKZfwVO7jK9qPLxRIrkRK3WopdQiR7oVYX6s1ovOu9hpkYB7II
2uU+wvjxvrluak8APOcorBb6qWKz/Pciv5MyLXhXJOLyzaYq5aMPxO7pTnXUwwYd
fJItsridgYGD9XZs8w56tGOqRPBv8M8ms46MEIGb7S59qmA6FjB/KWywcd5jKtCE
goDX5TfvOLZ2PPDWpYQg43dVy5YyET4FfWINrZ34y8m8Ga/VY8d/L/nzNexOzSwN
CWPwCksHcvvomBNeROtk4ZtHm5U7zyYN0Jkj7zKJl9OOz4Z4+8vhtwWBiMiMfEzj
nmWHb0xU7wOv9/vRyMvnVywnZj4/ouGks3jxCO7NiZDFk3UdwEhozsxQPRZhTmhQ
N8VvIKShoV9yhjwv2h0tk1x4C0Idquzpef3+ok8HkKC2YVSfY5e3W8st1JEBUUth
s1kENhRU19qB+ISYNJBw/DBsghoOJxYMv1CYZ70EzLtbLauqfL5+TRNWSCg7XjL7
rLXHvX9uuuuu5T/MagK1S8AfVGKg9sp+NeEgsMKwQHOOnUhQtpYbsojiZBBbqGYH
gdMBDoKsfyy1m3mEus0VlCj6/ERKmqgmsQwkmAgo4x7VrFQRqXa2jRJLEr/XRLmT
xR2ozqscCKCxUY80yTro
=7b+p
-----END PGP SIGNATURE-----



More information about the pkg-multimedia-maintainers mailing list