Spec URL: http://linux.davisnetworks.com/yumrepo/fc/16/SPEC/perl-DBIx-Array.spec SRPM URL: http://linux.davisnetworks.com/yumrepo/fc/16/SRPMS/perl-DBIx-Array-0.22-1.fc16.src.rpm Description: This module is for people who truly understand SQL and who understand Perl data structures. If you understand how to modify your SQL to meet your data requirements then this module is for you. --- This is my package from CPAN http://search.cpan.org/dist/DBIx-Array/ that I'd like to get into Fedora. FE-NEEDSPONSOR: This is not my first package on Fedora (e.g. perl-Geo-Ellipsoids) but this will be the first one that I'll be maintaining. So, I will need a sponsor. I've worked with Miroslav Lichvar <mlichvar> on gpsd in the past so I was hoping to con him into the task. Thanks, Mike mrdvt92
Hello Mike, Spec file looks good, rpmlint is silent on both spec file and SRPM. One question though: Requires: perl(DBI) Doesn't rpmbuild find this dependency? If it does, this Requires is redundant and should be removed.
$ rpm -q --requires perl-DBIx-Array perl(:MODULE_COMPAT_5.14.1) perl(DBI) perl(DBI) perl(base) perl(strict) perl(warnings) rpmlib(CompressedFileNames) <= 3.0.4-1 rpmlib(PayloadFilesHavePrefix) <= 4.0-1 I guess it is found. I don't understand why it matters but if that's what you need I'll drop it. I also fix three misspellings and will have the new version 0.23 on CPAN as soon as I burn it in.
Updated URLs for version 0.23 Spec URL: http://linux.davisnetworks.com/yumrepo/fc/16/SPEC/perl-DBIx-Array.spec SRPM URL: http://linux.davisnetworks.com/yumrepo/fc/16/SRPMS/perl-DBIx-Array-0.23-1.fc16.src.rpm hard coded perl(DBI) removed $ rpm -q --requires perl-DBIx-Array perl(:MODULE_COMPAT_5.14.2) perl(DBI) perl(base) perl(strict) perl(warnings) rpmlib(CompressedFileNames) <= 3.0.4-1 rpmlib(FileDigests) <= 4.6.0-1 rpmlib(PayloadFilesHavePrefix) <= 4.0-1 rpmlib(PayloadIsXz) <= 5.2-1 spec - rpmlint is clean srpm - rpmlint is clean
Just jumped into a random package review request for a very brief look: > %changelog > * Sun Aug 28 2011 Michael R. Davis (...) 0.22-1 > - See Changes files In the spec %changelog you're supposed to sum up only your changes related to the RPM packaging of the software. For example, mention what changed in the spec file (= the src.rpm): https://fedoraproject.org/wiki/Packaging:Guidelines#Changelogs During the review phase, if multiple updates of the src.rpm are needed, you can negotiate with the reviewer that you don't need to sum up every minor change of the spec file. Practising the creation of proper %changelog entries can be helpful training, however. ;)
Updated URLs for version 0.23-2 Spec URL: http://linux.davisnetworks.com/yumrepo/fc/16/SPEC/perl-DBIx-Array.spec SRPM URL: http://linux.davisnetworks.com/yumrepo/fc/16/SRPMS/perl-DBIx-Array-0.23-2.fc16.src.rpm %changelog +* Fri Nov 25 2011 Michael R. Davis (mdavis) 0.23-2 +- Updated changelog (#754892) + +* Mon Nov 21 2011 Michael R. Davis (mdavis) 0.23-1 +- Updated for version 0.23 +- Removed hard coded requires for DBI since rpmbuild finds them from sources (#754892) + * Sun Aug 28 2011 Michael R. Davis (mdavis) 0.22-1 -- See Changes files +- Created spec file with cpanspec I really do appreciate the review process. I hope that my next few packages will be correct right out of the gates. It's a learning experience thinking about how this can be run on multiple OSes both from the past and in the future. Thanks, Mike mrdvt92
Closing as requested by submitter in bug 759697.