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 166254

Summary: Review Request: perl-Imager
Product: [Fedora] Fedora Reporter: Gavin Henry <ghenry>
Component: Package ReviewAssignee: Jef Spaleta <jspaleta>
Status: CLOSED NEXTRELEASE QA Contact: David Lawrence <dkl>
Severity: medium Docs Contact:
Priority: medium    
Version: rawhideCC: alex, ddick, fedora-extras-list, rc040203, steve
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-03-16 00:18:16 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 Flags
spec file fixes none

Description Gavin Henry 2005-08-18 11:34:42 UTC
Spec Url:


Description: Imager is a module for creating and altering images. It can read and write various image formats, draw primitive shapes like lines,and polygons, blend multiple images together in various ways, scale, crop, render text and more.

Comment 1 Jef Spaleta 2005-08-19 03:20:00 UTC
built on against a rawhide box, rpmlint perl-Imager-0.45-1.i386.rpm:
W: perl-Imager unstripped-binary-or-object

 ->Ignorable warning

E: perl-Imager wrong-script-interpreter
/usr/share/doc/perl-Imager-0.45/samples/ "perl"
E: perl-Imager wrong-script-interpreter
/usr/share/doc/perl-Imager-0.45/samples/ "perl"

  ->Can you patch the sample scripts in the samples directory to use
    #!/usr/bin/perl -w  instead of #!perl -w   and should they be executable?

E: perl-Imager wrong-script-interpreter
/usr/lib/perl5/vendor_perl/5.8.6/i386-linux-thread-multi/Imager/Font/ "perl"
E: perl-Imager non-executable-script

  ->Same issue. is the only pm file in the Font subdirectory that has an  
    interpreter defined in the header. Either this needs to be corrected like 
    the sample scripts or the interpreter line should be removed. I'm not sure 
    which workaround is correct for  Is it meant to be run as a 
    stand-alone executable script?


Comment 2 Jef Spaleta 2005-08-19 05:58:45 UTC
additionally you'll need to clean up the same problems with this package as you
had with perl-Apache-LogRegex

- just use %check
- remove the buildrequires on perl
- need to include the license text in the docs section for the package.

I haven't made it through the full review of this yet.. but i just want to make
you aware that the same minor problems exist here. Most likely the other two
pending perl packages for sprog deps have the same easy to fix issues too, so
its a good idea to check over those packages as well for these 3 items.


Comment 3 Gavin Henry 2005-08-19 09:42:25 UTC
I'll get this done next week.

Comment 4 Jef Spaleta 2005-08-26 02:52:03 UTC
Looks like your need to apply 
%{__perl} Makefile.PL INSTALLDIRS=vendor OPTIMIZE="$RPM_OPT_FLAGS"
for this package to, as discussed for the other sprog deps so far.


Comment 5 Steven Pritchard 2005-08-29 17:49:17 UTC
I'm attaching a patch to the spec file that eliminates just about every issue I
can find (including the more-or-less bogus warnings from rpmlint).

I'll verify it with a mock build later today.

Comment 6 Steven Pritchard 2005-08-29 17:53:10 UTC
Created attachment 118219 [details]
spec file fixes

Remove explicit perl BuildRequires.
BuildRequires: perl(Parse::RecDescent) (not perl-Parse-RecDescent).
BuildRequires: t1lib-devel.
Various things to quiet rpmlint.
Pass OPTIMIZE to Makefile.PL.
"%check" not "%check || :".
Include COPYING and Artistic.

Comment 7 Gavin Henry 2005-08-29 17:58:20 UTC
Well, looks like people are being impatient ;-)

I was going to knock Sprog and all these RPMs off my list tomorrow and Wednesday.

I'll apply the patch/



Comment 9 Jef Spaleta 2005-09-15 00:37:27 UTC
Okay let me go through the full review now.  
builds in mock for fedora development on x86.

*naming is good
*perl specific spec file conventions are being used
*license tag matches the wording in the readme
*readme with licensing terms include in docs
*md5sum of sources match upstream
*rpmlint returns clean
*buildrequires look good, all the supprted image and font support appears to be
included at build time according to the build log. 
*no locale files to worry about
*no shared libraries in the linker path
*created directory ownership is good
*clean section good
*no pre/post scriptlets 
*no static libs
*files section looks good
*no executables installed
*no executables in docs section 
*rpmbuild --rebuild --target i686 works 
*sample scripts appear to work.. no errors or segfaults.

Okay this looks good to me. I'll start the clock on this. I'll wait 24 hour
before approving this in case someone finds a blocker.


Comment 10 Steven Pritchard 2005-09-16 20:47:02 UTC
It's been >24 hours Jef.  ;-)

Comment 11 Jef Spaleta 2005-09-17 00:11:11 UTC

Comment 12 Alex Lancaster 2006-02-25 23:47:41 UTC
Since this has been built in development, this bug should be CLOSED as
NEXTRELEASE.  (However I had troubles rebuilding this in development similar to
what happens in bug #182459 for FC-4).

Comment 13 Steven Pritchard 2006-03-16 00:12:31 UTC
Ping?  Any idea why this hasn't been closed yet?

Comment 14 Robin Lee 2011-01-21 15:44:51 UTC
*** Bug 665331 has been marked as a duplicate of this bug. ***

Comment 15 David Dick 2014-03-15 05:35:35 UTC
*** Bug 1076745 has been marked as a duplicate of this bug. ***