https://bugzilla.redhat.com/show_bug.cgi?id=1818936 --- Comment #1 from Petr Pisar <ppisar@xxxxxxxxxx> --- URL and Source0 addresses are usable. Ok. Source0 archive (SHA-256: e828a3256bb2217abd1b947c0cad145e09a5fc8e858aade3b04792d4b8a6a5b4) is original. Ok. Summary verified from lib/Mojolicious/Plugin/OpenAPI.pm. Ok. Description verified from lib/Mojolicious/Plugin/OpenAPI.pm. Ok. License verified from Makefile.PL, lib/Mojolicious/Plugin/OpenAPI.pm, README.md. Ok. MIT-licensed files not in any binary package: t/v3-style-array.t, t/v3-basic.t, t/spec/v2-petstore.json, t/v3-nullable.t. ASL 2.0-licensed files no in any binary package: t/spec/bundlecheck.json. TODO: Please, put a comment into the spec file above the License tag reading that these files have a different license. No XS code, noarch BuildArch is Ok. FIX: Build-require 'perl-generators'. You package does does not list all Perl run-time dependencies and it relies on the generators. FIX: Build-require 'perl-interpreter' (perl-Mojolicious-Plugin-OpenAPI.spec:51). FIX: Build-require 'make' (perl-Mojolicious-Plugin-OpenAPI.spec:52). The macro is provided by rpm that does not depend on make. TODO: Constrain 'perl(Mojolicious)' dependency with '>= 8.00' (META.json:34) if you believe that the metadata are correct. TODO: Unset DUMMY_DB_ERROR, JSON_VALIDATOR_DEBUG, and MOJO_OPENAPI_DEBUG environment variables before executing "make check". They are used through-out the code and could influence the tests. TODO: Suggest 'perl(Text::Markdown)' (lib/Mojolicious/Plugin/OpenAPI/SpecRenderer.pm:8). TODO: Recommend 'perl(Config)' (lib/JSON/Validator/OpenAPI/Mojolicious.pm:10). Test::Pod, Test::Pod::Coverage, Test::CPAN::Changes are optional. Ok. TODO: Build-require 'perl(Text::Markdown)' for optional tests (t/plugin-spec-renderer-standalone.t:43). All tests pass. Ok. TODO: There are suspicous warnings printed when running the tests. It would be greate to report them to the upstream: Format rule for 'unknown' is missing at /usr/share/perl5/vendor_perl/JSON/Validator.pm line 750. Format rule for 'binary' is missing at /usr/share/perl5/vendor_perl/JSON/Validator.pm line 750. Use of uninitialized value in string eq at /home/test/rpmbuild/BUILD/Mojolicious-Plugin-OpenAPI-3.30/blib/lib/JSON/Validator/OpenAPI/Mojolicious.pm line 58. Use of uninitialized value in string eq at /home/test/rpmbuild/BUILD/Mojolicious-Plugin-OpenAPI-3.30/blib/lib/JSON/Validator/OpenAPI/Mojolicious.pm line 58. $ rpmlint perl-Mojolicious-Plugin-OpenAPI.spec ../SRPMS/perl-Mojolicious-Plugin-OpenAPI-3.30-1.fc33.src.rpm ../RPMS/noarch/perl-Mojolicious-Plugin-OpenAPI-3.30-1.fc33.noarch.rpm sh: /usr/bin/python2: No such file or directory 2 packages and 1 specfiles checked; 0 errors, 0 warnings. rpmlint is OK. $ rpm -q -lv -p ../RPMS/noarch/perl-Mojolicious-Plugin-OpenAPI-3.30-1.fc33.noarch.rpm drwxr-xr-x 2 root root 0 Apr 8 16:54 /usr/share/doc/perl-Mojolicious-Plugin-OpenAPI -rw-r--r-- 1 root root 12242 Mar 24 02:24 /usr/share/doc/perl-Mojolicious-Plugin-OpenAPI/Changes -rw-r--r-- 1 root root 10945 Mar 24 02:24 /usr/share/doc/perl-Mojolicious-Plugin-OpenAPI/README.md -rw-r--r-- 1 root root 3104 Apr 8 16:53 /usr/share/man/man3/JSON::Validator::OpenAPI::Mojolicious.3pm.gz -rw-r--r-- 1 root root 4848 Apr 8 16:53 /usr/share/man/man3/Mojolicious::Plugin::OpenAPI.3pm.gz -rw-r--r-- 1 root root 4053 Apr 8 16:53 /usr/share/man/man3/Mojolicious::Plugin::OpenAPI::Cors.3pm.gz -rw-r--r-- 1 root root 4334 Apr 8 16:53 /usr/share/man/man3/Mojolicious::Plugin::OpenAPI::Guides::OpenAPIv2.3pm.gz -rw-r--r-- 1 root root 4908 Apr 8 16:53 /usr/share/man/man3/Mojolicious::Plugin::OpenAPI::Guides::OpenAPIv3.3pm.gz -rw-r--r-- 1 root root 1915 Apr 8 16:53 /usr/share/man/man3/Mojolicious::Plugin::OpenAPI::Guides::Swagger2.3pm.gz -rw-r--r-- 1 root root 2712 Apr 8 16:53 /usr/share/man/man3/Mojolicious::Plugin::OpenAPI::Security.3pm.gz -rw-r--r-- 1 root root 2935 Apr 8 16:53 /usr/share/man/man3/Mojolicious::Plugin::OpenAPI::SpecRenderer.3pm.gz drwxr-xr-x 2 root root 0 Apr 8 16:53 /usr/share/perl5/vendor_perl/JSON drwxr-xr-x 2 root root 0 Apr 8 16:53 /usr/share/perl5/vendor_perl/JSON/Validator drwxr-xr-x 2 root root 0 Apr 8 16:53 /usr/share/perl5/vendor_perl/JSON/Validator/OpenAPI -rw-r--r-- 1 root root 20604 Mar 24 02:23 /usr/share/perl5/vendor_perl/JSON/Validator/OpenAPI/Mojolicious.pm drwxr-xr-x 2 root root 0 Apr 8 16:53 /usr/share/perl5/vendor_perl/Mojolicious drwxr-xr-x 2 root root 0 Apr 8 16:53 /usr/share/perl5/vendor_perl/Mojolicious/Plugin drwxr-xr-x 2 root root 0 Apr 8 16:53 /usr/share/perl5/vendor_perl/Mojolicious/Plugin/OpenAPI -rw-r--r-- 1 root root 20868 Mar 24 02:24 /usr/share/perl5/vendor_perl/Mojolicious/Plugin/OpenAPI.pm -rw-r--r-- 1 root root 14203 Mar 6 01:01 /usr/share/perl5/vendor_perl/Mojolicious/Plugin/OpenAPI/Cors.pm drwxr-xr-x 2 root root 0 Apr 8 16:53 /usr/share/perl5/vendor_perl/Mojolicious/Plugin/OpenAPI/Guides -rw-r--r-- 1 root root 8423 Mar 3 07:53 /usr/share/perl5/vendor_perl/Mojolicious/Plugin/OpenAPI/Guides/OpenAPIv2.pod -rw-r--r-- 1 root root 10470 Mar 3 07:53 /usr/share/perl5/vendor_perl/Mojolicious/Plugin/OpenAPI/Guides/OpenAPIv3.pod -rw-r--r-- 1 root root 1782 Mar 3 07:53 /usr/share/perl5/vendor_perl/Mojolicious/Plugin/OpenAPI/Guides/Swagger2.pod -rw-r--r-- 1 root root 6974 Mar 6 01:01 /usr/share/perl5/vendor_perl/Mojolicious/Plugin/OpenAPI/Security.pm -rw-r--r-- 1 root root 34282 Mar 9 07:41 /usr/share/perl5/vendor_perl/Mojolicious/Plugin/OpenAPI/SpecRenderer.pm File permissions and layout are Ok. $ rpm -q --requires -p ../RPMS/noarch/perl-Mojolicious-Plugin-OpenAPI-3.30-1.fc33.noarch.rpm | sort -f | uniq -c 1 perl(:MODULE_COMPAT_5.30.2) 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 FIX: Build-requiere 'perl-generators'. FIX: Run-require 'perl(JSON::Validator) >= 3.16' explicitly for the versinon constrain. $ rpm -q --provides -p ../RPMS/noarch/perl-Mojolicious-Plugin-OpenAPI-3.30-1.fc33.noarch.rpm | sort -f | uniq -c 1 perl-Mojolicious-Plugin-OpenAPI = 3.30-1.fc33 FIX: Build-require 'perl-generators'. $ resolvedeps rawhide ../RPMS/noarch/perl-Mojolicious-Plugin-OpenAPI-3.30-1.fc33.noarch.rpm Binary dependencies are resolvable. Ok. 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: Package NOT approved. The package builds in Fedora 33 (https://koji.fedoraproject.org/koji/taskinfo?taskID=43131004). Ok. -- 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 _______________________________________________ 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
