Re: [VOTE] Release Apache Shindig version 2.5.2

2014-11-19 Thread Ichiro Furusato
Hi Ryan, I'll move my further comments to the other thread. Cheers, Ichiro On Thu, Nov 20, 2014 at 9:27 AM, Ryan Baxter wrote: > Ichiro, I do not think that this warrants a "no" vote on the release > at this point. Lets continue the dicussion about this issue on your > other email thread an

Re: [VOTE] Release Apache Shindig version 2.5.2

2014-11-19 Thread Davies,Douglas
Correct. On Nov 19, 2014, at 3:10 PM, Ryan Baxter wrote: > Doug, I assume this is a dependency in your own implementation? >

Re: [VOTE] Release Apache Shindig version 2.5.2

2014-11-19 Thread Ryan Baxter
Ichiro, I do not think that this warrants a "no" vote on the release at this point. Lets continue the dicussion about this issue on your other email thread and decide what to do. On Wed, Nov 19, 2014 at 3:10 PM, Ichiro Furusato wrote: > Hi Doug, > > While it may not be on everyone's radar, would

Re: [VOTE] Release Apache Shindig version 2.5.2

2014-11-19 Thread Ryan Baxter
Doug, I assume this is a dependency in your own implementation? On Wed, Nov 19, 2014 at 11:40 AM, Davies,Douglas wrote: > I found an issue with the updated guava version conflicting with our bonecp > version, but I've fixed it in our end. So still a +1 and looking forward to > the release. > >

Re: [VOTE] Release Apache Shindig version 2.5.2

2014-11-19 Thread Ichiro Furusato
Hi Doug, While it may not be on everyone's radar, would it be possible to remove the comments in the following JSON files so that they are valid JSON prior to the release, so that we can use Shindig without requiring a patch? The files are: ./config/gadget-admin.json ./config/oauth2.json ./

Re: [VOTE] Release Apache Shindig version 2.5.2

2014-11-19 Thread Davies,Douglas
I found an issue with the updated guava version conflicting with our bonecp version, but I've fixed it in our end. So still a +1 and looking forward to the release. doug On Nov 15, 2014, at 10:55 AM, Ryan Baxter wrote: > +1 from me. > > I one other PMC member could check the release than we