https://bugzilla.redhat.com/show_bug.cgi?id=1431322

Robin Lee <[email protected]> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
              Flags|fedora-review?              |fedora-review+



--- Comment #5 from Robin Lee <[email protected]> ---
1. There are still three whitespaces at EOLs.
2. Description of data subpackage should end with a period.
3. A comment should be included to specify which file is under BSD.
https://fedoraproject.org/wiki/Packaging:LicensingGuidelines?rd=Packaging/LicensingGuidelines#Multiple_Licensing_Scenarios

4. License and documents can be included only in the data subpackage. since it
is required by the base one.
5. update-desktop-database should not be used in Fedora 25 or later. You can
make a condition to filter some statements out:
%if 0%{?fedora} < 25
xxxx
%endif

After all, these are not blocking conditions. Just fix them before importing
the package.

This package is approved by cheeselee.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
_______________________________________________
package-review mailing list -- [email protected]
To unsubscribe send an email to [email protected]

Reply via email to