[jira] [Commented] (SLING-6679) Replace usage of org.apache.sling.commons.json.* and org.json

2017-04-13 Thread Karl Pauls (JIRA)
[ https://issues.apache.org/jira/browse/SLING-6679?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15967468#comment-15967468 ] Karl Pauls commented on SLING-6679: --- IC, well, I agree with Carsten that it is in general better to make

[jira] [Commented] (SLING-6679) Replace usage of org.apache.sling.commons.json.* and org.json

2017-04-13 Thread Radu Cotescu (JIRA)
[ https://issues.apache.org/jira/browse/SLING-6679?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15967432#comment-15967432 ] Radu Cotescu commented on SLING-6679: - Well, I have no preference, but seeing the last few

[jira] [Commented] (SLING-6679) Replace usage of org.apache.sling.commons.json.* and org.json

2017-04-13 Thread Karl Pauls (JIRA)
[ https://issues.apache.org/jira/browse/SLING-6679?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15967395#comment-15967395 ] Karl Pauls commented on SLING-6679: --- [~radu.cotescu], you can if you think it is better but personally,

[jira] [Commented] (SLING-6679) Replace usage of org.apache.sling.commons.json.* and org.json

2017-04-13 Thread Radu Cotescu (JIRA)
[ https://issues.apache.org/jira/browse/SLING-6679?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15967337#comment-15967337 ] Radu Cotescu commented on SLING-6679: - [~karlpauls], should we also change this code to use

[jira] [Commented] (SLING-6679) Replace usage of org.apache.sling.commons.json.* and org.json

2017-03-31 Thread Carsten Ziegeler (JIRA)
[ https://issues.apache.org/jira/browse/SLING-6679?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15950887#comment-15950887 ] Carsten Ziegeler commented on SLING-6679: - Ok, let's agree that we regard this differently. So all

[jira] [Commented] (SLING-6679) Replace usage of org.apache.sling.commons.json.* and org.json

2017-03-31 Thread Karl Pauls (JIRA)
[ https://issues.apache.org/jira/browse/SLING-6679?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15950809#comment-15950809 ] Karl Pauls commented on SLING-6679: --- I'm not sure I agree - but regardless: I'm wondering why you think

[jira] [Commented] (SLING-6679) Replace usage of org.apache.sling.commons.json.* and org.json

2017-03-31 Thread Carsten Ziegeler (JIRA)
[ https://issues.apache.org/jira/browse/SLING-6679?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15950730#comment-15950730 ] Carsten Ziegeler commented on SLING-6679: - Sure, but with that argument we could also say if one

[jira] [Commented] (SLING-6679) Replace usage of org.apache.sling.commons.json.* and org.json

2017-03-31 Thread Karl Pauls (JIRA)
[ https://issues.apache.org/jira/browse/SLING-6679?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15950714#comment-15950714 ] Karl Pauls commented on SLING-6679: --- [~cziegeler], hm, I'm not so sure that makes that much sense. If we

[jira] [Commented] (SLING-6679) Replace usage of org.apache.sling.commons.json.* and org.json

2017-03-30 Thread Carsten Ziegeler (JIRA)
[ https://issues.apache.org/jira/browse/SLING-6679?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15950327#comment-15950327 ] Carsten Ziegeler commented on SLING-6679: - Thanks for all the great work [~karlpauls] and everyone

[jira] [Commented] (SLING-6679) Replace usage of org.apache.sling.commons.json.* and org.json

2017-03-29 Thread Oliver Lietz (JIRA)
[ https://issues.apache.org/jira/browse/SLING-6679?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15946864#comment-15946864 ] Oliver Lietz commented on SLING-6679: - Are there any plans for modules in contrib, e.g. distribution?

[jira] [Commented] (SLING-6679) Replace usage of org.apache.sling.commons.json.* and org.json

2017-03-29 Thread Robert Munteanu (JIRA)
[ https://issues.apache.org/jira/browse/SLING-6679?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15946834#comment-15946834 ] Robert Munteanu commented on SLING-6679: Saw the commit just now, thanks! > Replace usage of

[jira] [Commented] (SLING-6679) Replace usage of org.apache.sling.commons.json.* and org.json

2017-03-29 Thread Karl Pauls (JIRA)
[ https://issues.apache.org/jira/browse/SLING-6679?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15946830#comment-15946830 ] Karl Pauls commented on SLING-6679: --- [~rombert], yeah - I removed one snaphot too many. Carsten just

[jira] [Commented] (SLING-6679) Replace usage of org.apache.sling.commons.json.* and org.json

2017-03-29 Thread Robert Munteanu (JIRA)
[ https://issues.apache.org/jira/browse/SLING-6679?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15946827#comment-15946827 ] Robert Munteanu commented on SLING-6679: [~karlpauls] - the launchpad does not start anymore after

[jira] [Commented] (SLING-6679) Replace usage of org.apache.sling.commons.json.* and org.json

2017-03-28 Thread Karl Pauls (JIRA)
[ https://issues.apache.org/jira/browse/SLING-6679?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15946031#comment-15946031 ] Karl Pauls commented on SLING-6679: --- I did some more clean-up and resolved all the sub-issues. Please

[jira] [Commented] (SLING-6679) Replace usage of org.apache.sling.commons.json.* and org.json

2017-03-28 Thread Karl Pauls (JIRA)
[ https://issues.apache.org/jira/browse/SLING-6679?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15945115#comment-15945115 ] Karl Pauls commented on SLING-6679: --- I just commited implementations for all the subissues (with the

[jira] [Commented] (SLING-6679) Replace usage of org.apache.sling.commons.json.*

2017-03-26 Thread Karl Pauls (JIRA)
[ https://issues.apache.org/jira/browse/SLING-6679?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15942467#comment-15942467 ] Karl Pauls commented on SLING-6679: --- I think we are getting closer. As far as I can see we have the

[jira] [Commented] (SLING-6679) Replace usage of org.apache.sling.commons.json.*

2017-03-24 Thread Karl Pauls (JIRA)
[ https://issues.apache.org/jira/browse/SLING-6679?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15941360#comment-15941360 ] Karl Pauls commented on SLING-6679: --- Ok, I'm done with a first path. There should be patches for all the

[jira] [Commented] (SLING-6679) Replace usage of org.apache.sling.commons.json.*

2017-03-24 Thread Karl Pauls (JIRA)
[ https://issues.apache.org/jira/browse/SLING-6679?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15940703#comment-15940703 ] Karl Pauls commented on SLING-6679: --- I did do that in one or two places where it seemed sensible. For

[jira] [Commented] (SLING-6679) Replace usage of org.apache.sling.commons.json.*

2017-03-24 Thread Stefan Seifert (JIRA)
[ https://issues.apache.org/jira/browse/SLING-6679?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15940670#comment-15940670 ] Stefan Seifert commented on SLING-6679: --- i had a look as the patch from SLING-6694 - looks good to

[jira] [Commented] (SLING-6679) Replace usage of org.apache.sling.commons.json.*

2017-03-24 Thread Karl Pauls (JIRA)
[ https://issues.apache.org/jira/browse/SLING-6679?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15940549#comment-15940549 ] Karl Pauls commented on SLING-6679: --- I updated the patches on SLING-6681, SLING-6683, SLING-6684,

[jira] [Commented] (SLING-6679) Replace usage of org.apache.sling.commons.json.*

2017-03-23 Thread Karl Pauls (JIRA)
[ https://issues.apache.org/jira/browse/SLING-6679?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15939343#comment-15939343 ] Karl Pauls commented on SLING-6679: --- I just converted (i.e., provided a patch for) the contentloader

[jira] [Commented] (SLING-6679) Replace usage of org.apache.sling.commons.json.*

2017-03-23 Thread Karl Pauls (JIRA)
[ https://issues.apache.org/jira/browse/SLING-6679?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15938441#comment-15938441 ] Karl Pauls commented on SLING-6679: --- I created a johnzon bundle in commons that exports javax.json. See

[jira] [Commented] (SLING-6679) Replace usage of org.apache.sling.commons.json.*

2017-03-23 Thread Karl Pauls (JIRA)
[ https://issues.apache.org/jira/browse/SLING-6679?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15938139#comment-15938139 ] Karl Pauls commented on SLING-6679: --- [~sseif...@pro-vision.de] I agree, we need to double check that -

[jira] [Commented] (SLING-6679) Replace usage of org.apache.sling.commons.json.*

2017-03-23 Thread Stefan Seifert (JIRA)
[ https://issues.apache.org/jira/browse/SLING-6679?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15938127#comment-15938127 ] Stefan Seifert commented on SLING-6679: --- another small point on the patches: JsonReader, JsonParser,

[jira] [Commented] (SLING-6679) Replace usage of org.apache.sling.commons.json.*

2017-03-23 Thread Karl Pauls (JIRA)
[ https://issues.apache.org/jira/browse/SLING-6679?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15938046#comment-15938046 ] Karl Pauls commented on SLING-6679: --- Ok, I will create a new org.apache.sling.commons.johnzon bundle for

[jira] [Commented] (SLING-6679) Replace usage of org.apache.sling.commons.json.*

2017-03-22 Thread Stefan Seifert (JIRA)
[ https://issues.apache.org/jira/browse/SLING-6679?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15937152#comment-15937152 ] Stefan Seifert commented on SLING-6679: --- bq. catch(Exception e) is going to catch RuntimeException

[jira] [Commented] (SLING-6679) Replace usage of org.apache.sling.commons.json.*

2017-03-22 Thread Karl Pauls (JIRA)
[ https://issues.apache.org/jira/browse/SLING-6679?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15937139#comment-15937139 ] Karl Pauls commented on SLING-6679: --- In regard to rolling our own johnzon bundle, I guess it wouldn't be

[jira] [Commented] (SLING-6679) Replace usage of org.apache.sling.commons.json.*

2017-03-22 Thread Karl Pauls (JIRA)
[ https://issues.apache.org/jira/browse/SLING-6679?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15937126#comment-15937126 ] Karl Pauls commented on SLING-6679: --- [~sseif...@pro-vision.de] I'm not sure I understand what you are

[jira] [Commented] (SLING-6679) Replace usage of org.apache.sling.commons.json.*

2017-03-22 Thread Stefan Seifert (JIRA)
[ https://issues.apache.org/jira/browse/SLING-6679?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15936995#comment-15936995 ] Stefan Seifert commented on SLING-6679: --- {quote} Furthermore, it is a question if we are happy with

[jira] [Commented] (SLING-6679) Replace usage of org.apache.sling.commons.json.*

2017-03-22 Thread Stefan Seifert (JIRA)
[ https://issues.apache.org/jira/browse/SLING-6679?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15936988#comment-15936988 ] Stefan Seifert commented on SLING-6679: --- thanks for your efforts this far! i had a brief look at

[jira] [Commented] (SLING-6679) Replace usage of org.apache.sling.commons.json.*

2017-03-22 Thread Karl Pauls (JIRA)
[ https://issues.apache.org/jira/browse/SLING-6679?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15936711#comment-15936711 ] Karl Pauls commented on SLING-6679: --- Ok, well I guess that is a none-started then. Oh well. > Replace

[jira] [Commented] (SLING-6679) Replace usage of org.apache.sling.commons.json.*

2017-03-22 Thread Stefan Seifert (JIRA)
[ https://issues.apache.org/jira/browse/SLING-6679?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15936705#comment-15936705 ] Stefan Seifert commented on SLING-6679: --- this was already discussed in the mailing lists:

[jira] [Commented] (SLING-6679) Replace usage of org.apache.sling.commons.json.*

2017-03-22 Thread Karl Pauls (JIRA)
[ https://issues.apache.org/jira/browse/SLING-6679?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15936696#comment-15936696 ] Karl Pauls commented on SLING-6679: --- I suppose instead of using the geronimo and johnzon bundles we

[jira] [Commented] (SLING-6679) Replace usage of org.apache.sling.commons.json.*

2017-03-22 Thread Karl Pauls (JIRA)
[ https://issues.apache.org/jira/browse/SLING-6679?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15936663#comment-15936663 ] Karl Pauls commented on SLING-6679: --- Ok, I hopefully created issue for all the important modules. Right

[jira] [Commented] (SLING-6679) Replace usage of org.apache.sling.commons.json.*

2017-03-22 Thread Karl Pauls (JIRA)
[ https://issues.apache.org/jira/browse/SLING-6679?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15936596#comment-15936596 ] Karl Pauls commented on SLING-6679: --- I started to prototype some replacements and created sub-tasks for