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 169405

Summary: Review Request: regexxer - A nifty GUI search/replace tool
Product: [Fedora] Fedora Reporter: Christoph Wickert <cwickert>
Component: Package ReviewAssignee: Aurelien Bompard <gauret>
Status: CLOSED NEXTRELEASE QA Contact: David Lawrence <dkl>
Severity: medium Docs Contact:
Priority: medium    
Version: rawhideCC: fedora-extras-list
Target Milestone: ---   
Target Release: ---   
Hardware: All   
OS: Linux   
URL: http://regexxer.sourceforge.net/
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2005-09-30 13:40:51 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:    
Bug Blocks: 163779    

Description Christoph Wickert 2005-09-27 21:52:48 UTC
Spec Name or Url: 
http://home.arcor.de/christoph.wickert/fedora/extras-review/SPECS/regexxer.spec
SRPM Name or Url: 
http://home.arcor.de/christoph.wickert/fedora/extras-review/SRPMS/regexxer-0.8-1.fc4.src.rpm
Description: 
Regexxer is a nifty GUI search/replace tool featuring Perl-style regular 
expressions. If you need project-wide substitution and youâre tired of hacking 
sed command lines together, then you should definitely give it a try.

Comments:
- license GPL and included 
- rpmlint looks ok

Comment 1 Aurelien Bompard 2005-09-27 23:15:47 UTC
Review for release 1.fc4:
* RPM name is OK
* Source regexxer-0.8.tar.gz is the same as upstream
* This is the latest version
* Builds fine in mock
* rpmlint of regexxer looks OK
* File list of regexxer looks OK
* Works fine

Minor:
* Requires: gconfmm26 is useless, rpm will pick it up automatically :
$ rpm -qpR regexxer-0.8-1.fc4.i386.rpm | grep gconf
gconfmm26 >= 2.6.1
libgconf-2.so.4
libgconfmm-2.6.so.1

* Duplicate BuildRequires: gtkmm24-devel (required by libglademm24-devel)

Please remove the hardcoded dependency on gconfmm26, and consider yourself APPROVED
(do as you wish with the duplicate BR, its really minor)

Comment 2 Thorsten Leemhuis 2005-09-28 11:57:35 UTC
Christoph, if you do changes (like the ones suggested by Aurelien in #1 that you
did to the regexxer.spec between review and cvs-import) you also should describe
those in the changelog and increase the release.

And IMHO the imported package also should always be exatly the one that was
reviewed. Additional changes like those suggest in #1 should be done directly
after import.

tia

Comment 3 Christoph Wickert 2005-09-28 14:34:12 UTC
Updated to release -2 with complete changelog info. 

Comment 4 Christoph Wickert 2005-09-30 13:40:51 UTC
Arrived at FE servers yesterday.