Bug 2097813 - Review Request: perl-MooseX-Extended - Extend Moose with safe defaults and useful features
Summary: Review Request: perl-MooseX-Extended - Extend Moose with safe defaults and u...
Keywords:
Status: CLOSED RAWHIDE
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Michal Josef Spacek
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On: 2097746 2097799
Blocks:
TreeView+ depends on / blocked
 
Reported: 2022-06-16 15:42 UTC by Ralf Corsepius
Modified: 2023-08-28 13:37 UTC (History)
3 users (show)

Fixed In Version: perl-MooseX-Extended-0.35-1.fc40 perl-MooseX-Extended-0.35-1.fc39
Doc Type: If docs needed, set a value
Doc Text:
Clone Of:
Environment:
Last Closed: 2023-08-28 13:37:58 UTC
Type: ---
Embargoed:
mspacek: fedora-review+


Attachments (Terms of Use)

Description Ralf Corsepius 2022-06-16 15:42:32 UTC
Spec URL: https://corsepiu.fedorapeople.org/packages/perl-MooseX-Extended.spec
SRPM URL: https://corsepiu.fedorapeople.org/packages/perl-MooseX-Extended-0.24-1.fc37.src.rpm
Description: 
This class attempts to create a safer version of Moose that defaults to
read-only attributes and is easier to read and write.

Fedora Account System Username: corsepiu

Comment 1 Ralf Corsepius 2022-06-16 15:48:32 UTC
This package is a new optional BR: of perl-Sub-HandlesVia.

Comment 5 Michal Josef Spacek 2023-05-22 15:56:56 UTC
There are issues:
* Remove COMPAT line (with coreutils dependency, which is used for echo in COMPAT line)

Optional:
* I am for adding comments like "Runtime:" and "Tests:" to BuildRequire list as in other Perl modules

Comment 6 Michal Josef Spacek 2023-05-22 16:08:33 UTC
Sorry coreutils is needed for %{_fixperms}

Comment 7 Ralf Corsepius 2023-06-26 16:13:46 UTC
(In reply to Michal Josef Spacek from comment #6)
> Sorry coreutils is needed for %{_fixperms}
Correct. Misinterpretations such as yours are the result of RHAT's ban of file deps. Incidents like this demonstrate this ban's harmfulness.

Updated package:
Spec URL: https://corsepiu.fedorapeople.org/packages/perl-MooseX-Extended.spec
SRPM URL: https://corsepiu.fedorapeople.org/packages/perl-MooseX-Extended-0.35-1.fc39.src.rpm

Comment 8 Michal Josef Spacek 2023-06-30 12:11:53 UTC
Hi Ralf,

There is one missing thing with not explicitly named files in spec file.
Need to name files in %{perl_vendorlib}/ and %{_mandir}/man3/
See https://docs.fedoraproject.org/en-US/packaging-guidelines/#_explicit_lists

Other seems ok.

Comment 9 Ralf Corsepius 2023-07-15 16:00:25 UTC
(In reply to Michal Josef Spacek from comment #8)
> Hi Ralf,
> 
> There is one missing thing with not explicitly named files in spec file.
> Need to name files in %{perl_vendorlib}/ and %{_mandir}/man3/
Hundreds of perl-packages use %{perl_vendorlib}/* and %{_mandir}/man3/*

I refuse to apply your suggestion.

Comment 10 Michal Josef Spacek 2023-07-27 12:18:24 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-MooseX-Extended-0.35-1.fc39.noarch.rpm | sort | uniq -c | grep -v rpmlib
      1 perl(B::Hooks::AtRuntime)
      1 perl(Carp)
      1 perl(constant)
      1 perl(Exporter)
      1 perl(feature)
      1 perl(Import::Into)
      1 perl-libs
      1 perl(Module::Load)
      1 perl(Moose)
      1 perl(Moose::Exporter)
      1 perl(Moose::Meta::Role)
      1 perl(Moose::Role)
      1 perl(Moose::Util)
      1 perl(MooseX::Extended)
      1 perl(MooseX::Extended::Core)
      1 perl(MooseX::Extended::Role)
      1 perl(MooseX::Extended::Types)
      1 perl(MooseX::Role::WarnOnConflict)
      1 perl(MooseX::StrictConstructor)
      1 perl(mro)
      1 perl(namespace::autoclean)
      1 perl(parent)
      1 perl(Ref::Util)
      1 perl(Storable)
      1 perl(strict)
      1 perl(true)
      1 perl(Type::Library)
      1 perl(Type::Params)
      1 perl(Types::Standard)
      1 perl(Type::Utils)
      1 perl(:VERSION) >= 5.20.0
      1 perl(warnings)

Binary requires are Ok.

$ rpm -qp --provides perl-MooseX-Extended-0.35-1.fc39.noarch.rpm | sort | uniq -c
      1 perl(Moose::Exception::InvalidAttributeDefinition) = 0.35
      1 perl(Moose::Exception::InvalidImportList) = 0.35
      1 perl(MooseX::Extended::Core) = 0.35
      1 perl(MooseX::Extended::Custom) = 0.35
      1 perl(MooseX::Extended::Role::Custom) = 0.35
      1 perl(MooseX::Extended::Role) = 0.35
      1 perl(MooseX::Extended::Types) = 0.35
      1 perl(MooseX::Extended) = 0.35
      1 perl-MooseX-Extended = 0.35-1.fc39

Binary provides are Ok.

$ rpmlint perl-MooseX-Extended-0.35-1.fc39.noarch.rpm
1 packages and 0 specfiles checked; 0 errors, 0 warnings, 0 badness; has taken 0.0 s
Rpmlint is ok

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

Resolution:
The package can be approved.

Comment 11 Fedora Admin user for bugzilla script actions 2023-08-23 15:32:06 UTC
The Pagure repository was created at https://src.fedoraproject.org/rpms/perl-MooseX-Extended


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