Bug 213907 (tclpro) - Review Request: tclpro - Tcl debugging environment
Summary: Review Request: tclpro - Tcl debugging environment
Keywords:
Status: CLOSED NEXTRELEASE
Alias: tclpro
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Paul F. Johnson
QA Contact: Fedora Package Reviews List
URL:
Whiteboard:
Depends On: tclcompiler tclchecker tcldebugger
Blocks: FE-ACCEPT
TreeView+ depends on / blocked
 
Reported: 2006-11-03 18:15 UTC by Wart
Modified: 2009-01-28 05:59 UTC (History)
0 users

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2006-11-26 01:47:07 UTC
Type: ---
Embargoed:
kevin: fedora-cvs+


Attachments (Terms of Use)

Description Wart 2006-11-03 18:15:49 UTC
Spec URL: http://www.kobold.org/~wart/fedora/tclpro.spec
SRPM URL: http://www.kobold.org/~wart/fedora/tclpro-1.5.0-6.20061030cvs.src.rpm
Description:
The TclPro debugging suite includes a graphical debugger, a static code checker,
a compiler that writes out Tcl byte codes, and a loader that reads
byte codes.

Note:  Upstream has not released official tarballs of this package.  When I asked about it, they said they did not have time for generating official releases and that CVS snapshots should be used instead.

Comment 1 Paul F. Johnson 2006-11-03 18:56:04 UTC
Release: 6 - eh? surely the release is the release relating to FE rather than
anything else!

Comment 2 Wart 2006-11-03 19:09:53 UTC
I have been using it in my own private repo for about a year now, and thought it
would be easier to 'upgrade' to the FE package once it was approved if I bumped
the release instead of resetting it to 1.  Release digits are cheap, so I hoped
it wouldn't be a problem.  :)

Comment 3 Paul F. Johnson 2006-11-08 01:12:12 UTC
rpmlint clean
mock clean

Review

Good

spec file in US English and clear
constant use of macros
includes documentation
noarch fine
no devel
includes clean
includes documentation
no problems with desktop file install
no problems with permissions
no ownership problems
correct use of cvs naming
no md5sum as it's a cvs checkout
software installs and runs without a problem

APPROVED

Comment 4 Wart 2006-11-08 01:17:43 UTC
Thanks for the review.  Since this depends on tcldebugger and tclchecker, I'll
wait until those are approved before importing this.

Comment 5 Wart 2006-11-26 01:47:07 UTC
Imported and built.  Thanks!

Comment 6 Wart 2009-01-28 00:59:35 UTC
Package Change Request
======================
Package Name: tclpro
New Branches: EL-4 EL-5
Owners: wart

Comment 7 Kevin Fenzi 2009-01-28 05:59:24 UTC
cvs done.


Note You need to log in before you can comment on or make changes to this bug.