[Bug 165329] Review Request: roundup-0.8.4-3

[bugzilla at redhat.com]

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

Summary: Review Request: roundup-0.8.4-3


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


gauret at free.fr changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |ASSIGNED
         AssignedTo|gdk at redhat.com              |gauret at free.fr
OtherBugsDependingO|163776                      |163778
              nThis|                            |




------- Additional Comments From gauret at free.fr  2005-09-04 07:45 EST -------
* The summary is too long, please change it to "Simple and flexible
issue-tracking system"
* The roundup service is enabled by default. We usually don't do that with
network-listening daemons, please change the "345" to "-" at the top of the init
script.
* man pages are placed in /usr/man instead of /usr/share/man
* The lang file are not tagged properly, please use the %{find_lang} macro
(those two points will prevent from using INSTALLED_FILES, sorry...)
* the package should contain the text of the license (COPYING.txt)
* %{_initrddir}/roundup should be tagged as %config(noreplace) because of the
port setting
* you could support condrestart in scriptlets:
  %post
    if [ "$1" -ge "1" ]; then
        /sbin/service <script> condrestart >/dev/null 2>&1
    fi

-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug, or are watching someone who is.