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 <ichiro.furus...@gmail.com> wrote: > 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 > ./config/oauth.json > ./content/sampledata/canonicaldb.json > > Also, the ./config/container.js file is being parsed as JSON, so comments > would need to > be removed from it. > > We're integrating Shindig into a larger framework and while apparently the > parsers being > used in Shindig development are lenient (i.e., permit invalid content), the > ones that we're > using are failing these files. We're using both the json.org parser, which > conforms to the > EMCA-404 JSON standard, and the Gson parser, which we don't set in lenient > mode (for > security and data integrity reasons). > > Thanks very much for your consideration, > > Ichiro > > > On Thu, Nov 20, 2014 at 5:40 AM, Davies,Douglas <davi...@oclc.org> 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. >> >> doug >> >> On Nov 15, 2014, at 10:55 AM, Ryan Baxter <rbaxte...@gmail.com> wrote: >> >> > +1 from me. >> > >> > I one other PMC member could check the release than we can publish it. >> > >> > On Thu, Nov 13, 2014 at 8:53 AM, Stanton Sievers <ssiev...@apache.org> >> wrote: >> >> +1 >> >> >> >> Signatures and hashes look good. Smoketest of the sample common >> container >> >> looks good. >> >> >> >> On Mon, Nov 10, 2014 at 8:41 PM, Ryan Baxter <rbaxte...@apache.org> >> wrote: >> >> >> >>> Hi, >> >>> >> >>> We solved 6 issues: >> >>> >> >>> >> https://issues.apache.org/jira/issues/?jql=project%20%3D%20SHINDIG%20AND%20fixVersion%20%3D%202.5.2%20AND%20status%20%3D%20Resolved%20ORDER%20BY%20priority%20DESC >> >>> >> >>> There are still a couple of issues left in JIRA: >> >>> >> >>> >> https://issues.apache.org/jira/secure/IssueNavigator.jspa?reset=true&pid=12310741&status=1 >> >>> >> >>> Staging repo: >> >>> >> https://repository.apache.org/content/repositories/orgapacheshindig-1001/ >> >>> >> >>> Web site: >> >>> http://shindig.apache.org/ >> >>> >> >>> Vote open for 72 hours. >> >>> >> >>> [ ] +1 >> >>> [ ] +0 >> >>> [ ] -1 >> >>> >> >> >>