[Bug 647586] Review Request: nzbget - Command-line based binary newsgrabber for nzb files

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

 



Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #4 from Peter Lemenkov <lemenkov@xxxxxxxxx> 2010-10-30 04:44:10 EDT ---
Notes:

a) You're installing config-file as a doc-file, but would it be better to
install in into /etc/ ? User will get ready-to-go application right after
installing then. However looking at the config-file I found that it has too
many options which should be adjusted for that (directory paths, server's
addresses an so on). Also I have no clue about typical workflow involving
nzbget, so feel free to disregard this advice entirely if it would require too
much manpower and offer only negligible income.

b) You should add AUTHORS files (it a generally good idea to mention people who
work hard on providing all that stuff for us) and you *MUST* add COPYING to
%doc 

c) You decided not to install init-script (nzbgetd) - is it intentional? Maybe
it's better to package it too. However (as I sated earlier)  I have no clue how
to use thos application. so I don't have strong opinion here.

Please, comment these notes, and I'll continue.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
------- 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]