Re: [sensinact-dev] JSON library issue

[Rémi Druilhe <remi.druilhe@xxxxxxxxx>]

I just modified the JSON library to test if it compiles. It seems that we need to do another minor change in the old library to make it work with our current code.

So, back to the point 1., another modification as to be taken into account into the amendment of the library: in the file JSONArray.java, the class should implements Iterable<Object>.

--
Rémi DRUILHE

2017-05-24 10:38 GMT+02:00 Rémi Druilhe <remi.druilhe@xxxxxxxxx>:

Hello Wayne,

Sorry for the late reply. We just had a discussion with Christophe and here is what we agreed to do based on your proposal:

1. We are going to modify the JSON library version 2008-09-18. The file that will be modified will be JSONException.java. The modification is done on line 8, we are changing Exception to RuntimeException. This modification is done to avoid a large refactor of our source code.
2. The new library will be added to our project as any others packages, thus there is no link to the public JSON library because it is embedded.
3. Eclipse will amend the library to consider this minor modification.
4. We will open a low-priority issue to migrate to another JSON library.

I tried to formalize as much as possible the various tasks. If this is okay for you, we will start the first task and continue the IP review.

Just a last question, because we had to do some minor modification to be compliant with Eclipse requirement on some third party libraries, should we create a new IP review of the sensiNact source code? Can we reopen the previous CQ and attach the new code? And, do we need to wait for a new approvement from you to push the code to the Git repository?

Thanks a lot,

Best regards.

--
Rémi DRUILHE