Bug 1315319 - Review Request: perl-ExtUtils-HasCompiler - Check for the presence of a compiler
Summary: Review Request: perl-ExtUtils-HasCompiler - Check for the presence of a compiler
Keywords:
Status: CLOSED ERRATA
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Petr Pisar
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks: 1315370
TreeView+ depends on / blocked
 
Reported: 2016-03-07 13:18 UTC by Jitka Plesnikova
Modified: 2016-03-26 18:18 UTC (History)
2 users (show)

Fixed In Version: perl-ExtUtils-HasCompiler-0.012-1.fc25
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2016-03-26 18:18:59 UTC
Type: ---
Embargoed:
ppisar: fedora-review+


Attachments (Terms of Use)

Description Jitka Plesnikova 2016-03-07 13:18:37 UTC
Spec URL: https://jplesnik.fedorapeople.org/perl-ExtUtils-HasCompiler/perl-ExtUtils-HasCompiler.spec
SRPM URL: https://jplesnik.fedorapeople.org/perl-ExtUtils-HasCompiler/perl-ExtUtils-HasCompiler-0.012-1.fc25.src.rpm
Description:
This module tries to check if the current system is capable of compiling, linking and loading an XS module.

Fedora Account System Username: jplesnik

Comment 1 Upstream Release Monitoring 2016-03-07 17:08:32 UTC
ppisar's scratch build of perl-ExtUtils-HasCompiler-0.012-1.fc25.src.rpm for f25 completed http://koji.fedoraproject.org/koji/taskinfo?taskID=13262026

Comment 2 Petr Pisar 2016-03-07 17:11:25 UTC
The standalone spec file has better dependencies. I will use it for this review.

URL and Source0 addresses are usable Ok.
Source archive is original (SHA-256: 4e762a454cf301ebb537a2080b942711c93640013204b2461452fdc60c0a2f85). Ok.
License verified from README, LICENSE, lib/ExtUtils/HasCompiler.pm, Makefile.PL. Ok.
Summary verified from lib/ExtUtils/HasCompiler.pm. Ok.
Description verified from lib/ExtUtils/HasCompiler.pm. Ok.

I agree the perl-devel should not be run-required as the module is designed to report missing compiler and Perl header files. I'm not sure about build-requring it. The tests use ExtUtils::CBuilder for that and it is build-required.

All tests pass. Ok

$ rpmlint perl-ExtUtils-HasCompiler.spec ../SRPMS/perl-ExtUtils-HasCompiler-0.012-1.fc25.src.rpm ../RPMS/noarch/perl-ExtUtils-HasCompiler-0.012-1.fc25.noarch.rpm 
2 packages and 1 specfiles checked; 0 errors, 0 warnings.
rpmlint is Ok.

$ rpm -q -lv -p ../RPMS/noarch/perl-ExtUtils-HasCompiler-0.012-1.fc25.noarch.rpm 
drwxr-xr-x    2 root    root                        0 Mar  7 18:03 /usr/share/doc/perl-ExtUtils-HasCompiler
-rw-r--r--    1 root    root                     1824 Jul 15  2015 /usr/share/doc/perl-ExtUtils-HasCompiler/Changes
-rw-r--r--    1 root    root                      384 Jul 15  2015 /usr/share/doc/perl-ExtUtils-HasCompiler/README
drwxr-xr-x    2 root    root                        0 Mar  7 18:03 /usr/share/licenses/perl-ExtUtils-HasCompiler
-rw-r--r--    1 root    root                    18358 Jul 15  2015 /usr/share/licenses/perl-ExtUtils-HasCompiler/LICENSE
-rw-r--r--    1 root    root                     1641 Mar  7 18:03 /usr/share/man/man3/ExtUtils::HasCompiler.3pm.gz
drwxr-xr-x    2 root    root                        0 Mar  7 18:03 /usr/share/perl5/vendor_perl/ExtUtils
-rw-r--r--    1 root    root                     6136 Jul 15  2015 /usr/share/perl5/vendor_perl/ExtUtils/HasCompiler.pm
File layout and permissions are Ok.

$ rpm -q --requires -p ../RPMS/noarch/perl-ExtUtils-HasCompiler-0.012-1.fc25.noarch.rpm | sort -f | uniq -c
      1 perl(:MODULE_COMPAT_5.22.1)
      1 perl(base)
      1 perl(Carp)
      1 perl(Config)
      1 perl(DynaLoader)
      1 perl(Exporter)
      1 perl(ExtUtils::Mksymlists)
      1 perl(File::Basename)
      1 perl(File::Spec::Functions)
      1 perl(File::Temp)
      1 perl(strict)
      1 perl(warnings)
      1 rpmlib(CompressedFileNames) <= 3.0.4-1
      1 rpmlib(FileDigests) <= 4.6.0-1
      1 rpmlib(PayloadFilesHavePrefix) <= 4.0-1
      1 rpmlib(PayloadIsXz) <= 5.2-1
Binary requires are Ok.

$ rpm -q --provides -p ../RPMS/noarch/perl-ExtUtils-HasCompiler-0.012-1.fc25.noarch.rpm | sort -f | uniq -c
      1 perl(ExtUtils::HasCompiler) = 0.012
      1 perl-ExtUtils-HasCompiler = 0.012-1.fc25
Binary provides are Ok.

$ resolvedeps rawhide ../RPMS/noarch/perl-ExtUtils-HasCompiler-0.012-1.fc25.noarch.rpm
Binary dependencies resolvable. Ok.

Package builds in F25 (http://koji.fedoraproject.org/koji/taskinfo?taskID=13262026). Ok.

This package is in line with Fedora and Perl packaging guidelines.
Resolution: Package APPROVED.

Comment 3 Gwyn Ciesla 2016-03-08 16:04:51 UTC
Package request has been approved: https://admin.fedoraproject.org/pkgdb/package/perl-ExtUtils-HasCompiler

Comment 4 Fedora Update System 2016-03-08 17:08:57 UTC
perl-ExtUtils-HasCompiler-0.012-1.fc24 has been submitted as an update to Fedora 24. https://bodhi.fedoraproject.org/updates/FEDORA-2016-a7ffad0e85

Comment 5 Jitka Plesnikova 2016-03-08 17:09:31 UTC
Thank you for the review and the repository.

Comment 6 Fedora Update System 2016-03-10 01:57:13 UTC
perl-ExtUtils-HasCompiler-0.012-1.fc24 has been pushed to the Fedora 24 testing repository. If problems still persist, please make note of it in this bug report.
See https://fedoraproject.org/wiki/QA:Updates_Testing for
instructions on how to install test updates.
You can provide feedback for this update here: https://bodhi.fedoraproject.org/updates/FEDORA-2016-a7ffad0e85

Comment 7 Fedora Update System 2016-03-26 18:18:57 UTC
perl-ExtUtils-HasCompiler-0.012-1.fc24 has been pushed to the Fedora 24 stable repository. If problems still persist, please make note of it in this bug report.


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