Bug 2256736 - Review Request: perl-meta - Meta-programming API
Summary: Review Request: perl-meta - Meta-programming API
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/meta
Whiteboard:
Depends On:
Blocks: 2256224 2256225 2256226
TreeView+ depends on / blocked
 
Reported: 2024-01-04 09:34 UTC by Petr Pisar
Modified: 2024-02-23 01:31 UTC (History)
2 users (show)

Fixed In Version: perl-meta-0.004-1.fc40
Clone Of:
Environment:
Last Closed: 2024-02-06 16:43:21 UTC
Type: ---
Embargoed:
jplesnik: fedora-review+


Attachments (Terms of Use)
The .spec file difference from Copr build 6860547 to 6973246 (611 bytes, patch)
2024-01-30 12:18 UTC, Fedora Review Service
no flags Details | Diff

Description Petr Pisar 2024-01-04 09:34:32 UTC
Spec URL: https://ppisar.fedorapeople.org/perl-meta/perl-meta.spec
SRPM URL: https://ppisar.fedorapeople.org/perl-meta/perl-meta-0.003-1.fc40.src.rpm
Description:
This package provides an API for Perl meta programming; that is, allowing code
to inspect or manipulate parts of its own program structure. Parts of the perl
interpreter itself can be accessed by means of "meta"-objects provided by this
package. Methods on these objects allow inspection of details, as well as
creating new items or removing existing ones.

Fedora Account System Username: ppisar

Comment 1 Fedora Review Service 2024-01-04 09:42:55 UTC
Copr build:
https://copr.fedorainfracloud.org/coprs/build/6860547
(succeeded)

Review template:
https://download.copr.fedorainfracloud.org/results/@fedora-review/fedora-review-2256736-perl-meta/fedora-rawhide-x86_64/06860547-perl-meta/fedora-review/review.txt

Found issues:

- Explicit dependency on perl-devel is not allowed unless building architecture-specific code which links to libperl.so
  Read more: https://docs.fedoraproject.org/en-US/packaging-guidelines/Perl/

Please know that there can be false-positives.

---
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 Petr Pisar 2024-01-30 12:12:11 UTC
I rebased the package the latest upstream version:

Spec URL: https://ppisar.fedorapeople.org/perl-meta/perl-meta.spec
SRPM URL: https://ppisar.fedorapeople.org/perl-meta/perl-meta-0.004-1.fc40.src.rpm

Comment 3 Fedora Review Service 2024-01-30 12:18:58 UTC
Created attachment 2014027 [details]
The .spec file difference from Copr build 6860547 to 6973246

Comment 4 Fedora Review Service 2024-01-30 12:19:02 UTC
Copr build:
https://copr.fedorainfracloud.org/coprs/build/6973246
(succeeded)

Review template:
https://download.copr.fedorainfracloud.org/results/@fedora-review/fedora-review-2256736-perl-meta/fedora-rawhide-x86_64/06973246-perl-meta/fedora-review/review.txt

Found issues:

- Explicit dependency on perl-devel is not allowed unless building architecture-specific code which links to libperl.so
  Read more: https://docs.fedoraproject.org/en-US/packaging-guidelines/Perl/

Please know that there can be false-positives.

---
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 Jitka Plesnikova 2024-02-06 12:19:26 UTC
Source file is ok
Summary is ok
License is ok
Description is ok
URL and Source0 are ok
All tests passed
BuildRequires are ok

$ rpm -qp --requires perl-meta-0.004-1.fc40.x86_64.rpm | sort | uniq -c | grep -v rpmlib
      1 perl-libs
      1 perl(:MODULE_COMPAT_5.38.2)
      1 perl(:VERSION) >= 5.14.0
      1 perl(warnings)
      1 perl(XSLoader)
      1 rtld(GNU_HASH)
Binary requires are Ok.

$ rpm -qp --provides perl-meta-0.004-1.fc40.x86_64.rpm | sort | uniq -c
      1 perl(meta) = 0.004
      1 perl-meta = 0.004-1.fc40
      1 perl-meta(x86-64) = 0.004-1.fc40
Binary provides are Ok.

$ rpmlint ./perl-meta*
==================================== rpmlint session starts ===================================
rpmlint: 2.4.0
configuration:
    /usr/lib/python3.11/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: 31, packages: 4
perl-meta.x86_64: E: shared-library-without-dependency-information /usr/lib64/perl5/vendor_perl/auto/meta/meta.so
perl-meta-tests.noarch: W: no-documentation
===== 3 packages and 1 specfiles checked; 1 errors, 1 warnings, 1 badness; has taken 0.3 s ====
Rpmlint is ok

TODO: Please consider to use macro style %{optflags} for optimization flags
      as for build root.

The package is in line with Fedora and Perl packaging guide lines.

Resolution:
Approved

Comment 6 Fedora Admin user for bugzilla script actions 2024-02-06 13:52:03 UTC
The Pagure repository was created at https://src.fedoraproject.org/rpms/perl-meta

Comment 7 Petr Pisar 2024-02-06 16:22:08 UTC
Thanks for the review. I replaced the variable with the macro.

Comment 8 Fedora Update System 2024-02-14 12:17:04 UTC
FEDORA-2024-513c1d2a8a (perl-meta-0.004-1.fc39, perl-Syntax-Operator-Divides-0.07-1.fc39, and 2 more) has been submitted as an update to Fedora 39.
https://bodhi.fedoraproject.org/updates/FEDORA-2024-513c1d2a8a

Comment 9 Fedora Update System 2024-02-14 12:18:28 UTC
FEDORA-2024-c4d455ce90 (perl-meta-0.004-1.fc38, perl-Syntax-Operator-Divides-0.07-1.fc38, and 2 more) has been submitted as an update to Fedora 38.
https://bodhi.fedoraproject.org/updates/FEDORA-2024-c4d455ce90

Comment 10 Fedora Update System 2024-02-15 01:37:04 UTC
FEDORA-2024-513c1d2a8a has been pushed to the Fedora 39 testing repository.
Soon you'll be able to install the update with the following command:
`sudo dnf upgrade --enablerepo=updates-testing --refresh --advisory=FEDORA-2024-513c1d2a8a`
You can provide feedback for this update here: https://bodhi.fedoraproject.org/updates/FEDORA-2024-513c1d2a8a

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

Comment 11 Fedora Update System 2024-02-15 01:52:47 UTC
FEDORA-2024-c4d455ce90 has been pushed to the Fedora 38 testing repository.
Soon you'll be able to install the update with the following command:
`sudo dnf upgrade --enablerepo=updates-testing --refresh --advisory=FEDORA-2024-c4d455ce90`
You can provide feedback for this update here: https://bodhi.fedoraproject.org/updates/FEDORA-2024-c4d455ce90

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

Comment 12 Fedora Update System 2024-02-23 01:23:32 UTC
FEDORA-2024-513c1d2a8a (perl-meta-0.004-1.fc39, perl-Syntax-Operator-Divides-0.07-1.fc39, and 2 more) has been pushed to the Fedora 39 stable repository.
If problem still persists, please make note of it in this bug report.

Comment 13 Fedora Update System 2024-02-23 01:31:50 UTC
FEDORA-2024-c4d455ce90 (perl-meta-0.004-1.fc38, perl-Syntax-Operator-Divides-0.07-1.fc38, and 2 more) has been pushed to the Fedora 38 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.