Note: This is a public test instance of Red Hat Bugzilla. The data contained within is a snapshot of the live data so any changes you make will not be reflected in the production Bugzilla. Email is disabled so feel free to test any aspect of the site that you want. File any problems you find or give feedback at bugzilla.redhat.com.

Bug 213907 (tclpro)

Summary:	Review Request: tclpro - Tcl debugging environment
Product:	[Fedora] Fedora	Reporter:	Wart <wart>
Component:	Package Review	Assignee:	Paul F. Johnson <paul>
Status:	CLOSED NEXTRELEASE	QA Contact:	Fedora Package Reviews List <fedora-package-review>
Severity:	medium	Docs Contact:
Priority:	medium
Version:	rawhide	Flags:	kevin: fedora-cvs+
Target Milestone:	---
Target Release:	---
Hardware:	All
OS:	Linux
Whiteboard:
Fixed In Version:		Doc Type:	Bug Fix
Doc Text:		Story Points:	---
Clone Of:		Environment:
Last Closed:	2006-11-26 01:47:07 UTC	Type:	---
Regression:	---	Mount Type:	---
Documentation:	---	CRM:
Verified Versions:		Category:	---
oVirt Team:	---	RHEL 7.3 requirements from Atomic Host:
Cloudforms Team:	---	Target Upstream Version:
Embargoed:
Bug Depends On:	213596, 213904, 213905
Bug Blocks:	163779

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.