Skip to content

Remove simplejson dependency for Python>=2.7#3

Open
laurentb wants to merge 1 commit intozacharyvoase:masterfrom
laurentb:json
Open

Remove simplejson dependency for Python>=2.7#3
laurentb wants to merge 1 commit intozacharyvoase:masterfrom
laurentb:json

Conversation

@laurentb
Copy link
Copy Markdown
Contributor

Even though the json module is in Python 2.6, OrderedDict is only present in 2.7.

@zacharyvoase
Copy link
Copy Markdown
Owner

One question: since we're installing simplejson in the install_requires anyway (for all Python versions), is there an advantage to using the stdlib versions when they are present?

@laurentb
Copy link
Copy Markdown
Contributor Author

When using pip to install the package, not really, unless there is a way to have a conditional dependency.
However I've seen and wrote Gentoo ebuilds with that logic.

Also, for Python 3 and onwards, simplejson is only available from the stdlib.

Even though the json module is in Python 2.6, OrderedDict
is only present in 2.7.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants