[Bug 3693] Review request: girl - GNOME Internet Radio Locator

Hi, Observation nº1 %setup -q and %setup -qn %{name}-%{version} do the same i.e. by default -n is %{name}-%{version} so you can use %setup -q

Observation nº2 About make install DESTDIR=%{buildroot} you could use %make_install I learn this today , | there are _two_ macros: | | 1. %makeinstall | 2. %make_install | The %makeinstall macro should be avoided, because it redefines many path | variables that can lead to ugly side-effects. | The %make_install macro does exactly what you want. Really do take a look at | "rpm -E %make_install" to see what the macro does. It's there to make | packaging easier.

Observation nº3 : E: invalid-appdata-file /usr/share/appdata/girl.appdata.xml why rpmlint is saying this ?

Observation nº4 : girl-debuginfo.x86_64: E: incorrect-fsf-address in 17 files please see [1] we should inform upstream that should correct the address in the license. For example [2] [1] https://fedoraproject.org/wiki/Common_Rpmlint_issues#incorrect-fsf-address [2] https://sourceforge.net/p/smb4k/tickets/40/

Even though, package APPROVED ! PS: on Package CVS request please just ask for: Branches: devel, F-20 is EOL, F-21 is devel and F-22 doesn't exist