Bug 2363932 - Review Request: perl-ExtUtils-Builder-Compiler - Interface around different compilers
Summary: Review Request: perl-ExtUtils-Builder-Compiler - Interface around different c...
Keywords:
Status: CLOSED ERRATA
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Jitka Plesnikova
QA Contact: Fedora Extras Quality Assurance
URL: https://metacpan.org/dist/ExtUtils-Bu...
Whiteboard:
Depends On: 2363931
Blocks: 2363933
TreeView+ depends on / blocked
 
Reported: 2025-05-04 05:55 UTC by Charles R. Anderson
Modified: 2025-09-12 19:21 UTC (History)
2 users (show)

Fixed In Version:
Clone Of:
Environment:
Last Closed: 2025-08-27 00:55:31 UTC
Type: ---
Embargoed:
jplesnik: fedora-review+


Attachments (Terms of Use)

Description Charles R. Anderson 2025-05-04 05:55:33 UTC
Spec URL: https://cra.fedorapeople.org/perl-ExtUtils-Builder-Compiler/perl-ExtUtils-Builder-Compiler.spec
SRPM URL: https://cra.fedorapeople.org/perl-ExtUtils-Builder-Compiler/perl-ExtUtils-Builder-Compiler-0.031-2.fc43.src.rpm
Description: This is an interface wrapping around different compilers. It's usually not
used directly but by a portability layer like
ExtUtils::Builder::Autodetect::C.

Fedora Account System Username: cra

Comment 1 Fedora Review Service 2025-05-04 05:57:23 UTC
Copr build:
https://copr.fedorainfracloud.org/coprs/build/8994458
(failed)

Build log:
https://download.copr.fedorainfracloud.org/results/@fedora-review/fedora-review-2363932-perl-extutils-builder-compiler/fedora-rawhide-x86_64/08994458-perl-ExtUtils-Builder-Compiler/builder-live.log.gz

Please make sure the package builds successfully at least for Fedora Rawhide.

- If the build failed for unrelated reasons (e.g. temporary network
  unavailability), please ignore it.
- If the build failed because of missing BuildRequires, please make sure they
  are listed in the "Depends On" field


---
This comment was created by the fedora-review-service
https://github.com/FrostyX/fedora-review-service

If you want to trigger a new Copr build, add a comment containing new
Spec and SRPM URLs or [fedora-review-service-build] string.

Comment 2 Jitka Plesnikova 2025-08-11 13:06:48 UTC
Source file is ok
Summary is ok
License is ok
Description is ok
URL and Source0 are ok
All tests passed

BuildRequires
FIX: Add build-require 'coreutils' it is needed for %{_fixperms}
FIX: Please add build-requires
     - perl(Config) - t/40-autodetect-executable.t:8
TODO: Please add version restriction for perl(ExtUtils::MakeMaker) >= 6.76
      NO_PACKLIST is provided since this version.

$ rpm -qp --requires perl-ExtUtils-Builder-Compiler-0.031-2.fc43.noarch.rpm | sort | uniq -c | grep -v rpmlib
      1 perl(Carp)
      1 perl(ExtUtils::Builder) >= 0.016
      1 perl(ExtUtils::Builder::Action::Command)
      1 perl(ExtUtils::Builder::ArgumentCollector)
      1 perl(ExtUtils::Builder::AutoDetect::C)
      1 perl(ExtUtils::Builder::Binary)
      1 perl(ExtUtils::Builder::Compiler)
      1 perl(ExtUtils::Builder::Linker)
      1 perl(ExtUtils::Builder::Linker::COFF)
      1 perl(ExtUtils::Builder::Linker::ELF::Any)
      1 perl(ExtUtils::Builder::Linker::Unixy)
      1 perl(ExtUtils::Builder::MultiLingual)
      1 perl(ExtUtils::Builder::Node)
      1 perl(ExtUtils::Builder::Planner)
      1 perl(ExtUtils::Builder::Planner) >= 0.007
      1 perl(ExtUtils::Builder::Planner::Extension)
      1 perl(ExtUtils::Builder::Util)
      2 perl(ExtUtils::Config) >= 0.007
      2 perl(ExtUtils::Helpers) >= 0.027
      1 perl(File::Basename)
      1 perl(File::Spec::Functions)
      1 perl(File::Temp)
      1 perl-libs
      1 perl(parent)
      1 perl(Perl::OSType)
      1 perl(sort)
      1 perl(strict)
      1 perl(warnings)
