Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Replaced the org.json JSON parser with jackson JSON parser #190

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

ramuramaiah
Copy link

Replaced the org.json JSON parser with jackson JSON parser

Replaced the org.json JSON parser with jackson JSON parser
@ramuramaiah ramuramaiah reopened this Jun 19, 2018
@ramuramaiah
Copy link
Author

erosb,
I have created a pull request.
The change is the following
"Replaced the org.json JSON parser with jackson JSON parser"

@ramuramaiah
Copy link
Author

Hi,
I have modified the original source code of everit JSON schema parser to make use of jackson as the underlying JSON schema parser instead of org.json due to the business unfriendly license of org.json JSON parser.
I have made sure the following

The original design and the interfaces remains the same. The modification is only minimal.
All tests pass.
Let me know if I can share the changes and anyone is interested to make use of the same.
I am not sure about the copyrights etc. Hence I don't want to create a new GitHub project.

@erosb
Copy link
Contributor

erosb commented Oct 24, 2018

Hello @ramuramaiah , the library is licensed under apache 2 license, which is a very permissive license. You are free to re-distribute the library and/or use a modified version in a commercial closed-source project.

The PR is not likely to be merged as-is.

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