[Bug 222888] Review Request: Tiquit - A PHP5-compatible help desk incident tracking/knowledgebase system

[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 report.

Summary: Review Request: Tiquit - A PHP5-compatible help desk incident tracking/knowledgebase system


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


tibbs@xxxxxxxxxxx changed:

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




------- Additional Comments From tibbs@xxxxxxxxxxx  2007-05-06 03:11 EST -------
Builds fine and rpmlint is silent.

Source0 should be a URL; I used the usual
  http://dl.sf.net/%{name}/%{name}-%{version}.tar.gz
so that I could grab the upstream source for comparison.

The link to config.php looks a bit odd:
  /usr/share/tiquit/config.php -> ../../..//etc/tiquit/config.php
%{_sysconfdir} includes a leading slash.  Normally I'd ask if there's a nicer
way to do this than via a symlink, but it's pretty obvious from the code that
you'd have to patch every source file.

Since this comes up locked down to localhost, it's probably worth adding a
README.Fedora file containing information about why you've done that, what the
user needs to in order to enable things, and if there's any mandatory
configuration they need to do before turning it on.

I'm not sure your dependency on php-gd does what you want it to.  Note that it
won't pull in a PHP interpreter, just php-common.

I'm not sure why you have
  %dir %{tiquitdir}
  %{tiquitdir}/*
instead of just
  %{tiquitdir}

* source files match upstream:
   1c7aefbde49ee7ee4f946d60218a9f19f5a5236df67f3e401ffa76f2ecfc563c  
   tiquit-2.4.tar.gz
* package meets naming and versioning guidelines.
* specfile is properly named, is cleanly written and uses macros consistently.
* dist tag is present.
* build root is OK.
* license field matches the actual license.
* license is open source-compatible.
* license text included in package.
* latest version is being packaged.
* BuildRequires are proper (none needed)
* %clean is present.
* package builds in mock (development, x86_64).
* package installs properly
* debuginfo package looks complete.
* rpmlint is silent.
? final provides and requires are sane:
   config(tiquit) = 2.4-1.fc7
   tiquit = 2.4-1.fc7
  =
   config(tiquit) = 2.4-1.fc7
   php-gd

* %check is not present; no test suite upstream.
* owns the directories it creates.
* doesn't own any directories it shouldn't.
* no duplicates in %files.
* file permissions are appropriate.
* no scriptlets present.
* code, not content.
* documentation is small, so no -docs subpackage is necessary.
* %docs are not necessary for the proper functioning of the package.

-- 
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.

_______________________________________________
Fedora-package-review mailing list
Fedora-package-review@xxxxxxxxxx
http://www.redhat.com/mailman/listinfo/fedora-package-review

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