Bug 2067185

Summary: Review Request: perl-YAML-Parser - Generated Reference Parser for YAML 1.2
Product: [Fedora] Fedora Reporter: Jitka Plesnikova <jplesnik>
Component: Package ReviewAssignee: Michal Josef Spacek <mspacek>
Status: CLOSED RAWHIDE QA Contact: Fedora Extras Quality Assurance <extras-qa>
Severity: medium Docs Contact:
Priority: medium    
Version: rawhideCC: mspacek, package-review
Target Milestone: ---Flags: mspacek: fedora-review+
Target Release: ---   
Hardware: All   
OS: Linux   
Whiteboard:
Fixed In Version: perl-YAML-Parser-0.0.5-1.fc37 Doc Type: If docs needed, set a value
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2022-03-25 14:21:56 UTC Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:
Bug Depends On:    
Bug Blocks: 2067186    

Description Jitka Plesnikova 2022-03-23 13:29:27 UTC 
Spec URL: https://jplesnik.fedorapeople.org/perl-YAML-Parser/perl-YAML-Parser.spec
SRPM URL: https://jplesnik.fedorapeople.org/perl-YAML-Parser/perl-YAML-Parser-0.0.5-1.fc37.src.rpm
Description:
YAML::Parser is the first 100% YAML 1.2 spec compliant parser for Perl. The
Perl code is generated directly from the YAML 1.2 specification.

Fedora Account System Username: jplesnik
Comment 1 Michal Josef Spacek 2022-03-25 11:28:42 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-YAML-Parser-0.0.5-1.fc37.noarch.rpm | sort | uniq -c | grep -v rpmlib
      1 perl(base)
      1 perl(boolean)
      1 perl(Carp)
      1 perl(constant)
      1 perl(Encode)
      1 perl(Exporter)
      1 perl(JSON::PP)
      1 perl(:MODULE_COMPAT_5.34.1)
      1 perl(overload)
      1 perl(Time::HiRes)
      1 perl(:VERSION) >= 5.12.0
      1 perl(XXX)
      1 perl(YAML::PP::Perl)
Binary requires are Ok.

$ rpm -qp --provides perl-YAML-Parser-0.0.5-1.fc37.noarch.rpm |sort | uniq -c
      1 perl(PerlYamlReferenceParserFunc)
      1 perl(PerlYamlReferenceParserGrammar)
      1 perl(PerlYamlReferenceParserParser)
      1 perl(PerlYamlReferenceParserPrelude)
      1 perl(PerlYamlReferenceParserReceiver)
      1 perl(PerlYamlReferenceParserTestReceiver)
      1 perl(YAML::Parser) = 0.0.5
      1 perl-YAML-Parser = 0.0.5-1.fc37
Binary provides are Ok.

$ rpmlint ./perl-YAML-Parser*
2 packages and 1 specfiles checked; 0 errors, 0 warnings, 0 badness; has taken 0.3 s
Rpmlint is ok

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

Resolution:
I am not a sponsor of packagers. 
However for me, the package can be approved.
Comment 2 Jitka Plesnikova 2022-03-25 12:16:53 UTC 
https://pagure.io/releng/fedora-scm-requests/issue/43169
Comment 3 Gwyn Ciesla 2022-03-25 13:49:07 UTC 
(fedscm-admin):  The Pagure repository was created at https://src.fedoraproject.org/rpms/perl-YAML-Parser