Bug 1130098 - Review Request: python-ejson - Extensible json serializer/deserializer library
Summary: Review Request: python-ejson - Extensible json serializer/deserializer library
Keywords:
Status: CLOSED NOTABUG
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Nobody's working on this, feel free to take it
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks: FE-DEADREVIEW 1130103
TreeView+ depends on / blocked
 
Reported: 2014-08-14 10:32 UTC by Pavel Alexeev
Modified: 2023-09-14 02:45 UTC (History)
3 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2019-01-16 01:32:30 UTC
Type: ---
Embargoed:


Attachments (Terms of Use)

Description Pavel Alexeev 2014-08-14 10:32:23 UTC
Spec URL: https://raw.githubusercontent.com/Hubbitus/Fedora-packaging/master/SPECS/python-ejson.spec
SRPM URL: http://hubbitus.info/rpm/Fedora20/python-ejson/python-ejson-0.1.6-1.fc20.src.rpm
Description: There are countless scenarios that we need to exchange data between
different systems, implemented in different languages and technologies.
Even in the same system, when implementing data exchange between the
backend and the frontend we face the need to convert the language data
types to another format and then do the oposite when the data arrives in
the other side of the wire.

A very simple and flexible format that seems to fit most of our needs is
the JavaScript Object Notation, or simple `json`. It is very hard to
find a programming language these days that does not support it, even
the low level ones, like C, C++, etc.

Json is enough when we need to exchange data types like integers,
doubles, strings, lists and hash tables. The problem starts when we need
to exchange a complex data type. And it's the exact aim of this
document: providing an API to extend the `json` library to make it easy
to register new serializers and new deserializers.

Fedora Account System Username: Hubbitus

Comment 1 Igor Gnatenko 2016-07-18 18:29:34 UTC
unfortunately it is completely out of sync with guidelines.

* must be split to python2- and python3- subpackages
* must use %python_provide macro for them
* should use new macroses %py2_build, %py3_build, %py2_install, %py3_install
* must not use unversioned %{python_sitelib}
* should drop %{!?python_sitelib: %global python_sitelib %(%{__python} -c "from distutils.sysconfig import get_python_lib; print(get_python_lib())")} and sitearch macroses like that (it's needed only for EL5)

Comment 2 Germano Massullo 2016-07-18 18:30:44 UTC
Hello. Your spec file is not compliant to Fedora Python Packaging Guidelines
https://fedoraproject.org/wiki/Packaging:Python#Example_common_spec_file
for instance, there is not distinction between Python 2 and 3 packages.
You can take as example my spec file of python-netdiff
http://pkgs.fedoraproject.org/cgit/rpms/python-netdiff.git/tree/python-netdiff.spec

Comment 3 Igor Gnatenko 2016-11-14 17:11:43 UTC
ping?

Comment 4 Elliott Sales de Andrade 2019-01-16 01:32:30 UTC
I think it's time to close this.

Comment 5 Red Hat Bugzilla 2023-09-14 02:45:41 UTC
The needinfo request[s] on this closed bug have been removed as they have been unresolved for 1000 days


Note You need to log in before you can comment on or make changes to this bug.