[Bug 213180] Review Request: tcl-thread - Thread extension for Tcl

[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: tcl-thread - Thread extension for Tcl


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





------- Additional Comments From wart@xxxxxxxxxx  2006-11-02 18:04 EST -------
GOOD
====
* Package and spec named appropriately:  The upstream name is the simply
  'thread', which is far too generic.  Following the examples for python
  and perl modules, the name tcl-thread is acceptable.
* Spec file is legible and in Am. English
* Source matches upstream:
  3c69b4a891590f23bb79a1fa98d879f7  thread2.6.5.tar.gz
* No unnecessary BuildRequires
* No locales
* No shared libraries in the default linker path; the shared library
  that is produced is loaded by Tcl via dlopen.
* RPM_BUILD_ROOT cleaned where appropriate
* Not relocatable
* No duplicate %files
* File permissions look ok
* No need for a -devel subpackage
* Not a gui program; no need for a .desktop file
* Package loads into Tcl as expected and passes its own test suite.
* Consistent use of macros
* Does not own any directories that it should not own.

MUSTFIX
=======
 * License does not match upstream.  Should be BSD.
 * License file 'license.terms' not included.
 * Add the README and ChangeLog files to %doc

 * Does not own all directories that it creates.  In %files, change
%{_libdir}/thread%{version}/*
to
%{_libdir}/thread%{version}

 * Does not build properly on x86_64 in mock.  The attached patch
   fixes the problem.

 * The dependency on gdbm is picked up automatically.  You can drop
   Requires: gdbm.

SHOULDFIX
=========
* Missing a %check section for running the unit tests.

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