Packaging review for the Elisa music player

Sandro Knauß hefee at debian.org
Tue Jan 15 18:10:54 GMT 2019


Hey	 Aurélien,

d/copyright:
* You have a block "License: LGPL-2 or GPL-3"
you should remove this block and replace it with:
License: LGPL-2
...
License: GPL-3

> - elisa seems heavily based on QML, although there are no runtime 
dependencies on QML modules
> - autopkgtest is meant to test packages as available in the archive, not to 
be a generic CI for running the test suite on a new package build; the current 
'testsuite' autopkgtest does not test the in-archivepackage, and as such it is 
not a valid autopkgtest test (and it can even produce wrong results)

Because elisa uses QML and QML has no  equivalent of ABI tracking via symbol 
files, I would recommend to use autopkgtests to check, that the used QML 
modules are still fine. Start with simple "smoke" tests. aka:
Can you execute "elisa --help"?
Can you startup elisa by commandline without any breakage, this should test 
the QML stuff.
These two small tests should make sure, that most of the dependencies you need 
are in place. 

hefee
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 833 bytes
Desc: This is a digitally signed message part.
URL: <http://alioth-lists.debian.net/pipermail/pkg-kde-talk/attachments/20190115/360f63fe/attachment.sig>


More information about the pkg-kde-talk mailing list