Spec URL: http://ktdreyer.fedorapeople.org/reviews/perl-Net-INET6Glue.spec SRPM URL: http://ktdreyer.fedorapeople.org/reviews/perl-Net-INET6Glue-0.5-1.fc20.src.rpm Description: Net::INET6Glue is a collection of modules to make common modules IPv6 ready by hotpatching them. Fedora Account System Username: ktdreyer F20 scratch build: http://koji.fedoraproject.org/koji/taskinfo?taskID=5389896 EL6 scratch build: http://koji.fedoraproject.org/koji/taskinfo?taskID=5389920 (I won't be supporting EL5 in this package.)
(In reply to comment #0) > (I won't be supporting EL5 in this package.) Non-issue: AFAICT, in this case, you can remove BuildRoot, rm -rf %{buildroot} etc. from the spec (I currently can't verify). However, there is one major issue: This package's Makefile.PL expects interactive input: This causes rpmbuild'ing the package (outside of mock) to wait for keyboard input, i.e. rpmbuild to hang: # rpmbuild --rebuild /tmp/perl-Net-INET6Glue-0.5-1.fc20.src.rpm ... Should I do external tests? These tests will fail if there is no internet connection or if a firewall blocks active or passive IPv6 FTP traffic. [y/N] [n]
Thanks Ralf, here's release 2. * Sun May 19 2013 Ken Dreyer <ktdreyer> 0.5-2 - Run Makefile.pl with PERL_MM_USE_DEFAULT=1 so that we skip the prompt during an interactive rpmbuild session (eg. outside of mock). - Remove BuildRoot and rm -rf $RPM_BUILD_ROOT. (EL5 compat not required.) Spec URL: http://ktdreyer.fedorapeople.org/reviews/perl-Net-INET6Glue.spec SRPM URL: http://ktdreyer.fedorapeople.org/reviews/perl-Net-INET6Glue-0.5-2.fc20.src.rpm
Sorry, for this review having taken so long, but I simply had forgotten about it. Nothing much to say about it: APPROVED Two minor issues: 1) Redundant "Requires:" # rpm -q --requires -p perl-Net-INET6Glue-0.5-2.fc18.noarch.rpm ... perl(IO::Socket::INET6) perl(IO::Socket::INET6) >= 2.54 ... The first R: perl(IO::Socket::INET6) is redundant to the second one. Consider to filter the first one from inside of the spec rsp. to remove the versioned one from the spec. 2) Consider to add these perl-modules to "BR:" perl(Carp) perl(IO::Socket::INET) perl(IO::Socket::INET6) perl(Net::FTP) perl(Socket) perl(strict) perl(warnings) Other reviewers would insist on them, I don't.
New Package SCM Request ======================= Package Name: perl-Net-INET6Glue Short Description: Make common modules IPv6 ready by hotpatching them Owners: ktdreyer Branches: f19 f20 el6
Please add InitialCC: perl-sig as described on http://fedoraproject.org/wiki/Packaging:Perl
New Package SCM Request ======================= Package Name: perl-Net-INET6Glue Short Description: Make common modules IPv6 ready by hotpatching them Owners: ktdreyer Branches: f19 f20 el6 InitialCC: perl-sig
Git done (by process-git-requests).
perl-Net-INET6Glue-0.5-3.fc20 has been submitted as an update for Fedora 20. https://admin.fedoraproject.org/updates/perl-Net-INET6Glue-0.5-3.fc20
perl-Net-INET6Glue-0.5-3.fc19 has been submitted as an update for Fedora 19. https://admin.fedoraproject.org/updates/perl-Net-INET6Glue-0.5-3.fc19
perl-Net-INET6Glue-0.5-3.el6 has been submitted as an update for Fedora EPEL 6. https://admin.fedoraproject.org/updates/perl-Net-INET6Glue-0.5-3.el6
Thank you Ralf for the thorough review. In Git I pushed the initial "-2" release, and then a second Git commit for release "-3" with your suggested changes for Requires/BuildRequires.
perl-Net-INET6Glue-0.5-3.el6 has been pushed to the Fedora EPEL 6 testing repository.
perl-Net-INET6Glue-0.5-3.fc19 has been pushed to the Fedora 19 stable repository.
perl-Net-INET6Glue-0.5-3.fc20 has been pushed to the Fedora 20 stable repository.
perl-Net-INET6Glue-0.5-3.el6 has been pushed to the Fedora EPEL 6 stable repository.