[Bug 822959] Review Request: nitrotasks - Super awesome task management

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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

Brenton Leanhardt <bleanhar@xxxxxxxxxx> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |bleanhar@xxxxxxxxxx

--- Comment #3 from Brenton Leanhardt <bleanhar@xxxxxxxxxx> ---
Other things that need to be fixed:

When I tried to build with mock the 'intltool' BuildRequire was missing.

It's also preferable to require 'python-devel'.  That will BuildRequire python
as well as give you access to convenient macros like %{__python}
https://fedoraproject.org/wiki/Packaging:Python#BuildRequires

You need to install nitrotasks.desktop.in as mentioned here:
http://fedoraproject.org/wiki/Packaging/Guidelines#Desktop_files

You need to verify that permission has been granted to distribute logos for
DropBox, Ubuntu.  There is also an image of Justin Bieber that is probably not
free to distribute.

http://fedoraproject.org/wiki/Packaging/Guidelines#Tags
The Vendor tag should not be used. It is set automatically by the build system

It's adviseable to work with upstream to clearly state the copyright holder in
the COPYING file.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
_______________________________________________
package-review mailing list
package-review@xxxxxxxxxxxxxxxxxxxxxxx
https://admin.fedoraproject.org/mailman/listinfo/package-review



[Index of Archives]     [Fedora Legacy]     [Fedora Desktop]     [Fedora SELinux]     [Yosemite News]     [KDE Users]     [Fedora Tools]