https://bugzilla.rpmfusion.org/show_bug.cgi?id=4640

Nicolas Chauvet <kwiz...@gmail.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |ASSIGNED
             Blocks|2, 30                       |3
           Assignee|rpmfusion-package-review@rp |kwiz...@gmail.com
                   |mfusion.org                 |
              Flags|                            |fedora-review?

--- Comment #2 from Nicolas Chauvet <kwiz...@gmail.com> ---
Please drop every requires, unless you rely on the binary been available those
are expected to be added automatically.

rm -rf $RPM_BUILD_ROOT at %install step is not needed anymore on modern RPM
distro

Please use desktop-file-validate and use BuildRequires: desktop-file-utils:
https://fedoraproject.org/wiki/Packaging:Guidelines#desktop-file-install_usage

I can sponsor you for RPM Fusion (you will need to seek another sponsor for
fedora).

there is a weird 0.01× speed. character in your description. Are you use to
use a utf8 file ?

The version field is incorrect, please have a look on:
https://fedoraproject.org/wiki/Packaging:Versioning


Referenced Bugs:

https://bugzilla.rpmfusion.org/show_bug.cgi?id=2
[Bug 2] Tracker: New packages awaiting review
https://bugzilla.rpmfusion.org/show_bug.cgi?id=3
[Bug 3] Tracker: Packages under review.
https://bugzilla.rpmfusion.org/show_bug.cgi?id=30
[Bug 30] Tracker : Sponsorship Request
-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are the assignee for the bug.
_______________________________________________
rpmfusion-developers mailing list -- rpmfusion-developers@lists.rpmfusion.org
To unsubscribe send an email to rpmfusion-developers-le...@lists.rpmfusion.org

Reply via email to