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

Bug 175502 (perl-Gtk2-Spell)

Summary: Review Request: perl-Gtk2-Spell
Product: [Fedora] Fedora Reporter: Chris Weyl <cweyl>
Component: Package ReviewAssignee: Chris Weyl <cweyl>
Status: CLOSED NEXTRELEASE QA Contact: Fedora Package Reviews List <fedora-package-review>
Severity: medium Docs Contact:
Priority: medium    
Version: rawhide   
Target Milestone: ---   
Target Release: ---   
Hardware: All   
OS: Linux   
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2006-04-25 22:08:49 UTC Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Bug Depends On:    
Bug Blocks: 163779    

Description Chris Weyl 2005-12-12 00:17:15 UTC
Spec Name or Url:
SRPM Name or Url:

Description: gtkspell bindings for perl(Gtk2)
Perl bindings to GtkSpell, used in concert with Gtk2::TextView. Provides mis-spelled word highlighting in red and offers a right click pop-up menu with suggested corrections.

Please ignore the ".ckw.fc4" in the srpm's filename -- I have %dist set to that in my ~/.rpmmacros.

This is my first extras package, and I'm seeking a sponsor! :)

Comment 1 Kevin Fenzi 2005-12-14 01:00:29 UTC
NOTE: I cannot sponsor, but here's a review. :)

- Package name is good. 
- License is good and matches upstream (LGPL)
- Source is good and matches upstream
- builds fine on fc4/devel. 
- package seems to work as expected using the include test perl script. 

- rpmlint warning: 
W: perl-Gtk2-Spell devel-file-in-non-devel-package
- why bother to use the %realname macro? Is the package name expected to change
very often? :) 
- why the 'echo n | ' in the build section? 
- Might include the Changelog file as a doc. 

Comment 2 Chris Weyl 2005-12-14 18:26:47 UTC
Updated spec/srpm:
Spec Name or Url:
SRPM Name or Url:

%realname is used as I was creating a bunch of module rpms for my own use; this
macro allowed me to quickly take a template spec file and cusomize it.

Minor issues addressed:
- 'echo n |' in %build nixed.
- ChangeLog included as a doc

Comment 3 Josh Boyer 2006-01-27 19:30:25 UTC

All of the above and no more rpmlint warnings


- perl is not needed as a BuildRequires
- ditch the %realname macro

Fixup those two remaining issues and I think this is ready for approval

Comment 4 Chris Weyl 2006-02-17 01:40:03 UTC
Updated spec/srpm:
Spec Name or Url:
SRPM Name or Url:

TODOs from #3 addressed; release tag reset to 1.

Comment 5 Ralf Corsepius 2006-02-17 10:50:17 UTC
Please remove the perl_* stuff at the very beginning of the spec.

Also, please don't reset the release tag, and increment it instead even during

Comment 6 Chris Weyl 2006-02-18 16:27:45 UTC
Updated spec/srpm:
Spec Name or Url:
SRPM Name or Url:

Updated per #5.

Comment 7 Josh Boyer 2006-02-20 13:02:23 UTC
This package is accepted.

Chris, you'll need to apply for CVS access here:

In the meantime, I've imported the SRPM into CVS.  Once you get CVS access, you
can request a build and create any branches you'd like.

Comment 8 Christian Iseli 2006-03-28 15:25:02 UTC
Any particular reason why this package is not yet built and in owners.list ?

Comment 9 Chris Weyl 2006-03-28 15:46:03 UTC
Yes, Real World/Life issues.  I'm just at the tail end of a transition from
living/working in Connecticut to California; I should be back up & running by
the end of the week.  Assuming the movers show up with my stuff today, of course :)

Comment 10 Hans de Goede 2006-06-08 07:40:24 UTC
Remving form FE_NEEDSPONSOR tracker bug, since this is closed.