Bug 539863 - Review Request: perl-Pod-PseudoPod - Extending the POD tags for book manuscripts
Summary: Review Request: perl-Pod-PseudoPod - Extending the POD tags for book manuscripts
Keywords:
Status: CLOSED ERRATA
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Lubomir Rintel
QA Contact: Fedora Extras Quality Assurance
URL: http://search.cpan.org/dist/Pod-Pseud...
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2009-11-21 11:43 UTC by Gerd Pokorra
Modified: 2010-02-08 06:55 UTC (History)
4 users (show)

Fixed In Version: 0.15-1.fc12
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2009-12-24 20:39:18 UTC
Type: ---
lkundrak: fedora-review+
kevin: fedora-cvs+


Attachments (Terms of Use)

Description Gerd Pokorra 2009-11-21 11:43:36 UTC
Spec URL: 
ftp://ftp.uni-siegen.de/pub/parrot.rpms/PseudoPod/perl-Pod-PseudoPod.spec

SRPM URL: 
http://koji.fedoraproject.org/koji/getfile?taskID=1821155&name=perl-Pod-PseudoPod-0.15-1.fc12.src.rpm

Description:
PseudoPOD is an extended set of POD tags for book manuscripts.
Standard POD doesn't have all the markup options you might want for
marking up files for publishing production. PseudoPOD adds a few extra
tags for footnotes and tables, and has extended uses of "=begin"
elements. See Pod::PseudoPod::Tutorial.

Comment 1 Gerd Pokorra 2009-11-25 15:49:03 UTC
The URL of the latest scratch build is:

http://koji.fedoraproject.org/koji/taskinfo?taskID=1830099

Comment 2 Gerd Pokorra 2009-11-25 18:34:14 UTC
The "BuildArch is set to "noarch".

The file under the Spec-URL is updated.
ftp://ftp.uni-siegen.de/pub/parrot.rpms/PseudoPod/perl-Pod-PseudoPod.spec

You can find latest SRPM at the URL:
ftp://ftp.uni-siegen.de/pub/parrot.rpms/PseudoPod/perl-Pod-PseudoPod-0.15-1.fc11.src.rpm

The latest scratch build is:
http://koji.fedoraproject.org/koji/taskinfo?taskID=1830833

The rpmlint output reports: 0 errors, 0 warnings

Comment 3 Gerd Pokorra 2009-11-30 13:43:35 UTC
Changed the basis to an autogenerated specfile.
ExtUtils::MakeMaker is replaced with Module::Build.

The file under the URLs are updated:
ftp://ftp.uni-siegen.de/pub/parrot.rpms/PseudoPod/perl-Pod-PseudoPod.spec
ftp://ftp.uni-siegen.de/pub/parrot.rpms/PseudoPod/perl-Pod-PseudoPod-0.15-1.fc11.src.rpm

The latest scratch build is:
http://koji.fedoraproject.org/koji/taskinfo?taskID=1838062

The rpmlint output reports: 0 errors, 0 warnings

Comment 4 Lubomir Rintel 2009-12-19 15:37:05 UTC
* Named and versioned in accordance with guidelines
* License tag correct, license ok
* SPEC file clean and legible
* Builds fine in mock
* RPMlint silent and happy
* Dependencies sane
* Source archive matches upstream

Nothing that would block the issues,
the package is

APPROVED

A few small suggestions:

1.) You do not need to explicitly require perl(HTML::Entities)

It's automatically in by autoreq (you probably still need the rest, as you seem to need specific versions).

2.) You have redundant entries in %files, even RPM complains:

warning: File listed twice: /usr/share/perl5/Pod/PseudoPod/Checker.pm
warning: File listed twice: /usr/share/perl5/Pod/PseudoPod/DocBook.pm

There are two solutions to the problem. One is prepending the "%{perl_vendorlib}/Pod/PseudoPod" line with "%doc" so that RPM doesn't automatically add its descendants recursively. Alternatively, you can leave the line as is and remove all its descendants below.

Comment 5 Gerd Pokorra 2009-12-21 16:07:42 UTC
Few little changes are made, which reflect the suggestions.
The URLs and version number remain the same.

Comment 6 Gerd Pokorra 2009-12-21 16:14:05 UTC
New Package CVS Request
=======================
Package Name: perl-Pod-PseudoPod
Short Description: Framework for extending the POD tags for book manuscripts
Owners: gerd
Branches: F-11 F-12
InitialCC: gerd

Comment 7 Kevin Fenzi 2009-12-21 19:46:19 UTC
cvs done.

Comment 8 Fedora Update System 2009-12-22 09:18:50 UTC
perl-Pod-PseudoPod-0.15-1.fc12 has been submitted as an update for Fedora 12.
http://admin.fedoraproject.org/updates/perl-Pod-PseudoPod-0.15-1.fc12

Comment 9 Fedora Update System 2009-12-22 09:31:56 UTC
perl-Pod-PseudoPod-0.15-1.fc11 has been submitted as an update for Fedora 11.
http://admin.fedoraproject.org/updates/perl-Pod-PseudoPod-0.15-1.fc11

Comment 10 Fedora Update System 2009-12-24 20:39:14 UTC
perl-Pod-PseudoPod-0.15-1.fc11 has been pushed to the Fedora 11 stable repository.  If problems still persist, please make note of it in this bug report.

Comment 11 Fedora Update System 2009-12-24 20:45:11 UTC
perl-Pod-PseudoPod-0.15-1.fc12 has been pushed to the Fedora 12 stable repository.  If problems still persist, please make note of it in this bug report.

Comment 12 Gerd Pokorra 2010-02-01 11:52:30 UTC
New Package CVS Request
=======================
Package Name: perl-Pod-PseudoPod
Owners: gerd
Branches: EL-5

Comment 13 Christoph Wickert 2010-02-04 20:18:55 UTC
This is the correct form:

Package Change Request
======================
Package Name: perl-Pod-PseudoPod
New Branches: EL-5
Owners: gerd

Comment 14 Gerd Pokorra 2010-02-06 09:11:51 UTC
Package Change Request
======================
Package Name: perl-Pod-PseudoPod
New Branches: EL-5
Owners: gerd

This package also builds on the EL-5 brunch. You can see it at this scratch build:
http://koji.fedoraproject.org/koji/taskinfo?taskID=1955811

Comment 15 Kevin Fenzi 2010-02-07 21:22:10 UTC
Yes, thats the correct form. ;) 

Thanks. 

cvs done.

Comment 16 Lubomir Rintel 2010-02-08 06:55:16 UTC
"EL-5 brunch". Love that :)
Sorry, I couldn't help myself


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