[Bug 2277730] Review Request: perl-Data-Fake - Declaratively generate fake structured data for testing

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



https://bugzilla.redhat.com/show_bug.cgi?id=2277730

Petr Pisar <ppisar@xxxxxxxxxx> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
              Flags|fedora-review?              |fedora-review+



--- Comment #2 from Petr Pisar <ppisar@xxxxxxxxxx> ---
Standalone spec file and packages spec file differ. I will use the standalone
one because it's newer.

URL and Source0 addresses are Ok.
Source0 archive (SHA-512
1d36a9f66c0cd90ae93200752b2e2786df02d10fbe61be45464edadb6bae5ac157830c0b56bf2943e59a9949b82e8e96f1351258a48ef6f12535cdc89f20d171)
is original. Ok.
Summary verified from lib/Data/Fake.pm. OK.
Description verified from lib/Data/Fake.pm. OK.
License verified from LICENSE, t/core.t, t/text.t, t/internet.t, t/api.t,
t/names.t, t/dates.t, t/company.t, README, lib/Data/Fake.pm,
lib/Data/Fake/Examples.pod, lib/Data/Fake/Company.pm,
lib/Data/Fake/Internet.pm, lib/Data/Fake/Core.pm, lib/Data/Fake/Names.pm,
lib/Data/Fake/Text.pm, lib/Data/Fake/Dates.pm, Makefile.PL, and dist.ini. Ok.
No XS code, noarch BuildArch is Ok.

TODO: Unset AUTHOR_TESTING environment variable in %check phase as it
influences t/00-report-prereqs.t.

All tests pass. Ok.

$ rpmlint perl-Data-Fake.spec ../SRPMS/perl-Data-Fake-0.006-1.fc41.src.rpm
../RPMS/noarch/perl-Data-Fake-0.006-1.fc41.noarch.rpm 
======================================== rpmlint session starts
=======================================
rpmlint: 2.5.0
configuration:
    /usr/lib/python3.12/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

