Fedora Account System
Red Hat Associate
Red Hat Customer
Spec URL: https://eseyman.fedorapeople.org/perl-Parallel-Prefork/perl-Parallel-Prefork.spec SRPM URL: https://eseyman.fedorapeople.org/perl-Parallel-Prefork/perl-Parallel-Prefork-0.18-30.fc44.src.rpm Description: Parallel::Prefork is much like Parallel::ForkManager, but supports graceful shutdown and run-time reconfiguration. Fedora Account System Username: eseyman
Copr build: https://copr.fedorainfracloud.org/coprs/build/10690460 (failed) Build log: https://download.copr.fedorainfracloud.org/results/@fedora-review/fedora-review-2497527-perl-parallel-prefork/fedora-rawhide-x86_64/10690460-perl-Parallel-Prefork/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.
Source file is ok Summary is ok License is ok Description is ok URL and Source0 are ok All tests passed (after fixing the typo, see below) Package FAILED to build due to missing/wrong dependencies: No match for argument: perl(List::Utils) BuildRequires: FIX: perl(List::Utils) does not exist - it should be perl(List::Util) FIX: Add 'perl(Module::Install::Metadata)' and 'perl(Module::Install::WriteAll)' to BuildRequires - they provide 'test_requires' and 'WriteAll' used in Makefile.PL and the bundled inc/ is removed in %prep. FIX: Add 'perl-interpreter' to BuildRequires TODO: Use rather 'perl' instead '/usr/bin/perl' The package does not build. Please fix 'FIX' items, consider fixing 'TODO' item and provide a new spec file. Resolution: NOT Approved
(In reply to Jitka Plesnikova from comment #2) > > FIX: perl(List::Utils) does not exist - it should be perl(List::Util) > FIX: Add 'perl(Module::Install::Metadata)' and > 'perl(Module::Install::WriteAll)' > to BuildRequires - they provide 'test_requires' and 'WriteAll' used in > Makefile.PL and the bundled inc/ is removed in %prep. > FIX: Add 'perl-interpreter' to BuildRequires Done. > TODO: Use rather 'perl' instead '/usr/bin/perl' Not a huge fan of this. Done, nonetheless. Spec URL: https://eseyman.fedorapeople.org/perl-Parallel-Prefork/perl-Parallel-Prefork.spec SRPM URL: https://eseyman.fedorapeople.org/perl-Parallel-Prefork/perl-Parallel-Prefork-0.18-31.fc44.src.rpm
Created attachment 2148436 [details] The .spec file difference from Copr build 10690460 to 10711197
Copr build: https://copr.fedorainfracloud.org/coprs/build/10711197 (failed) Build log: https://download.copr.fedorainfracloud.org/results/@fedora-review/fedora-review-2497527-perl-parallel-prefork/fedora-rawhide-x86_64/10711197-perl-Parallel-Prefork/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.
The changes look good. Resolution: Approved
Unretired and imported in rawhide, rebuilt: https://koji.fedoraproject.org/koji/buildinfo?buildID=3034995