FIX: Add run-require
     - perl(DynaLoader) - lib/ExtUtils/Builder/ParseXS.pm:72
FIX: Remove duplicit run-requires:
     - perl(ExtUtils::Config) >= 0.007
     - perl(ExtUtils::Helpers) >= 0.027

Otherwise binary requires are Ok.

$ rpm -qp --provides perl-ExtUtils-Builder-Compiler-0.031-2.fc43.noarch.rpm | sort | uniq -c
      1 perl(ExtUtils::Builder::ArgumentCollector) = 0.031
      1 perl(ExtUtils::Builder::AutoDetect::C) = 0.031
      1 perl(ExtUtils::Builder::AutoDetect::Cpp) = 0.031 
      1 perl(ExtUtils::Builder::Binary) = 0.031
      1 perl(ExtUtils::Builder::Compiler) = 0.031
      1 perl-ExtUtils-Builder-Compiler = 0.031-2.fc43
      1 perl(ExtUtils::Builder::Compiler::MSVC) = 0.031
      1 perl(ExtUtils::Builder::Compiler::Unixy) = 0.031
      1 perl(ExtUtils::Builder::Compiler::VMS) = 0.031
      1 perl(ExtUtils::Builder::Conf) = 0.031
      1 perl(ExtUtils::Builder::Linker) = 0.031
      1 perl(ExtUtils::Builder::Linker::Ar) = 0.031
      1 perl(ExtUtils::Builder::Linker::COFF) = 0.031
      1 perl(ExtUtils::Builder::Linker::ELF::Any) = 0.031
      1 perl(ExtUtils::Builder::Linker::ELF::GCC) = 0.031
      1 perl(ExtUtils::Builder::Linker::Mach::GCC) = 0.031
      1 perl(ExtUtils::Builder::Linker::PE::GCC) = 0.031
      1 perl(ExtUtils::Builder::Linker::PE::MSVC) = 0.031
      1 perl(ExtUtils::Builder::Linker::Unixy) = 0.031
      1 perl(ExtUtils::Builder::Linker::XCOFF) = 0.031
      1 perl(ExtUtils::Builder::MultiLingual) = 0.031
      1 perl(ExtUtils::Builder::ParseXS) = 0.031
      1 perl(ExtUtils::Builder::Profile::Perl) = 0.031
Binary provides are Ok.

$ rpmlint ./perl-ExtUtils-Builder-Compiler*
============================ rpmlint session starts ============================
rpmlint: 2.6.1
configuration:
    /usr/lib/python3.13/site-packages/rpmlint/configdefaults.toml
    /etc/xdg/rpmlint/fedora-legacy-licenses.toml
    /etc/xdg/rpmlint/fedora-spdx-licenses.toml
    /etc/xdg/rpmlint/fedora.toml
    /etc/xdg/rpmlint/scoring.toml
    /etc/xdg/rpmlint/users-groups.toml
    /etc/xdg/rpmlint/warn-on-functions.toml
checks: 32, packages: 3
 2 packages and 1 specfiles checked; 0 errors, 0 warnings, 8 filtered, 0 badness; has taken 0.2 s

Rpmlint is ok

TODO: Explicitly name packaged files in %{_mandir}/man* and %{perl_vendorlib} directories
      https://docs.fedoraproject.org/en-US/packaging-guidelines/#_explicit_lists.

Otherwise the package is in line with Fedora and Perl packaging guide lines.
Please correct all 'FIX' items, consider fixing 'TODO' items and provide a new spec file.

Resolution:
NOT Approved

