[Bug 1096264] New: Review Request: perl-JSON-MaybeXS - Use Cpanel::JSON::XS with a fallback to JSON::XS and JSON::PP

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

 



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

            Bug ID: 1096264
           Summary: Review Request: perl-JSON-MaybeXS - Use
                    Cpanel::JSON::XS with a fallback to JSON::XS and
                    JSON::PP
           Product: Fedora
           Version: rawhide
         Component: Package Review
          Severity: medium
          Priority: medium
          Assignee: nobody@xxxxxxxxxxxxxxxxx
          Reporter: paul@xxxxxxxxxxxx
        QA Contact: extras-qa@xxxxxxxxxxxxxxxxx
                CC: package-review@xxxxxxxxxxxxxxxxxxxxxxx



Spec URL:
http://subversion.city-fan.org/repos/cfo-repo/perl-JSON-MaybeXS/branches/fedora/perl-JSON-MaybeXS.spec
SRPM URL:
http://www.city-fan.org/~paul/extras/perl-JSON-MaybeXS/perl-JSON-MaybeXS-1.002002-2.fc21.src.rpm

Description:
This module first checks to see if either Cpanel::JSON::XS or JSON::XS
is already loaded, in which case it uses that module. Otherwise it tries
to load Cpanel::JSON::XS, then JSON::XS, then JSON::PP in order, and
either uses the first module it finds or throws an error.

It then exports the "encode_json" and "decode_json" functions from the
loaded module, along with a "JSON" constant that returns the class name
for calling "new" on.

If you're writing fresh code rather than replacing JSON.pm usage, you
might want to pass options as constructor args rather than calling
mutators, so we provide our own "new" method that supports that.


Fedora Account System Username: pghmcfc

Note: this package takes the approach of adding a hard dependency on
upstream's preferred back-end, Cpanel::JSON::XS, rather than using
a virtual provides/requires arrangement so that any of the supported
back-ends could be used. This is not only much simpler and does not
involve modifications to the back-end packages, but it also makes for
consistent results as we're always using the same, most-tested
back-end.

-- 
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]