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
>> >>>
>>
>>
>>

Reply via email to