Comment 3 Charles R. Anderson 2025-08-20 16:48:03 UTC
(In reply to Jitka Plesnikova from comment #2)
> FIX: Add build-require 'coreutils' it is needed for %{_fixperms}
> FIX: Please add build-requires
>      - perl(Config) - t/40-autodetect-executable.t:8
> TODO: Please add version restriction for perl(ExtUtils::MakeMaker) >= 6.76
>       NO_PACKLIST is provided since this version.
> FIX: Add run-require
>      - perl(DynaLoader) - lib/ExtUtils/Builder/ParseXS.pm:72
> FIX: Remove duplicit run-requires:
>      - perl(ExtUtils::Config) >= 0.007
>      - perl(ExtUtils::Helpers) >= 0.027
> TODO: Explicitly name packaged files in %{_mandir}/man* and
> %{perl_vendorlib} directories

The above items are all fixed:

Spec URL: https://cra.fedorapeople.org/perl-ExtUtils-Builder-Compiler/perl-ExtUtils-Builder-Compiler.spec
SRPM URL: https://cra.fedorapeople.org/perl-ExtUtils-Builder-Compiler/perl-ExtUtils-Builder-Compiler-0.031-3.fc43.src.rpm

Thanks.

Comment 4 Fedora Review Service 2025-08-20 16:48:10 UTC
There seems to be some problem with the following file.
SRPM URL: https://cra.fedorapeople.org/perl-ExtUtils-Builder-Compiler/perl-ExtUtils-Builder-Compiler-0.031-3.fc43.src.rpm
Fetching it results in a 404 Not Found error.
Please make sure the URL is correct and publicly available.


---
This comment was created by the fedora-review-service
https://github.com/FrostyX/fedora-review-service

If you want to trigger a new Copr build, add a comment containing new
Spec and SRPM URLs or [fedora-review-service-build] string.

Comment 5 Charles R. Anderson 2025-08-20 16:50:20 UTC
Corrected URLs:

Spec URL: https://cra.fedorapeople.org/perl-ExtUtils-Builder-Compiler/perl-ExtUtils-Builder-Compiler.spec
SRPM URL: https://cra.fedorapeople.org/perl-ExtUtils-Builder-Compiler/perl-ExtUtils-Builder-Compiler-0.031-3.fc44.src.rpm

(In reply to Fedora Review Service from comment #4)
> There seems to be some problem with the following file.
> SRPM URL:
> https://cra.fedorapeople.org/perl-ExtUtils-Builder-Compiler/perl-ExtUtils-
> Builder-Compiler-0.031-3.fc43.src.rpm
> Fetching it results in a 404 Not Found error.
> Please make sure the URL is correct and publicly available.

Comment 6 Fedora Review Service 2025-08-20 16:52:19 UTC
Copr build:
https://copr.fedorainfracloud.org/coprs/build/9450912
(failed)

Build log:
https://download.copr.fedorainfracloud.org/results/@fedora-review/fedora-review-2363932-perl-extutils-builder-compiler/fedora-rawhide-x86_64/09450912-perl-ExtUtils-Builder-Compiler/builder-live.log.gz

Please make sure the package builds successfully at least for Fedora Rawhide.

- If the build failed for unrelated reasons (e.g. temporary network
  unavailability), please ignore it.
- If the build failed because of missing BuildRequires, please make sure they
  are listed in the "Depends On" field


---
This comment was created by the fedora-review-service
https://github.com/FrostyX/fedora-review-service

If you want to trigger a new Copr build, add a comment containing new
Spec and SRPM URLs or [fedora-review-service-build] string.

Comment 7 Jitka Plesnikova 2025-08-21 14:19:55 UTC
(In reply to Charles R. Anderson from comment #3)
> (In reply to Jitka Plesnikova from comment #2)
> > FIX: Add build-require 'coreutils' it is needed for %{_fixperms}
> > FIX: Please add build-requires
> >      - perl(Config) - t/40-autodetect-executable.t:8
> > TODO: Please add version restriction for perl(ExtUtils::MakeMaker) >= 6.76
> >       NO_PACKLIST is provided since this version.
> > FIX: Add run-require
> >      - perl(DynaLoader) - lib/ExtUtils/Builder/ParseXS.pm:72
> > FIX: Remove duplicit run-requires:
> >      - perl(ExtUtils::Config) >= 0.007
> >      - perl(ExtUtils::Helpers) >= 0.027
> > TODO: Explicitly name packaged files in %{_mandir}/man* and
> > %{perl_vendorlib} directories
> 
> The above items are all fixed:

8a11
+ BuildRequires:  coreutils
15a19
+ BuildRequires:  perl(Config)
26c30
- BuildRequires:  perl(ExtUtils::MakeMaker)
---
+ BuildRequires:  perl(ExtUtils::MakeMaker) >= 6.76
39,40c43
- Requires:       perl(ExtUtils::Config) >= 0.007
- Requires:       perl(ExtUtils::Helpers) >= 0.027
---
+ Requires:       perl(DynaLoader)
66,67c69,71
- %{perl_vendorlib}/*
- %{_mandir}/man3/*
---
+ %dir %{perl_vendorlib}/ExtUtils
+ %{perl_vendorlib}/ExtUtils/Builder
+ %{_mandir}/man3/ExtUtils::Builder*

The changes look good.

Resolution:
Approved

Comment 8 Fedora Admin user for bugzilla script actions 2025-08-26 19:02:19 UTC
The Pagure repository was created at https://src.fedoraproject.org/rpms/perl-ExtUtils-Builder-Compiler

Comment 9 Charles R. Anderson 2025-08-27 00:55:31 UTC
FEDORA-2025-c784200a53 (perl-ExtUtils-Builder-Compiler-0.031-3.fc44) has been submitted as an update to Fedora 44.
https://bodhi.fedoraproject.org/updates/FEDORA-2025-c784200a53

Comment 10 Fedora Update System 2025-08-27 01:58:07 UTC
FEDORA-2025-77febb7f1a (perl-ExtUtils-Builder-Compiler-0.031-3.fc43) has been submitted as an update to Fedora 43.
https://bodhi.fedoraproject.org/updates/FEDORA-2025-77febb7f1a

Comment 11 Fedora Update System 2025-08-27 02:31:33 UTC
FEDORA-2025-601ac1de0d (perl-ExtUtils-Builder-Compiler-0.031-3.fc42) has been submitted as an update to Fedora 42.
https://bodhi.fedoraproject.org/updates/FEDORA-2025-601ac1de0d

Comment 12 Fedora Update System 2025-08-27 02:33:20 UTC
FEDORA-2025-a1f5789e7c (perl-ExtUtils-Builder-Compiler-0.031-3.fc41) has been submitted as an update to Fedora 41.
https://bodhi.fedoraproject.org/updates/FEDORA-2025-a1f5789e7c

Comment 13 Fedora Update System 2025-08-28 01:35:00 UTC
FEDORA-2025-77febb7f1a has been pushed to the Fedora 43 testing repository.
Soon you'll be able to install the update with the following command:
`sudo dnf install --enablerepo=updates-testing --refresh --advisory=FEDORA-2025-77febb7f1a \*`
You can provide feedback for this update here: https://bodhi.fedoraproject.org/updates/FEDORA-2025-77febb7f1a

See also https://fedoraproject.org/wiki/QA:Updates_Testing for more information on how to test updates.

Comment 14 Fedora Update System 2025-08-28 01:52:24 UTC
FEDORA-2025-601ac1de0d has been pushed to the Fedora 42 testing repository.
Soon you'll be able to install the update with the following command:
`sudo dnf install --enablerepo=updates-testing --refresh --advisory=FEDORA-2025-601ac1de0d \*`
You can provide feedback for this update here: https://bodhi.fedoraproject.org/updates/FEDORA-2025-601ac1de0d

See also https://fedoraproject.org/wiki/QA:Updates_Testing for more information on how to test updates.

Comment 15 Fedora Update System 2025-08-28 02:25:42 UTC
FEDORA-2025-a1f5789e7c has been pushed to the Fedora 41 testing repository.
Soon you'll be able to install the update with the following command:
`sudo dnf install --enablerepo=updates-testing --refresh --advisory=FEDORA-2025-a1f5789e7c \*`
You can provide feedback for this update here: https://bodhi.fedoraproject.org/updates/FEDORA-2025-a1f5789e7c

See also https://fedoraproject.org/wiki/QA:Updates_Testing for more information on how to test updates.

Comment 16 Fedora Update System 2025-09-05 02:11:26 UTC
FEDORA-2025-a1f5789e7c (perl-ExtUtils-Builder-Compiler-0.031-3.fc41) has been pushed to the Fedora 41 stable repository.
If problem still persists, please make note of it in this bug report.

Comment 17 Fedora Update System 2025-09-05 02:25:32 UTC
FEDORA-2025-601ac1de0d (perl-ExtUtils-Builder-Compiler-0.031-3.fc42) has been pushed to the Fedora 42 stable repository.
If problem still persists, please make note of it in this bug report.

Comment 18 Fedora Update System 2025-09-12 19:21:58 UTC
FEDORA-2025-77febb7f1a (perl-ExtUtils-Builder-Compiler-0.031-3.fc43) has been pushed to the Fedora 43 stable repository.
If problem still persists, 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.