Spec URL: http://people.parinux.org/~seyman/fedora/perl-Palm/perl-Palm.spec SRPM URL: http://people.parinux.org/~seyman/fedora/perl-Palm/perl-Palm-1.014-1.fc21.src.rpm Description: This module provides functions and handlers to manipulate files used by Palm PDAs (AddressBook, ToDo, Memo, ...). Fedora Account System Username: eseyman Rpmlint output: 1 packages and 1 specfiles checked; 0 errors, 0 warnings. Note that this is a re-review for a package rename. The perl-Palm package is a renaming of the perl-p5-Palm one, upstream having decided to do a name-change.
The standalone spec differs from the one included in the SRPM.
(In reply to Petr Šabata from comment #1) > The standalone spec differs from the one included in the SRPM. Wow, I'm off to a great start on this one. Source RPM rebuilt.
(In reply to Emmanuel Seyman from comment #2) > (In reply to Petr Šabata from comment #1) > > The standalone spec differs from the one included in the SRPM. > > Wow, I'm off to a great start on this one. Source RPM rebuilt. Indeed. This is not a simple rename. The distribution got split into two, at least -- this one and Palm::PDB. The latter needs to be packaged before this one as it's your (and lbdb's) dependency. Wrong lettercase on the Obsoletes line. Mark the LICENSE file with %license; this is now mandatory. Missing BR: perl The minimum required version of EU::MM should be 6.76. That's when support for NO_PACKLIST was introduced. You can substitute PERL_INSTALL_ROOT with DESTDIR. The URL still points to p5-Palm.
(In reply to Petr Šabata from comment #3) > > This is not a simple rename. The distribution got split into two, at least > -- this one and Palm::PDB. The latter needs to be packaged before this one > as it's your (and lbdb's) dependency. Yup, perl-Palm-PDB was reviewed in bug #1187873 and hit rawhide on 2015-02-02. > Wrong lettercase on the Obsoletes line. > Mark the LICENSE file with %license; this is now mandatory. > Missing BR: perl > The minimum required version of EU::MM should be 6.76. That's when support > for NO_PACKLIST was introduced. > You can substitute PERL_INSTALL_ROOT with DESTDIR. > The URL still points to p5-Palm. All fixed. Spec URL: http://people.parinux.org/~seyman/fedora/perl-Palm/perl-Palm.spec SRPM URL: http://people.parinux.org/~seyman/fedora/perl-Palm/perl-Palm-1.014-2.fc21.src.rpm
(In reply to Emmanuel Seyman from comment #4) > (In reply to Petr Šabata from comment #3) > > > > This is not a simple rename. The distribution got split into two, at least > > -- this one and Palm::PDB. The latter needs to be packaged before this one > > as it's your (and lbdb's) dependency. > > Yup, perl-Palm-PDB was reviewed in bug #1187873 and hit rawhide on > 2015-02-02. I see. Totally missed that. > > Wrong lettercase on the Obsoletes line. > > Mark the LICENSE file with %license; this is now mandatory. > > Missing BR: perl > > The minimum required version of EU::MM should be 6.76. That's when support > > for NO_PACKLIST was introduced. > > You can substitute PERL_INSTALL_ROOT with DESTDIR. > > The URL still points to p5-Palm. > > All fixed. > > Spec URL: http://people.parinux.org/~seyman/fedora/perl-Palm/perl-Palm.spec > SRPM URL: > http://people.parinux.org/~seyman/fedora/perl-Palm/perl-Palm-1.014-2.fc21. > src.rpm Ack, everything fixed. Approving.
(In reply to Petr Šabata from comment #5) > > Ack, everything fixed. Approving. Thanks, Petr. New Package SCM Request ======================= Package Name: perl-Palm Short Description: Palm OS utility functions Upstream URL: http://search.cpan.org/dist/Palm/ Owners: eseyman Branches: InitialCC: perl-sig
Git done (by process-git-requests).
Built for rawhide and mashed.