https://bugzilla.redhat.com/show_bug.cgi?id=1187873

Emmanuel Seyman <[email protected]> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
              Flags|                            |fedora-cvs?



--- Comment #2 from Emmanuel Seyman <[email protected]> ---
(In reply to David Dick from comment #1)
> 
> include the following BRs
> 
> perl(warnings)
> perl(strict)

Done.

> if you are building only for F21, 

Rawhide only, actually.
Fedora has the package perl-p5-Palm. A while back, the CPAN distribution was:

a) renamed to simply Palm
b) split into Palm and Palm-PDB
c) updated

So the plan is to package Palm-PDB (this bug) in rawhide and then rename
perl-p5-Palm to perl-Palm (which will require another review).

> "find $RPM_BUILD_ROOT -type f -name .packlist -exec rm -f {} \;" looks like
> it can be replaced with NO_PACKLIST=1 as a Makefile.PL argument.

Done

Spec URL:
http://people.parinux.org/~seyman/fedora/perl-Palm-PDB/perl-Palm-PDB.spec
SRPM URL:
http://people.parinux.org/~seyman/fedora/perl-Palm-PDB/perl-Palm-PDB-1.016-2.fc21.src.rpm

Thanks for the review, David.
Requesting SCM

New Package SCM Request
=======================
Package Name: perl-Palm-PDB
Short Description: Parse Palm database files
Upstream URL: http://search.cpan.org/dist/Palm-PDB//pkgname
Owners: eseyman
Branches:
InitialCC: perl-sig

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
_______________________________________________
package-review mailing list
[email protected]
https://admin.fedoraproject.org/mailman/listinfo/package-review

Reply via email to