perl-Data-Fake.noarch: E: spelling-error ('Declaratively', 'Summary(en_US)
Declaratively -> Declarative, Decoratively, Attractively')
perl-Data-Fake.src: E: spelling-error ('Declaratively', 'Summary(en_US)
Declaratively -> Declarative, Decoratively, Attractively')
=== 2 packages and 1 specfiles checked; 2 errors, 0 warnings, 6 filtered, 2
badness; has taken 0.4 s ==

rpmlint is Ok.


$ rpm -q -lv -p ../RPMS/noarch/perl-Data-Fake-0.006-1.fc41.noarch.rpm 
drwxr-xr-x    2 root     root                        0 Apr 29 02:00
/usr/share/doc/perl-Data-Fake
-rw-r--r--    1 root     root                     3460 Feb 26  2021
/usr/share/doc/perl-Data-Fake/CONTRIBUTING.mkdn
-rw-r--r--    1 root     root                      983 Feb 26  2021
/usr/share/doc/perl-Data-Fake/Changes
-rw-r--r--    1 root     root                     6763 Feb 26  2021
/usr/share/doc/perl-Data-Fake/README
drwxr-xr-x    2 root     root                        0 Apr 29 02:00
/usr/share/doc/perl-Data-Fake/examples
-rw-r--r--    1 root     root                      455 Feb 26  2021
/usr/share/doc/perl-Data-Fake/examples/synopsis.pl
-rw-r--r--    1 root     root                      279 Feb 26  2021
/usr/share/doc/perl-Data-Fake/examples/urls.pl
drwxr-xr-x    2 root     root                        0 Apr 29 02:00
/usr/share/licenses/perl-Data-Fake
-rw-r--r--    1 root     root                    11500 Feb 26  2021
/usr/share/licenses/perl-Data-Fake/LICENSE
-rw-r--r--    1 root     root                     3461 Apr 29 02:00
/usr/share/man/man3/Data::Fake.3pm.gz
-rw-r--r--    1 root     root                     1297 Apr 29 02:00
/usr/share/man/man3/Data::Fake::Company.3pm.gz
-rw-r--r--    1 root     root                     3258 Apr 29 02:00
/usr/share/man/man3/Data::Fake::Core.3pm.gz
-rw-r--r--    1 root     root                     1599 Apr 29 02:00
/usr/share/man/man3/Data::Fake::Dates.3pm.gz
-rw-r--r--    1 root     root                     1252 Apr 29 02:00
/usr/share/man/man3/Data::Fake::Examples.3pm.gz
-rw-r--r--    1 root     root                     1393 Apr 29 02:00
/usr/share/man/man3/Data::Fake::Internet.3pm.gz
-rw-r--r--    1 root     root                     1402 Apr 29 02:00
/usr/share/man/man3/Data::Fake::Names.3pm.gz
-rw-r--r--    1 root     root                     1470 Apr 29 02:00
/usr/share/man/man3/Data::Fake::Text.3pm.gz
drwxr-xr-x    2 root     root                        0 Apr 29 02:00
/usr/share/perl5/vendor_perl/Data
drwxr-xr-x    2 root     root                        0 Apr 29 02:00
/usr/share/perl5/vendor_perl/Data/Fake
-rw-r--r--    1 root     root                     7111 Feb 26  2021
/usr/share/perl5/vendor_perl/Data/Fake.pm
-rw-r--r--    1 root     root                     4890 Feb 26  2021
/usr/share/perl5/vendor_perl/Data/Fake/Company.pm
-rw-r--r--    1 root     root                    17523 Feb 26  2021
/usr/share/perl5/vendor_perl/Data/Fake/Core.pm
-rw-r--r--    1 root     root                     4643 Feb 26  2021
/usr/share/perl5/vendor_perl/Data/Fake/Dates.pm
-rw-r--r--    1 root     root                      876 Feb 26  2021
/usr/share/perl5/vendor_perl/Data/Fake/Examples.pod
-rw-r--r--    1 root     root                     3783 Feb 26  2021
/usr/share/perl5/vendor_perl/Data/Fake/Internet.pm
-rw-r--r--    1 root     root                    24630 Feb 26  2021
/usr/share/perl5/vendor_perl/Data/Fake/Names.pm
-rw-r--r--    1 root     root                     4411 Feb 26  2021
/usr/share/perl5/vendor_perl/Data/Fake/Text.pm
TODO: I recommend not packaging CONTRIBUTING.mkdn as it does not provide
information helpful for the packaged files (the documented procedure only works
if you have unpacked the source archive) and it isn't specific to this piece of
software.

$ rpm -q --requires -p ../RPMS/noarch/perl-Data-Fake-0.006-1.fc41.noarch.rpm |
sort -f | uniq -c
      1 perl(:VERSION) >= 5.8.1
      1 perl(Carp)
      1 perl(Data::Fake::Core)
      1 perl(Data::Fake::Names)
      1 perl(Data::Fake::Text)
      1 perl(Exporter) >= 5.57
      1 perl(Import::Into) >= 1.002005
      1 perl(List::Util)
      1 perl(strict)
      1 perl(Text::Lorem)
      1 perl(Time::Piece) >= 1.27
      1 perl(warnings)
      1 perl-libs
      1 rpmlib(CompressedFileNames) <= 3.0.4-1
      1 rpmlib(FileDigests) <= 4.6.0-1
      1 rpmlib(PayloadFilesHavePrefix) <= 4.0-1
      1 rpmlib(PayloadIsZstd) <= 5.4.18-1
Binary requires are Ok.

$ rpm -q --provides -p ../RPMS/noarch/perl-Data-Fake-0.006-1.fc41.noarch.rpm |
sort -f | uniq -c
      1 perl(Data::Fake) = 0.006
      1 perl(Data::Fake::Company) = 0.006
      1 perl(Data::Fake::Core) = 0.006
      1 perl(Data::Fake::Dates) = 0.006
      1 perl(Data::Fake::Internet) = 0.006
      1 perl(Data::Fake::Names) = 0.006
      1 perl(Data::Fake::Text) = 0.006
      1 perl-Data-Fake = 0.006-1.fc41
Binary provides are Ok.

$ resolvedeps rawhide ../RPMS/noarch/perl-Data-Fake-0.006-1.fc41.noarch.rpm 
Binary dependencies are resolvable. Ok.

The package builds in Fedora 41
(https://koji.fedoraproject.org/koji/taskinfo?taskID=117152051). Ok.

The package is in line with Fedora and Perl packaging guidelines.
Please consider fixing the TODO items before building this package.
Resolution: Package APPROVED.


-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
https://bugzilla.redhat.com/show_bug.cgi?id=2277730

Report this comment as SPAM: https://bugzilla.redhat.com/enter_bug.cgi?product=Bugzilla&format=report-spam&short_desc=Report%20of%20Bug%202277730%23c2
--
_______________________________________________
package-review mailing list -- package-review@xxxxxxxxxxxxxxxxxxxxxxx
To unsubscribe send an email to package-review-leave@xxxxxxxxxxxxxxxxxxxxxxx
Fedora Code of Conduct: https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: https://lists.fedoraproject.org/archives/list/package-review@xxxxxxxxxxxxxxxxxxxxxxx
Do not reply to spam, report it: https://pagure.io/fedora-infrastructure/new_issue




[Index of Archives]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Yosemite Conditions]     [KDE Users]

  Powered by Linux