[Bug 180015] Review Request: OTRS - Open Ticket Request System

[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: OTRS - Open Ticket Request System


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


mitr at redhat.com changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |ASSIGNED




------- Additional Comments From mitr at redhat.com  2006-03-06 20:59 EST -------
Formal blockers:
* The %files section does not specify %defattr
* Kernel/cpan-lib seems to be duplicates of separate perl packages, at least
  some currently in Fedora.

Other issues:
* scriptlets should properly guard against upgrades, see "Running scriptlets
  only in certain situations" in PackagingGuidelines
* I don't think just installing a package should change the MaxRequestsPerChild
  option in httpd; does otrs really leak that badly?
* Starting httpd: [Tue Mar 07 02:44:25 2006] [warn] Useless use of AllowOverride
in line 20.
  Can this be avoided?
* The package seems to install most files world-readable, owned by root;
  INSTALL.Redhat seems to suggest the permissions should be already configured
  by the RPM, probably similar to the SetPermissions.sh script.
  This breaks at least the spell checker:
[Tue Mar 07 02:54:14 2006] [error] [Tue Mar  7 02:54:14 2006] -e: Error in
tempfile() using /var/www/otrs/var/tmp/XXXXXXXXXX.tmp: Parent directory
(/var/www/otrs/var/tmp/) is not writable\n[Tue Mar  7 02:54:14 2006] -e:  at
/var/www/otrs//Kernel/System/FileTemp.pm line 91\n
* The Requires(postun) is not necessary AFAICS
* Requires: perl perl-DBI perl-DBD-MySQL  is autodetected
* I can't see any use of perl-URI from a quick grep
* Does the package really require the sendmail program, or just an
  /usr/sbin/sendmail implementation?

* It would be nice if the initscripts used the same way to report
  success/failure as the other Fedora initscripts (success()/failure()).

-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the QA contact for the bug, or are watching the QA contact.