[Bug 1085975] Review Request: perl-Cpanel-JSON-XS - JSON::XS for Cpanel, fast and correct serializing, also for 5.6.2

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

 



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



--- Comment #1 from David Dick <ddick@xxxxxxxx> ---
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.

-- 
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
https://admin.fedoraproject.org/mailman/listinfo/package-review





[Index of Archives]     [Fedora Legacy]     [Fedora Desktop]     [Fedora SELinux]     [Yosemite News]     [KDE Users]     [Fedora Tools]