Mailing-List: contact dev-help@cordova.apache.org; run by ezmlm
Precedence: bulk
Reply-To: dev@cordova.apache.org
Received-SPF: pass (nike.apache.org: domain of kamrik@google.com designates
 209.85.192.48 as permitted sender)
MIME-Version: 1.0
In-Reply-To: 
 <CAK_TSXKo-XR+7DsxXZOt0t6T80jrLY=_EKSC+MfvPmjSGb6x9A@mail.gmail.com>
References: 
 <CAMSbhBj1VyWmDZX4kQ7iVzgcFop8Za8z0+DMOeEL+QYuyx+cfw@mail.gmail.com>
 <CABiQX1WgE3wrCVW4U58tBE9gH0Focy4gzezwByegnpHG98Y6Dw@mail.gmail.com>
 <CA+GVa3Zshxu_r3oVzwreeEL0ZJBnik82t07OQ_j8Mv5T1CktgQ@mail.gmail.com>
 <CAK_TSXKo-XR+7DsxXZOt0t6T80jrLY=_EKSC+MfvPmjSGb6x9A@mail.gmail.com>
From: Mark Koudritsky <kamrik@google.com>
Date: Mon, 30 Mar 2015 16:51:15 -0400
Message-ID: 
 <CA+GVa3bjbFp26b+WjLGuNDrB3gTT3__T20b38nc4T-OX_96SAQ@mail.gmail.com>
Subject: Re: [VOTE] Cordova-Plugin-Whitelist and
 Cordova-Plugin-Legacy-Whitelist
 Plugins Release (attempt 2)
To: dev@cordova.apache.org
Content-Type: multipart/alternative; boundary=001a113ab834d7da13051287a5a7

--001a113ab834d7da13051287a5a7
Content-Type: text/plain; charset=UTF-8

Another quick option for now: we can pin hello-world version in
cordova-lib's package.json to 3.8.0 which doesn't use the whitelist
plugin.  Later, when we release the tools pinned to cordova-android 4.0 we
will have to change the hello-world dep to "^3.9.0".
This way it's a minimal change to cordova-lib and we can go ahead with this
vote.

On Mon, Mar 30, 2015 at 3:49 PM, Ian Clelland <iclelland@chromium.org>
wrote:

> So this isn't a -1, unless we can't fix the problem in cordova-lib. And
> it's okay, I think, to fix it in cordova-lib/master and release that as
> well. Nothing is actually broken by publishing either app-hello-world or
> plugin-whitelist until we release a version of tools that is pinned to the
> new hello-world.
>
>
> On Mon, Mar 30, 2015 at 3:40 PM, Mark Koudritsky <kamrik@google.com>
> wrote:
>
> > Looks like this breaks on cordova-android 3.7.1 (currently latest
> released)
> >
> > cordova-app-hello-world requires the whitelist plugin
> > whitelist plugin depends on cordova-android 4.0 via the engine tag
> > <engine name="cordova-android" version=">=4.0.0-dev" />
> > The resulting error message is
> > Plugin doesn't support this project's cordova-android version.
> > cordova-android: 3.7.1, failed version requirement: >=4.0.0-dev
> >
> > Simply removing the <engine> tag results in compile errors. (change
> > in CustomConfigXmlParser.parse() signature).
> >
> > Any ideas on how to solve this would be appreciated. Ideas we came up
> with
> > in a quick discussion:
> >
> > 1. Add "ignore" attribute to the <engine> tag that would result in plugin
> > being silently ignored for incompatible engines rather than going down in
> > flames.
> > 2. Add either <version> child or "version" attribute to the <platform>
> tag
> > in plugin.xml
> >
>
> This is my favorite idea so far -- I think we could do something like this:
> <plugin ... >
>   <platform name="android">
>     <version match=">=4.0.0">
>       <!-- tags -->
>     </version>
>   </platform>
> </plugin>
>
> I think this is overdue, and we'll find it useful as we move from 3.x to
> 4.x on other platforms as well. We would allow multiple <version> tags
> inside of each <platform>, and any tags outside of the <version> tags are
> the default if nothing matches.
>
> In this case, there won't be any tags outside of the <version> tag, so the
> fallback for 3.x is to install nothing, but succeed.
>
> Ian
>
>
> >
> > On Fri, Mar 27, 2015 at 4:31 PM, Andrew Grieve <agrieve@chromium.org>
> > wrote:
> >
> > > +1
> > >
> > > * Tested that both plugins compile on Android
> > > * Tested that the non-legacy plugin does what its supposed to on
> Android
> > >
> > > On Thu, Mar 26, 2015 at 7:46 PM, Steven Gill <stevengill97@gmail.com>
> > > wrote:
> > >
> > > > Please review and vote on the releases of cordova-plugin-whitelist
> and
> > > > cordova-plugin-legacy-whitelist plugins.
> > > >
> > > > Release issue: https://issues.apache.org/jira/browse/CB-8739
> > > >
> > > > The plugins have been published to
> > > > dist/dev:https://dist.apache.org/repos/dist/dev/cordova/CB-8739/
> > > >
> > > > The packages were published from their corresponding git tags:
> > > > cordova-plugin-whitelist: 1.0.0
> > > > (38f5a9fa2c)cordova-plugin-legacy-whitelist: 1.0.0 (1ce26eba64)
> > > >
> > > > Upon a successful vote I will upload the archives to dist/ and upload
> > > > them to npm.
> > > >
> > > > Voting guidelines:
> > > >
> > >
> >
> https://github.com/apache/cordova-coho/blob/master/docs/release-voting.md
> > > >
> > > > Voting will go on for a minimum of 48 hours.
> > > >
> > > > I vote +1:
> > > > * Ran coho audit-license-headers over the relevant repos
> > > > * Ran coho check-license to ensure all dependencies and
> > > > subdependencies have Apache-compatible licenses
> > > >
> > >
> >
>

--001a113ab834d7da13051287a5a7--