Spec URL: http://subversion.city-fan.org/repos/cfo-repo/perl-Cpanel-JSON-XS/branches/fedora/perl-Cpanel-JSON-XS.spec SRPM URL: http://www.city-fan.org/~paul/extras/perl-Cpanel-JSON-XS/perl-Cpanel-JSON-XS-2.3404-1.fc21.src.rpm Description: This module converts Perl data structures to JSON and vice versa. Its primary goal is to be correct and its secondary goal is to be fast. To reach the latter goal it was written in C. Fedora Account System Username: pghmcfc
FIX: BR perl(Carp) # line 134 of XS.pm BR perl(Hash::Util) # line 13 of t/12_blessed.t Req perl(Carp) # line 134 of XS.pm Req perl(Data::Dump) # line 212 of bin/cpanel_json_xs Req perl(YAML) # line 180 of bin/cpanel_json_xs Req perl(Compress::LZF) # line 179 of bin/cpanel_json_xs Req perl(Convert::Bencode) # line 178 of bin/cpanel_json_xs Given that 1) i'm trying to package Compress::LZF at https://bugzilla.redhat.com/show_bug.cgi?id=1074129 2) i'm stalled b/c of missing files in liblzf at https://bugzilla.redhat.com/show_bug.cgi?id=1075911 3) Compress::LZF is an optional argument in a "test" command line client that will handle itself correctly if the "lzf" option is removed i don't mind if you resubmit with a patch to just remove the lzf option. Example patch attached. Or block on #1074129 and wait with me :) TODO: Maybe remove ", also for 5.6.2" from the summary? perl 5.6.2 was a while ago.
Created attachment 884806 [details] Proposed removal of lzf compression option from command line
All issues addressed in -2: Spec URL: http://subversion.city-fan.org/repos/cfo-repo/perl-Cpanel-JSON-XS/branches/fedora/perl-Cpanel-JSON-XS.spec SRPM URL: http://www.city-fan.org/~paul/extras/perl-Cpanel-JSON-XS/perl-Cpanel-JSON-XS-2.3404-2.fc21.src.rpm When you eventually get Compress::LZF approved, do you plan to do EL-6/7 branches? If not, I would like to take them.
My plan is to do EL6 and EL7.
I've checked the git logs for liblzf and Steve Traylen (owner of liblzf) has only added lzfP.h to the package. Therefore, i have to strip out all the _best functionality of Compress::LZF as noted in that bug. I'm doing that at the moment and hope to have the Compress::LZF ready for re-submission soon.
At any rate, package APPROVED. I'll let you know when Compress::LZF is ready.
New Package SCM Request ======================= Package Name: perl-Cpanel-JSON-XS Short Description: JSON::XS for Cpanel, fast and correct serializing Owners: pghmcfc Branches: f19 f20 el6 epel7 InitialCC: perl-sig Thanks for the review David,
Git done (by process-git-requests).
perl-Cpanel-JSON-XS-2.3404-2.el6 has been submitted as an update for Fedora EPEL 6. https://admin.fedoraproject.org/updates/perl-Cpanel-JSON-XS-2.3404-2.el6
perl-Cpanel-JSON-XS-2.3404-2.fc19 has been submitted as an update for Fedora 19. https://admin.fedoraproject.org/updates/perl-Cpanel-JSON-XS-2.3404-2.fc19
perl-Cpanel-JSON-XS-2.3404-2.fc20 has been submitted as an update for Fedora 20. https://admin.fedoraproject.org/updates/perl-Cpanel-JSON-XS-2.3404-2.fc20
perl-Cpanel-JSON-XS-2.3404-2.fc20 has been pushed to the Fedora 20 stable repository.
perl-Cpanel-JSON-XS-2.3404-2.fc19 has been pushed to the Fedora 19 stable repository.
perl-Cpanel-JSON-XS-2.3404-2.el6 has been pushed to the Fedora EPEL 6 stable repository.
*** Bug 1526170 has been marked as a duplicate of this bug. ***