Whoops - forwarding to the RFS bug (to record the comments there)
-------- Forwarded Message -------- Subject: Re: Bug#800406: RFS sayonara/0.8.2 Resent-Date: Sun, 21 Feb 2016 16:22:24 +0000 (UTC) Resent-From: debian-ment...@lists.debian.org Date: Sun, 21 Feb 2016 17:21:46 +0100 From: Ross Gammon <r...@the-gammons.net> To: debian-ment...@lists.debian.org Hi Lucio, Sorry I did not spot this email after talking you into contacting Debian Mentors :-) I am not a Debian Developer, so I cannot sponsor the package, but I will write some comments inline below that will hopefully help you on your way. On 02/15/2016 11:30 AM, Lucio Carreras wrote: > Dear Debian mentors, > > I am looking for a sponsor for the package "sayonara". > > * Package name: sayonara > * Version: 0.8.2 > * Author: Lucio Carreras <luciocarre...@gmail.com > <mailto:luciocarre...@gmail.com>> > * Url: http://www.sayonara-player.com > * License: GPL3 > * Section: audio > > Sayonara is a clear and fast audio player with its main focus on > managing your music library. There are many features for library > organization, like a tag editor or a special genre view. It also has > tabbed playlists, handles webstreams and podcasts, has a stream > recorder, a Soundcloud plugin and much more. > > > There's a git repository at > > https://git.sayonara-player.com/sayonara.git I have cloned your git repo to do the review, but in the future, I recommend that you sign up for an account on Debian Mentors, and upload your package there. This has some advantages: 1. Some sponsors are used to this workflow and prefer to take packages from there (at least the first time) 2. You go through a similar process to what you will go through when you are eventually granted upload permisions (it is good practise for later) 3. The web interface gives a sponsor a good indication of the health of the package (e.g. number of lintian warnings) without having to download it. See Section 4 of mentors.debian.net/intro-maintainers > There also exist a Ubuntu PPA > (https://launchpad.net/~lucioc/+archive/ubuntu/sayonara) and so there > also exist Debian packages (available at the download section). The > source code in the git repository also contains a debian directory with > all neccessary files. It is also tested regularly with a Debian unstable > chroot, if there are any warnings or errors. This is a good sign. Some more detailed comments: debian/changelog: - As this will be the first upload to Debian, it is best that the changelog has one simple entry "Initial Release (Closes: #<your ITP Bug Number>)". This will allow the bug to be automatically closed when the package is uploaded to the archive. See https://www.debian.org/doc/manuals/developers-reference/best-pkging-practices.html#bpp-debian-changelog - You have set the distribution to stable. New packages should always target "unstable". It will automatically migrate from there to "testing" (if there are no problems), and then one day be released as part of the next stable release. debian/control - I recommend joining the Debian Multimedia Team eventually. Then the Vcs fields can be finalised (surrently commented out). But lets improve the packaging a little bit more first. - A space between "Description:" and the short description would be nice - The long description should probably start with "Sayonara" rather than "Its" because it will not be clear if read without the short description. debian/copyright - The "License:" stanza should be the short name of the license (e.g. GPL-3+). - Then the complete license text should be included at the bottom. See https://www.debian.org/doc/packaging-manuals/copyright-format/1.0/ debian/rules - I was a bit surprised to so much stuff in debian/rules, as most of the time the common build systems don't need a lot more than: %: dh $@ It might be worth looking at tweaking your cmake build system (as you are also upstream) so that it is not necessary to override the clean, build & install targets in debian/rules. This would probably benefit other non Debian based Linux Distros as well. Probably not critical - depending on the sponsor though. You are missing a watch file. This file watches for new tarballs on your website, and alerts everyone when there is a new version waiting to be packaged. See https://wiki.debian.org/debian/watch/ debian/README.Debian? This should probably be deleted. You will find a sponsor much faster if you promise to maintain Saynora yourself (or in a team). :-) debian/md5sums - I am not sure this file is needed if building the package happens the normal way with debhelper? debian/license.txt - a license.txt in the root of the sourcecode, and the summarised in debian/copyright should be enough. Please remove this file. The package is currently building. I will let you know later if there are problems with that. I hope that helps you on the way. If there is something in all that which is not clear, or you need more help to fix it, just ask on the list. Keep up the good work, and thanks for taking an interest in getting Sayonara into Debian proper. Regards, Ross
signature.asc
Description: OpenPGP digital signature