https://bugzilla.redhat.com/show_bug.cgi?id=1482867 Emmanuel Seyman <emmanuel@xxxxxxxxx> changed: What |Removed |Added ---------------------------------------------------------------------------- Blocks| |1472755 --- Comment #2 from Emmanuel Seyman <emmanuel@xxxxxxxxx> --- (In reply to Jitka Plesnikova from comment #1) > > FIX: Please remove execute permission from examples/tokenize-synopsis.pl, > it is part of documentation. Done. > BuildRequires are almost ok. > FIX: Please add perl-devel and gcc, because it is XS package Done. Spec URL: http://people.parinux.org/~seyman/fedora/perl-JSON-Parse/perl-JSON-Parse.spec SRPM URL: http://people.parinux.org/~seyman/fedora/perl-JSON-Parse/perl-JSON-Parse-0.49-2.fc26.src.rpm > Otherwise, the package looks good. > Approved. \o/ Thanks for the review, Jitka. Repo requested : https://pagure.io/releng/fedora-scm-requests/issue/130 Referenced Bugs: https://bugzilla.redhat.com/show_bug.cgi?id=1472755 [Bug 1472755] perl-JSON-Path-0.411 is available -- 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
