Bug 216785 - Review Request: perl-Gnome2-Print - Perl wrappers for the Gnome Print utilities
Summary: Review Request: perl-Gnome2-Print - Perl wrappers for the Gnome Print utilities
Keywords:
Status: CLOSED NEXTRELEASE
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Parag AN(पराग)
QA Contact: Fedora Package Reviews List
URL: http://search.cpan.org/dist/Gnome2-Pr...
Whiteboard:
Depends On:
Blocks: FE-ACCEPT
TreeView+ depends on / blocked
 
Reported: 2006-11-22 00:03 UTC by Chris Weyl
Modified: 2007-11-30 22:11 UTC (History)
1 user (show)

Fixed In Version:
Clone Of:
Environment:
Last Closed: 2006-11-22 15:46:23 UTC
Type: ---
Embargoed:


Attachments (Terms of Use)

Description Chris Weyl 2006-11-22 00:03:52 UTC
SRPM URL: http://home.comcast.net/~ckweyl/perl-Gnome2-Print-1.000-1.fc6.src.rpm
SPEC URL: http://home.comcast.net/~ckweyl/perl-Gnome2-Print.spec

Description:
This module allows you to use the GNOME Print libraries within your
applications written using the gtk2-perl wrapper. The GNOME Print libraries
(also known as libgnomeprint and libgnomeprintui) allow you to create
printable documents (using various frontends) and offer standard widgets in
order to mainatin a UI consistent for all GNOME applications.

Comment 1 Parag AN(पराग) 2006-11-22 04:01:38 UTC
Tell me how can i use this package or test its working in FC6 core?

Comment 2 Chris Weyl 2006-11-22 04:10:58 UTC
There are a number of sample scripts that come in the examples directory (or,
when the rpm is installed, under %doc).  Running one or more of those should
enable you to examine the functionailty of the module proper.

Comment 3 Parag AN(पराग) 2006-11-22 05:58:25 UTC
Did mock build and found under build.log
Unrecognized argument in LIBS ignored: '-pthread'
Got some rpmlint Warnings
W: perl-Gnome2-Print devel-file-in-non-devel-package
/usr/lib/perl5/vendor_perl/5.8.8/i386-linux-thread-multi/Gnome2/Print/Install/gnomeprintperl-autogen.h
A development file (usually source code) is located in a non-devel
package. If you want to include source code in your package, be sure to
create a development package.

W: perl-Gnome2-Print devel-file-in-non-devel-package
/usr/lib/perl5/vendor_perl/5.8.8/i386-linux-thread-multi/Gnome2/Print/Install/gnomeprintperl-version.h
A development file (usually source code) is located in a non-devel
package. If you want to include source code in your package, be sure to
create a development package.

W: perl-Gnome2-Print devel-file-in-non-devel-package
/usr/lib/perl5/vendor_perl/5.8.8/i386-linux-thread-multi/Gnome2/Print/Install/gnomeprintperl.h
A development file (usually source code) is located in a non-devel
package. If you want to include source code in your package, be sure to
create a development package.

are those can be ignored for perl packages??

Comment 4 Chris Weyl 2006-11-22 06:05:34 UTC
Those warnings can be safely ignored for perl packages.

Comment 5 Parag AN(पराग) 2006-11-22 06:32:52 UTC
Not able to find license file(Is am i missing something)?
Still Unrecognized argument in LIBS ignored: '-pthread' message coming
not understanding which causes this message to appear?

Comment 6 Ralf Corsepius 2006-11-22 06:46:34 UTC
(In reply to comment #5)
> Not able to find license file(Is am i missing something)?
Yes, you are missing something: Nobody is obliged to ship one.

Perl module files typically contain licenses embedded into their source files.

> Still Unrecognized argument in LIBS ignored: '-pthread' message coming
> not understanding which causes this message to appear?
IIRC, this is a bug/crappy code somewhere in some of the perl-Gtk/Gnome
infrastructure. I recall this had appeared several times before, but I don't
recall all details on how this had been resolved. Googling/searching bugzilla
might provide further insights.

Comment 7 Parag AN(पराग) 2006-11-22 07:07:45 UTC
Does that be a blocker?

Comment 8 Ralf Corsepius 2006-11-22 08:01:50 UTC
(In reply to comment #7)
> Does that be a blocker?
I just found a reference:
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=166252

... so people seem to preferr to ignore this bug :(

Comment 9 Parag AN(पराग) 2006-11-22 09:27:39 UTC
Review:
+ package builds in mock (development i386).
+ rpmlint is silent for SRPM and but NOT for RPMS.
  But those warnings are ignored.
+ source files match upstream.
66578c2ffaebbe035a0735e65ad71c3f  Gnome2-Print-1.000.tar.gz
+ package meets naming and packaging guidelines.
+ specfile is properly named, is cleanly written
+ Spec file is written in American English.
+ Spec file is legible.
+ dist tag is present.
+ build root is correct.
+ license is open source-compatible.  License text included in package.
+ %doc is small; no -doc subpackage required.
+ %doc does not affect runtime.
+ BuildRequires are proper.
+ %clean is present.
+ package installed properly.
+ Macro use appears rather consistent.
+ Package contains code, not content.
+ no headers or static libraries.
+ .pc file present.
+ no -devel subpackage exists
+ no .la files.
+ no translations are available
+ Dose owns the directories it creates.
+ no duplicates in %files.
+ file permissions are appropriate.
+ Followed perl packaging guidelines.
APPROVED.


Comment 10 Parag AN(पराग) 2006-11-22 09:31:26 UTC
Don't Forget to CLOSE this with NEXTRELEASE

Comment 11 Chris Weyl 2006-11-22 15:46:23 UTC
Imported and building for devel; branches requested for FC5,6.

Thanks for the review! :)


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