org.json library

[Johannes]

Hi,
Since Jedis 4.0, the library org.json is used. Is it planned to continue using this library? The reason I ask is that its license file is not accepted by our FOSS guidelines. 

Is there a chance to replace it with a different library? 

Thank you.

[Sazzadul Hoque]

Hi,
Thanks for the suggestion. This is the first time this topic has come up, so we haven't had this request until now.

But, I'd love to look over a PR if you're interested in contributing one!
But org.json looks very similar to an MIT License.

--
You received this message because you are subscribed to the Google Groups "Jedis" group.
To unsubscribe from this group and stop receiving emails from it, send an email to jedis_redis...@googlegroups.com.
To view this discussion on the web visit https://groups.google.com/d/msgid/jedis_redis/13428326-8bb3-42bc-bb29-fda11195b98bn%40googlegroups.com.

[Johannes]

Thanks for your reply.

The license text is similar but from a legal perspective, it appears not to be ok.The following page provides some more details on this:

https://github.com/fabric8io/docker-maven-plugin/issues/1016

[Sazzadul Hoque]

Ahh, okay.

Could you list all possible non-issue Java JSON libraries sorted by feasibility? It would be helpful, IF we ever decide to replace org.json and/or someone wants to provide the PR.

--

You received this message because you are subscribed to the Google Groups "Jedis" group.
To unsubscribe from this group and stop receiving emails from it, send an email to jedis_redis...@googlegroups.com.

To view this discussion on the web visit https://groups.google.com/d/msgid/jedis_redis/b327ea90-dac3-44e4-97ea-7bf6215aac3en%40googlegroups.com.

[Johannes]

Sure. Alternatives could be GSON or JSON-simple. A comparison of JSON libraries is available here.

[Sazzadul Hoque]

Thanks

To view this discussion on the web visit https://groups.google.com/d/msgid/jedis_redis/c06d5e66-10e9-4ec8-885a-7bf185949722n%40googlegroups.com.

[Johannes]

I saw that there is a pull request that will allow replacing org.json: https://github.com/redis/jedis/pull/3191

This is awesome and will enable us to keep using Jedis!

Thanks a lot.

Johannes

[Sazzadul Hoque]

Hi Johannes,

That PR will not be merged.

But you can now easily exclude 'org.json:json' with 'com.vaadin.external.google:android-json' after https://github.com/redis/jedis/pull/3242.

To view this discussion on the web visit https://groups.google.com/d/msgid/jedis_redis/8015c081-d05b-4d64-92f6-946f6a653b19n%40googlegroups.com.