Return-Path: X-Original-To: apmail-cordova-dev-archive@www.apache.org Delivered-To: apmail-cordova-dev-archive@www.apache.org Received: from mail.apache.org (hermes.apache.org [140.211.11.3]) by minotaur.apache.org (Postfix) with SMTP id C692E17631 for ; Thu, 26 Mar 2015 16:47:37 +0000 (UTC) Received: (qmail 40907 invoked by uid 500); 26 Mar 2015 16:47:37 -0000 Delivered-To: apmail-cordova-dev-archive@cordova.apache.org Received: (qmail 40863 invoked by uid 500); 26 Mar 2015 16:47:37 -0000 Mailing-List: contact dev-help@cordova.apache.org; run by ezmlm Precedence: bulk List-Help: List-Unsubscribe: List-Post: List-Id: Reply-To: dev@cordova.apache.org Delivered-To: mailing list dev@cordova.apache.org Received: (qmail 40849 invoked by uid 99); 26 Mar 2015 16:47:37 -0000 Received: from athena.apache.org (HELO athena.apache.org) (140.211.11.136) by apache.org (qpsmtpd/0.29) with ESMTP; Thu, 26 Mar 2015 16:47:37 +0000 X-ASF-Spam-Status: No, hits=1.7 required=5.0 tests=FREEMAIL_ENVFROM_END_DIGIT,HTML_MESSAGE,RCVD_IN_DNSWL_LOW,SPF_PASS X-Spam-Check-By: apache.org Received-SPF: pass (athena.apache.org: domain of csantana23@gmail.com designates 209.85.214.182 as permitted sender) Received: from [209.85.214.182] (HELO mail-ob0-f182.google.com) (209.85.214.182) by apache.org (qpsmtpd/0.29) with ESMTP; Thu, 26 Mar 2015 16:47:33 +0000 Received: by obbgg8 with SMTP id gg8so50629903obb.1 for ; Thu, 26 Mar 2015 09:47:12 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20120113; h=mime-version:in-reply-to:references:date:message-id:subject:from:to :content-type; bh=bZKOsL50tpCLjzViTL1aOqpZFVFce5UsBQJHa3sCP9E=; b=ge1cRYKDhQXRlxcPK3Da9Iz8mkcG+B8oWxGi5H8vIyPG596wbnr2HqQT5rFY8kQR9f 0tzG9dQXPFHsI8/vCACx/w3VB9OG3jV4jGxKRB924nFtaJepZYhzw9A/3FNHWU7VPPCo W+YMZGXjlkOVrcSiKfAxj8RanEnJq4f9PkUjYb2Zy1x2EuWACXuh7vV1E6I+aJbus/nw /cVjlXzSXjwWAd/HaNcyX7EasZq72g1DahcuhrT59PT3dpwlH+nrqXaSlBZK3J+y79IS VnjiUPFC5OOQ4vb81+HG1BYclOqMN+JsLaPvXJGBoiQj+xYjAFm6l4rKFlx4unr3ud8j A9pg== MIME-Version: 1.0 X-Received: by 10.182.166.71 with SMTP id ze7mr11039379obb.73.1427388432862; Thu, 26 Mar 2015 09:47:12 -0700 (PDT) Received: by 10.60.103.177 with HTTP; Thu, 26 Mar 2015 09:47:12 -0700 (PDT) In-Reply-To: <68323E8E-A649-479D-933E-E9254953BF30@gmail.com> References: <68323E8E-A649-479D-933E-E9254953BF30@gmail.com> Date: Thu, 26 Mar 2015 12:47:12 -0400 Message-ID: Subject: Re: Cordova WP8: switching to Newtonsoft.Json From: Carlos Santana To: "dev@cordova.apache.org" Content-Type: multipart/alternative; boundary=e89a8ff1cf467f53d6051233c4ff X-Virus-Checked: Checked by ClamAV on apache.org --e89a8ff1cf467f53d6051233c4ff Content-Type: text/plain; charset=UTF-8 Jesse dumb question, what else is new 8-] Can we do that Mark pointed out to add a reference to a location where the windows project can get the dependency? Like adding a nuget dependency entry, and then when app get's compile, visual studio takes care of downloading Newtonsoft.Json from nuget repository? On Tue, Mar 24, 2015 at 1:38 PM, Jesse wrote: > I am looking into this. We need to include source code, as we cannot > distribute a dll. > The pull-req is definitely dead except for the fact it contain part of > this conversation. > > > > > > On Mar 24, 2015, at 6:37 AM, Ian Clelland > wrote: > > > > I'm not sure about whether Cordova has any specific policies -- there's > no > > hard rule that says we can't use third-party code, and even include it in > > our distributions (see Cordova-Android and okhttp, for instance), but we > > should probably discuss it on-list first. > > > > There are definitely rules, policies, and guidelines at the Apache level > -- > > no distribution of binaries, for instance, and there are probably > questions > > of license interaction as well between Apache and MIT licenses, if we're > > combining the code at all. > > > > > > > > On Tue, Mar 24, 2015 at 5:04 AM, Sergey Grebnov (Akvelon) < > > v-segreb@microsoft.com> wrote: > > > >> This[1] PR is being opened for a while so let's make a final decision if > >> we switch to Newtonsoft.Json or reject it. > >> > >> The idea of adding Newtonsoft.Json (MIT license) [1] looks good to me > and > >> I can quickly add necessary improvements to make sure everything looks > good > >> and works - as per mobile spec tests there are no new issues found. > >> > >> But I'm not sure about our policies regarding adding new third party > >> dependencies. Should it be discussed/voted here before merge? Does it > >> documented somewhere? > >> > >> PS. If we go w/ Newtonsoft.Json I propose to just add reference to > >> compiled dll and not to keep source code due to size(200kb vs 19mb) and > >> compile time reasons. > > > > Distributing the .dll without the source will *definitely* be a problem. > > Distributing the .dll with the source *might* be a problem. The only way > > we've been able to do this in the past (see Gradle and Crosswalk for two > > recent examples) is to have a script that the user runs (even as part of > > the platform scripts) that downloads the library from somewhere trusted > and > > installs it. We can't ship binaries as part of an Apache project. > > > > > >> > >> [1] https://github.com/apache/cordova-wp8/pull/62 > >> > >> Thx! > >> Sergey > >> > >> > >> --------------------------------------------------------------------- > >> To unsubscribe, e-mail: dev-unsubscribe@cordova.apache.org > >> For additional commands, e-mail: dev-help@cordova.apache.org > >> > >> > > --------------------------------------------------------------------- > To unsubscribe, e-mail: dev-unsubscribe@cordova.apache.org > For additional commands, e-mail: dev-help@cordova.apache.org > > -- Carlos Santana --e89a8ff1cf467f53d6051233c4ff--