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 601A31914C for ; Tue, 5 Apr 2016 15:20:33 +0000 (UTC) Received: (qmail 62066 invoked by uid 500); 5 Apr 2016 15:20:33 -0000 Delivered-To: apmail-cordova-dev-archive@cordova.apache.org Received: (qmail 62026 invoked by uid 500); 5 Apr 2016 15:20:33 -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 62014 invoked by uid 99); 5 Apr 2016 15:20:32 -0000 Received: from pnap-us-west-generic-nat.apache.org (HELO spamd2-us-west.apache.org) (209.188.14.142) by apache.org (qpsmtpd/0.29) with ESMTP; Tue, 05 Apr 2016 15:20:32 +0000 Received: from localhost (localhost [127.0.0.1]) by spamd2-us-west.apache.org (ASF Mail Server at spamd2-us-west.apache.org) with ESMTP id 030FE1A4772 for ; Tue, 5 Apr 2016 15:20:32 +0000 (UTC) X-Virus-Scanned: Debian amavisd-new at spamd2-us-west.apache.org X-Spam-Flag: NO X-Spam-Score: 3.05 X-Spam-Level: *** X-Spam-Status: No, score=3.05 tagged_above=-999 required=6.31 tests=[DKIM_SIGNED=0.1, DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1, HTML_MESSAGE=2, HTTP_ESCAPED_HOST=1.621, KAM_LOTSOFHASH=0.25, RCVD_IN_DNSWL_LOW=-0.7, RCVD_IN_MSPIKE_H3=-0.01, RCVD_IN_MSPIKE_WL=-0.01, SPF_PASS=-0.001] autolearn=disabled Authentication-Results: spamd2-us-west.apache.org (amavisd-new); dkim=pass (2048-bit key) header.d=gmail.com Received: from mx1-lw-us.apache.org ([10.40.0.8]) by localhost (spamd2-us-west.apache.org [10.40.0.9]) (amavisd-new, port 10024) with ESMTP id lbeQAYEa28gR for ; Tue, 5 Apr 2016 15:20:25 +0000 (UTC) Received: from mail-oi0-f42.google.com (mail-oi0-f42.google.com [209.85.218.42]) by mx1-lw-us.apache.org (ASF Mail Server at mx1-lw-us.apache.org) with ESMTPS id 7CDEF5F5CB for ; Tue, 5 Apr 2016 15:20:24 +0000 (UTC) Received: by mail-oi0-f42.google.com with SMTP id y204so21449717oie.3 for ; Tue, 05 Apr 2016 08:20:24 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20120113; h=mime-version:in-reply-to:references:from:date:message-id:subject:to; bh=+4jGl8mYtTHtvh4S0b3DrqChw1pZe+y5NV1mVKrIvpE=; b=yh2ZSYYhegV/xAxBwJDupiA7EeBoPKMkSTqLE8RXM3N6UoppBl5sa1vRxlzSAv9M3U +lWqVtYFF/JngmpxStubsgXYoM6v5h0cxCY1mVbT6b6IUV7m5YtSZpxxbYhyxzNFlFjf CIYSFndztnCYKeabxvhNSJTZkgPKo43G/DDU7t7XzB27m39Er+jGH0bNbk4Jr3pLCgP/ GA00C+vDPS0oUWRRhkhQeRFFjt/RmWa8z5y5Jms2sMyxQTF5CF5yZv9rfzeYj6don+i8 wSSpEUe5CpSdz0mEVpMxLMSFQxP5UbWRTlGk3pNI9xEccBkFOET7V/kMP4lAeXND94Ax L2Hw== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20130820; h=x-gm-message-state:mime-version:in-reply-to:references:from:date :message-id:subject:to; bh=+4jGl8mYtTHtvh4S0b3DrqChw1pZe+y5NV1mVKrIvpE=; b=e+HRs7C8nVcsI7h5UdMymeizCL82+y4Kmt4vEEnj3jimQH2v2jG1IH1TARz5IScbMH Gi7GsjYFchGV4wfnWMv0WKVZPnqKfHq4gHZywHGwuynRMcg6jrj2OJx1SXXYzoZiobWS 0LZCE00McRG/2XrE8z2v31K1GArI1updNvTso9nFNkPomDPb3VDn0NikkrbAOK9o7XR+ M0yo4BNeX6wPNzIIJG81cDYQ97m2ZevVRyA7jSDAHvP7NTVNQGXT2XhdMgQlxPEtaji9 8k0WTkTqsGW743ee271BHj8QxUS6KLralsHGohv971F3IYwc4CWwsxEH+/Z2/4+t4w0E I2Rw== X-Gm-Message-State: AD7BkJLP27JvnKUzYgO2yQjo3mTjTg4xJgLyHRzncG1N29XR0fC18W+HtHdaA1ePAyLeArif/DoB2IMTPWk9Aw== X-Received: by 10.157.5.209 with SMTP id 75mr13296439otd.159.1459869622187; Tue, 05 Apr 2016 08:20:22 -0700 (PDT) MIME-Version: 1.0 Received: by 10.157.18.168 with HTTP; Tue, 5 Apr 2016 08:20:02 -0700 (PDT) In-Reply-To: References: <10f86117090c4ed093729758d67b28c7@DFM-CO1MBX15-02.exchange.corp.microsoft.com> <4f17f1a36dd94258a5dda927e45e6f9c@DFM-CO1MBX15-02.exchange.corp.microsoft.com> <054B0B86-562F-4CA6-B345-D56F1F7B359E@intel.com> <97E1F3CC-6F68-4D07-877B-94E0EA487095@intel.com> <996fe7900d0f4fccb3974c4511e643fd@DFM-CO1MBX15-02.exchange.corp.microsoft.com> From: Simon MacDonald Date: Tue, 5 Apr 2016 11:20:02 -0400 Message-ID: Subject: Re: [DISCUSS] Core Plugins and Android API 23 To: "dev@cordova.apache.org" Content-Type: multipart/alternative; boundary=001a1136f8663fbc6b052fbe62fd --001a1136f8663fbc6b052fbe62fd Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: quoted-printable Just an update: Task list: 1) create cordova-plugin-compat repo, populate with code. COMPLETE Apache Repo: https://git-wip-us.apache.org/repos/asf?p=3Dcordova-plugin-compat.git Github Mirror: https://github.com/apache/cordova-plugin-compat NPM: https://www.npmjs.com/package/cordova-plugin-compat#cordova-plugin-compat 2) migrate core plugins to use cordova-plugin-compat 3) kick off plugins release 4) write blog post Simon Mac Donald http://hi.im/simonmacdonald On Mon, Apr 4, 2016 at 4:08 PM, Simon MacDonald wrote: > I'm moving forward with 1 as I've requested a new git repo and github > mirror from infra: > > https://issues.apache.org/jira/servicedesk/customer/portal/1/INFRA-11592 > https://issues.apache.org/jira/servicedesk/customer/portal/1/INFRA-11593 > > once that is done I'll get on to migrating the core plugins to use this a= s > a dependency once published to NPM. > > > Simon Mac Donald > http://hi.im/simonmacdonald > > On Mon, Apr 4, 2016 at 1:38 PM, Richard Knoll > wrote: > >> That sounds right to me. No major version bumps needed. >> >> Richard >> >> -----Original Message----- >> From: Simon MacDonald [mailto:simon.macdonald@gmail.com] >> Sent: Monday, April 4, 2016 10:35 AM >> To: dev@cordova.apache.org >> Subject: Re: [DISCUSS] Core Plugins and Android API 23 >> >> In cordova-android? No, I don't think so. For the plugins I'd bump their >> minor version. Make sense? >> >> >> Simon Mac Donald >> >> https://na01.safelinks.protection.outlook.com/?url=3Dhttp%3a%2f%2fhi.im%= 2fsimonmacdonald&data=3D01%7c01%7criknoll%40microsoft.com%7c8bdd5fa551a8427= 5174b08d35caf7ef9%7c72f988bf86f141af91ab2d7cd011db47%7c1&sdata=3DTx2MCYz9NA= hmUMBCaa6fPoOLPb8R%2be5%2f6hCvsHapVaI%3d >> >> On Mon, Apr 4, 2016 at 1:32 PM, Joe Bowser wrote: >> >> > Does this require a major version change? >> > >> > On Mon, Apr 4, 2016, 10:30 AM Richard Knoll >> wrote: >> > >> > > Sorry for being late to comment on this. If we were the only ones >> > > using the class, I would say we should just keep copying around. >> > > However, since other plugins are using it I think it's a good idea >> > > to put it in a common place and a plugin is the best way to do it. >> > > LGTM! Should someone write >> > up >> > > a blog post about this so that third-party plugins know to update? >> > > >> > > Thanks, >> > > Richard >> > > >> > > -----Original Message----- >> > > From: Simon MacDonald [mailto:simon.macdonald@gmail.com] >> > > Sent: Sunday, April 3, 2016 12:26 PM >> > > To: dev@cordova.apache.org >> > > Subject: Re: [DISCUSS] Core Plugins and Android API 23 >> > > >> > > Seems like we have momentum on this. So barring any negative >> > > comments >> > from >> > > Microsoft about this approach whom do I contact to setup a new repo >> > > on Apache GIT? >> > > >> > > >> > > Simon Mac Donald >> > > >> > > >> > https://na01.safelinks.protection.outlook.com/?url=3Dhttp%3a%2f%2fhi.i= m% >> > 2fsimonmacdonald&data=3D01%7c01%7criknoll%40microsoft.com%7c6d4d4a706f= 45 >> > 4da19d7f08d35bf5d1e8%7c72f988bf86f141af91ab2d7cd011db47%7c1&sdata=3DyG= zi >> > Fs1bd1KAZjUEloAiJ07c9EipxtrtSYxU8e5oO7M%3d >> > > >> > > On Sun, Apr 3, 2016 at 2:26 PM, julio cesar sanchez < >> > > jcesarmobile@gmail.com> >> > > wrote: >> > > >> > > > I like it too >> > > > A lot of 3rd party plugins are adding that classes too >> > > > >> > > > BTW, there is a JIRA issue to add >> > > > https://na01.safelinks.protection.outlook.com/?url=3DPermissionHel= pe >> > > > r.ja >> > > > va&data=3D01%7c01%7criknoll%40microsoft.com%7c6d4d4a706f454da19d7f= 08 >> > > > d35b >> > > > f5d1e8%7c72f988bf86f141af91ab2d7cd011db47%7c1&sdata=3DKXea2ILjY6Ja= 2Q >> > > > r4XF %2b8FoLrOpCRvinHQIaqi%2faMSXw%3d to cordova-android >> > > > https://na01.safelinks.protection.outlook.com/?url=3Dhttps%3a%2f%2= fi >> > > > ssue >> > > > s.apache.org%2fjira%2fbrowse%2fCB-10656&data=3D01%7c01%7criknoll%4= 0m >> > > > icro >> > > > soft.com%7c6d4d4a706f454da19d7f08d35bf5d1e8%7c72f988bf86f141af91ab >> > > > 2d7c >> > > > d011db47%7c1&sdata=3DY6ygyF0AqvvK2VqTcTRSNflpowAn89Xaa%2fTaMFC%2fq= lI >> > > > %3d >> > > > >> > > > >> > > > >> > > > 2016-04-03 18:43 GMT+02:00 Carlos Santana : >> > > > >> > > > > Simon I like approach >> > > > > Can we move to this approach now? Yes Then once the downstream >> > > > > distro's support API 23 and the new API methods in >> > > > > CordovaInterface we can move this into Cordova Android proper? >> > > > > Yes >> > > > > >> > > > > +1 >> > > > > >> > > > > On Thu, Mar 31, 2016 at 6:38 PM Joe Bowser >> > wrote: >> > > > > >> > > > > > LGTM! >> > > > > > >> > > > > > On Thu, Mar 31, 2016 at 3:02 PM, Simon MacDonald < >> > > > > > simon.macdonald@gmail.com> >> > > > > > wrote: >> > > > > > >> > > > > > > I really hate the fact we need to add that >> > > > > > > https://na01.safelinks.protection.outlook.com/?url=3DPermiss= io >> > > > > > > nHel >> > > > > > > per.java&data=3D01%7c01%7criknoll%40microsoft.com%7c6d4d4a70= 6f >> > > > > > > 454d >> > > > > > > a19d7f08d35bf5d1e8%7c72f988bf86f141af91ab2d7cd011db47%7c1&sd >> > > > > > > ata=3D KXea2ILjY6Ja2Qr4XF%2b8FoLrOpCRvinHQIaqi%2faMSXw%3d fi= le >> > > > > to >> > > > > > > each and every plugin that requires API 23 permission suppor= t. >> > > > > Especially >> > > > > > > since the package the class is in is different for each >> > > > > > > plugin so we >> > > > > can >> > > > > > > two, three or more copies of the same code in an app. That's >> > > > > > > just >> > > > > > wasteful. >> > > > > > > >> > > > > > > Joe responded to my bitching by saying "Maybe we should >> > > > > > > create a permissions plugin for this and make everything go >> through that" >> > > > which >> > > > > > set >> > > > > > > my mind a racing (perhaps I was just over caffeinated) and I >> > > > > > > decided >> > > > to >> > > > > > > prototype it this afternoon. >> > > > > > > >> > > > > > > I've created a new plugin called cordova-plugin-compat over >> > > > > > > at https://github.com/macdonst/cordova-plugin-compat and >> > > > > > > all it does >> > > > is >> > > > > > > deliver the org.apache.cordova.PermissionHelper class. I >> > > > > > > then turned >> > > > > > around >> > > > > > > and removed the class from the camera plugin and make >> > > > > > cordova-plugin-camera >> > > > > > > depend on cordova-plugin-compat. >> > > > > > > >> > > > > > > You can see that change over on my camera fork: >> > > > > > > >> > > > > > > https://github.com/macdonst >> > > > > > > >> > > > /cordova-plugin-camera/commit/58b72cb5c3109481df9f337ad97607858681 >> > > > b37e >> > > > > > > >> > > > > > > Can we move to this approach now? Then once the downstream >> > > > > > > distro's >> > > > > > support >> > > > > > > API 23 and the new API methods in CordovaInterface we can >> > > > > > > move this >> > > > > into >> > > > > > > Cordova Android proper? >> > > > > > > >> > > > > > > Thoughts? Are there use cases this approach doesn't work for= ? >> > > > > > > >> > > > > > > >> > > > > > > Simon Mac Donald >> > > > > > > https://na01.safelinks.protection.outlook.com/?url=3Dhttp%3a= %2 >> > > > > > > f%2f >> > > > > > > hi.im%2fsimonmacdonald&data=3D01%7c01%7criknoll%40microsoft.= co >> > > > > > > m%7c >> > > > > > > 6d4d4a706f454da19d7f08d35bf5d1e8%7c72f988bf86f141af91ab2d7cd >> > > > > > > 011d >> > > > > > > b47%7c1&sdata=3DyGziFs1bd1KAZjUEloAiJ07c9EipxtrtSYxU8e5oO7M%= 3d >> > > > > > > >> > > > > > > On Fri, Feb 5, 2016 at 2:32 PM, Richard Knoll >> > > > > > > > > > > > >> > > > > > > wrote: >> > > > > > > >> > > > > > > > Yep, I left it out of the first round by mistake; I was >> > > > > > > > looking at >> > > > an >> > > > > > out >> > > > > > > > of date version of cordova-plugin-media at the time. I've >> > > > > > > > got a PR >> > > > > open >> > > > > > > > that adds the helper here: >> > > > > > > > >> > > > > > > > https://github.com/apache/cordova-plugin-media/pull/79 >> > > > > > > > >> > > > > > > > Thanks, >> > > > > > > > Richard >> > > > > > > > >> > > > > > > > -----Original Message----- >> > > > > > > > From: julio cesar sanchez [mailto:jcesarmobile@gmail.com] >> > > > > > > > Sent: Friday, February 5, 2016 7:20 AM >> > > > > > > > To: dev@cordova.apache.org >> > > > > > > > Subject: Re: [DISCUSS] Core Plugins and Android API 23 >> > > > > > > > >> > > > > > > > Yeah, that one. >> > > > > > > > >> > > > > > > > >> > > > > > > > El viernes, 5 de febrero de 2016, Homer, Tony < >> > > > tony.homer@intel.com> >> > > > > > > > escribi=C3=B3: >> > > > > > > > >> > > > > > > > > It took me a while to find it, but I guess you mean >> CB-10455? >> > > > > > > > > >> > > > > > > > > >> > > > > > > > > >> > > > > > > > > On 2/5/16, 7:56 AM, "julio cesar sanchez" < >> > > > jcesarmobile@gmail.com >> > > > > > > > > > wrote: >> > > > > > > > > >> > > > > > > > > >I think we just forgot, somebody mentioned this on >> > > > > > > > > >slack and >> > > > there >> > > > > > is >> > > > > > > > > >an issue open on JIRA. >> > > > > > > > > > >> > > > > > > > > >2016-02-05 13:52 GMT+01:00 Homer, Tony >> > > > > > > > > >> > > > > > > > > >: >> > > > > > > > > > >> > > > > > > > > >> Earlier in this thread, Richard stated: >> > > > > > > > > >> >> This change has been merged into the file, camera, >> > > > > geolocation, >> > > > > > > > > >> >> and >> > > > > > > > > >> contacts plugins (media did not actually require any >> > > change). >> > > > > > > > > >> >> > > > > > > > > >> >> > > > > > > > > >> Media was updated on 1/15 but it still has a >> > > > > > > > > >> "cordova-android >> > > > > > > > > >=3D5.0.0-dev" >> > > > > > > > > >> engine tag. >> > > > > > > > > >> I couldn't find any other discussion about this on >> > > > > > > > > >> the list >> > > > and >> > > > > > > > > >> wondered why it was retained for media. >> > > > > > > > > >> Was this an oversight? >> > > > > > > > > >> >> > > > > > > > > >> >> > > > > > > > > >> Thanks! >> > > > > > > > > >> Tony >> > > > > > > > > >> >> > > > > > > > > >> >> > > > > > > > > >> >> > > > > > > > > >> >> > > > > > > > > >> On 1/15/16, 4:40 PM, "Simon MacDonald" < >> > > > > simon.macdonald@gmail.com >> > > > > > > > > > wrote: >> > > > > > > > > >> >> > > > > > > > > >> >LGTM >> > > > > > > > > >> > >> > > > > > > > > >> >+1 >> > > > > > > > > >> > >> > > > > > > > > >> >Simon Mac Donald >> > > > > > > > > >> > >> > > > > > https://na01.safelinks.protection.outlook.com/?url=3Dhttp%3a%2= f% >> > > > > > 2fhi >> > > > > > > > > >> >> > > > > > >> >.im%2fsimonmacdonald&data=3D01%7c01%7cRIKNOLL%40exchange.microsoft. >> > > > > > >c >> > > > > > > > > >> >> > > > > > >om%7cacc2d663ce574637d10908d32e3fd62d%7c72f988bf86f141af91ab2 >> > > > > > >d7cd >> > > > > > >0 >> > > > > > > > > >> >> > > > > >11db47%7c1&sdata=3DwCGbmJR3ZKj6lABfwOcjoLypjztI9HNX8%2bak6AhuBq= Y% >> > > > > >3d >> > > > > > > > > >> > >> > > > > > > > > >> >On Fri, Jan 15, 2016 at 4:32 PM, Joe Bowser < >> > > > bowserj@gmail.com >> > > > > > > > > > wrote: >> > > > > > > > > >> > >> > > > > > > > > >> >> Sorry, wrong PR: >> > > > > > > > > >> >> >> > > > > https://github.com/apache/cordova-plugin-inappbrowser/pull/136 >> > > > > > > > > >> >> >> > > > > > > > > >> >> On Fri, Jan 15, 2016 at 11:42 AM, Joe Bowser < >> > > > > > bowserj@gmail.com >> > > > > > > > > > wrote: >> > > > > > > > > >> >> >> > > > > > > > > >> >> > So, what does the timeline look like for a >> > > > > > > > > >> >> > Plugins >> > > > release. >> > > > > > I >> > > > > > > > > >> >> > just >> > > > > > > > > >> did a >> > > > > > > > > >> >> > major change in the InAppBrowser to fix tel: and >> sms: >> > > > URIs >> > > > > so >> > > > > > > > > >> >> > they >> > > > > > > > > do >> > > > > > > > > >> >> > stuff, and it'd be cool if this also made if >> > > > > > > > > >> >> > this got >> > > > > looked >> > > > > > > > > >> >> > over >> > > > > > > > > and >> > > > > > > > > >> >> > landed before the next release. >> > > > > > > > > >> >> > >> > > > > > > > > >> >> > >> > > > > > https://github.com/apache/cordova-plugin-inappbrowser/pull/135 >> > > > > > > > > >> >> > >> > > > > > > > > >> >> > On Thu, Jan 14, 2016 at 2:02 PM, Carlos Santana >> > > > > > > > > >> >> > < >> > > > > > > > > csantana23@gmail.com >> > > > > > > > > >> > >> > > > > > > > > >> >> > wrote: >> > > > > > > > > >> >> > >> > > > > > > > > >> >> >> I agree with final compromise, I also had a >> > > > > > > > > >> >> >> discussion >> > > > > with >> > > > > > > > > >> >> >> Nikhil >> > > > > > > > > >> >> during >> > > > > > > > > >> >> >> holiday break offline and understood the >> > > > > > > > > >> >> >> situation, some >> > > > > 3rd >> > > > > > > > > >> >> >> party >> > > > > > > > > >> >> plugins >> > > > > > > > > >> >> >> are not open source and customers/devs don't >> > > > > > > > > >> >> >> move as >> > > > fast >> > > > > as >> > > > > > > > > >> >> >> we >> > > > > > > > > wish, >> > > > > > > > > >> >> but >> > > > > > > > > >> >> >> having a CLI 6 with latest plugins being >> > > > > > > > > >> >> >> backwards >> > > > > > compatible >> > > > > > > > > >> >> >> with >> > > > > > > > > >> lower >> > > > > > > > > >> >> >> Android API is good base for developers to >> > > > > > > > > >> >> >> start >> > > > > upgrading. >> > > > > > > > > >> >> >> >> > > > > > > > > >> >> >> >> > > > > > > > > >> >> >> On Thu, Jan 14, 2016 at 1:23 PM Steven Gill < >> > > > > > > > > stevengill97@gmail.com > >> > > > > > > > > >> >> >> wrote: >> > > > > > > > > >> >> >> >> > > > > > > > > >> >> >> > Awesome! I'll kick up a discuss thread >> > > > > > > > > >> >> >> > >> > > > > > > > > >> >> >> > On Wed, Jan 13, 2016 at 5:42 PM, Richard >> > > > > > > > > >> >> >> > Knoll < >> > > > > > > > > >> riknoll@microsoft.com >> > > > > > > > > >> >> > >> > > > > > > > > >> >> >> > wrote: >> > > > > > > > > >> >> >> > >> > > > > > > > > >> >> >> > > Hey all, >> > > > > > > > > >> >> >> > > >> > > > > > > > > >> >> >> > > This change has been merged into the file, >> > > > > > > > > >> >> >> > > camera, >> > > > > > > > > geolocation, >> > > > > > > > > >> and >> > > > > > > > > >> >> >> > > contacts plugins (media did not actually >> > > > > > > > > >> >> >> > > require any >> > > > > > > > change). >> > > > > > > > > I >> > > > > > > > > >> was >> > > > > > > > > >> >> >> able >> > > > > > > > > >> >> >> > to >> > > > > > > > > >> >> >> > > build and run mobilespec on cordova-android >> > > > > > > > > >> >> >> > > 4.0.0 >> > > > and >> > > > > > > > > >> >> >> cordova-android's >> > > > > > > > > >> >> >> > > master. We should probably push for a >> > > > > > > > > >> >> >> > > plugins >> > > > release >> > > > > > > > > >> >> >> > > soon, >> > > > > > > > > since >> > > > > > > > > >> >> >> there >> > > > > > > > > >> >> >> > are >> > > > > > > > > >> >> >> > > also some major bug fixes to the contacts >> > > > > > > > > >> >> >> > > and camera plugins >> > > > > > > > > that >> > > > > > > > > >> >> were >> > > > > > > > > >> >> >> > > recently pushed. >> > > > > > > > > >> >> >> > > >> > > > > > > > > >> >> >> > > Thanks, >> > > > > > > > > >> >> >> > > Richard >> > > > > > > > > >> >> >> > > >> > > > > > > > > >> >> >> > > -----Original Message----- >> > > > > > > > > >> >> >> > > From: Nikhil Khandelwal [mailto: >> > > > > nikhilkh@microsoft.com >> > > > > > > > > ] >> > > > > > > > > >> >> >> > > Sent: Friday, January 8, 2016 12:12 PM >> > > > > > > > > >> >> >> > > To: dev@cordova.apache.org >> > > > > > > > > >> >> >> > > Subject: RE: [DISCUSS] Core Plugins and >> > > > > > > > > >> >> >> > > Android API >> > > > 23 >> > > > > > > > > >> >> >> > > >> > > > > > > > > >> >> >> > > We had a good discussion about this offline >> > > > > > > > > >> >> >> > > with >> > > > Joe, >> > > > > > > > > >> >> >> > > Simon, >> > > > > > > > > >> Steve, >> > > > > > > > > >> >> >> > Jesse, >> > > > > > > > > >> >> >> > > Parashu and Jason. We came to the following >> > > > agreement: >> > > > > > > > > >> >> >> > > This is a temporary fix to the plugins >> > > > > > > > > >> >> >> > > which we need >> > > > > to >> > > > > > > > > >> >> >> > > take >> > > > > > > > > to >> > > > > > > > > >> >> ensure >> > > > > > > > > >> >> >> > > Cordova 5.x tools behavior of using the >> > > > > > > > > >> >> >> > > edge plugins >> > > > > > does >> > > > > > > > > >> >> >> > > not >> > > > > > > > > >> break >> > > > > > > > > >> >> >> for >> > > > > > > > > >> >> >> > our >> > > > > > > > > >> >> >> > > users. We will pursue the change for all >> > > > > > > > > >> >> >> > > the >> > > > affected >> > > > > > > > plugins. >> > > > > > > > > >> This >> > > > > > > > > >> >> >> will >> > > > > > > > > >> >> >> > go >> > > > > > > > > >> >> >> > > out in the next PLUGINS release which will >> > > > > > > > > >> >> >> > > happen >> > > > > before >> > > > > > > > > >> >> >> > > the >> > > > > > > > > >> Cordova >> > > > > > > > > >> >> >> 6.0 >> > > > > > > > > >> >> >> > > release. Since this a temporary fix, we aim >> > > > > > > > > >> >> >> > > to >> > > > remove >> > > > > it >> > > > > > > > > >> >> >> > > in 6 >> > > > > > > > > >> months >> > > > > > > > > >> >> >> > > (giving reasonable time for our user base >> > > > > > > > > >> >> >> > > to move to Cordova >> > > > > > > > > 6.x+ >> > > > > > > > > >> >> >> which >> > > > > > > > > >> >> >> > > will have a better plugin version pinning >> > > > mechanism). >> > > > > > > > > >> >> >> > > >> > > > > > > > > >> >> >> > > Thanks, >> > > > > > > > > >> >> >> > > Nikhil >> > > > > > > > > >> >> >> > > >> > > > > > > > > >> >> >> > > -----Original Message----- >> > > > > > > > > >> >> >> > > From: Simon MacDonald [mailto: >> > > > > simon.macdonald@gmail.com >> > > > > > > > > ] >> > > > > > > > > >> >> >> > > Sent: Thursday, January 7, 2016 4:00 PM >> > > > > > > > > >> >> >> > > To: dev@cordova.apache.org >> > > > > > > > > >> >> >> > > Subject: Re: [DISCUSS] Core Plugins and >> > > > > > > > > >> >> >> > > Android API >> > > > 23 >> > > > > > > > > >> >> >> > > >> > > > > > > > > >> >> >> > > I'm going to play devils advocate a bit her= e: >> > > > > > > > > >> >> >> > > >> > > > > > > > > >> >> >> > > >> > > > > > > > > >> >> >> > > > There are valid reasons why many app >> > > > > > > > > >> >> >> > > > developers >> > > > > might >> > > > > > > > > >> >> >> > > > not be >> > > > > > > > > >> ready >> > > > > > > > > >> >> >> to >> > > > > > > > > >> >> >> > > > move to API level 23: >> > > > > > > > > >> >> >> > > > 1) They have an app which is >> > > > > > > > > >> >> >> > > > stabilizing or in maintenance >> > > > > > > > > >> mode >> > > > > > > > > >> >> >> and >> > > > > > > > > >> >> >> > > > they don't want to risk destabilization >> > > > > > > > > >> >> >> > > > by moving >> > > > > to a >> > > > > > > > > >> >> >> > > > new >> > > > > > > > > >> major >> > > > > > > > > >> >> >> > > > Cordova version, Cordova Android >> > > > > > > > > >> >> >> > > > platform, and >> > > > > Android >> > > > > > > > > >> >> >> > > > API >> > > > > > > > > >> level. >> > > > > > > > > >> >> >> > > > >> > > > > > > > > >> >> >> > > >> > > > > > > > > >> >> >> > > If the developer is not ready to move to a >> > > > > > > > > >> >> >> > > new >> > > > version >> > > > > > of >> > > > > > > > > Cordova >> > > > > > > > > >> >> >> Android >> > > > > > > > > >> >> >> > > or the Android API would it not also follow >> > > > > > > > > >> >> >> > > that >> > > > they >> > > > > > > > > shouldn't >> > > > > > > > > >> be >> > > > > > > > > >> >> >> > changing >> > > > > > > > > >> >> >> > > their plugin versions? >> > > > > > > > > >> >> >> > > >> > > > > > > > > >> >> >> > > >> > > > > > > > > >> >> >> > > > 2) They are using a 3rd-party plugin >> > > > > > > > > >> >> >> > > > which has >> > > > not >> > > > > > > > > >> >> >> > > > yet >> > > > > > > > > been >> > > > > > > > > >> >> >> updated >> > > > > > > > > >> >> >> > > > to request Android permissions as >> > > > > > > > > >> >> >> > > > required by API >> > > > > > level >> > > > > > > > 23. >> > > > > > > > > >> There >> > > > > > > > > >> >> >> are >> > > > > > > > > >> >> >> > > > probably a lot of plugins affected, since >> > > > > > > > > >> >> >> > > > access >> > > > to >> > > > > > any >> > > > > > > > > >> >> >> > > > of >> > > > > > > > > the >> > > > > > > > > >> >> >> > > > following things on Android M requires >> > > > > > > > > >> >> >> > > > runtime permission >> > > > > > > > > >> >> requests: >> > > > > > > > > >> >> >> > > > calendar, contacts, phone, camera, >> > > > > > > > > >> >> >> > > > microphone, location, >> > > > > > > > > >> beacons, >> > > > > > > > > >> >> >> > > sensors, SMS, storage. >> > > > > > > > > >> >> >> > > > >> > > > > > > > > >> >> >> > > >> > > > > > > > > >> >> >> > > Right, so if the 3rd party plugin they are >> > > > > > > > > >> >> >> > > using >> > > > > doesn't >> > > > > > > > > support >> > > > > > > > > >> >> >> Android >> > > > > > > > > >> >> >> > M >> > > > > > > > > >> >> >> > > they should either a) not upgrade or b) >> > > > > > > > > >> >> >> > > send a PR to >> > > > > the >> > > > > > > > > plugin >> > > > > > > > > >> >> >> > maintainer >> > > > > > > > > >> >> >> > > so everyone can benefit. >> > > > > > > > > >> >> >> > > >> > > > > > > > > >> >> >> > > >> > > > > > > > > >> >> >> > > > 3) They might not have the capacity or >> > > > > > > > > >> >> >> > > > ability >> > > > to >> > > > > > > > > >> >> >> > > > test >> > > > > > > > > their >> > > > > > > > > >> app >> > > > > > > > > >> >> >> on >> > > > > > > > > >> >> >> > > > devices running Android M. Because API 23 >> > > > > > > > > >> >> >> > > > enables >> > > > > the >> > > > > > > > > >> >> >> > > > new >> > > > > > > > > >> >> >> permissions >> > > > > > > > > >> >> >> > > > model only on Android M, it requires >> > > > > > > > > >> >> >> > > > testing on >> > > > that >> > > > > > > > > platform. >> > > > > > > > > >> >> >> > > > >> > > > > > > > > >> >> >> > > > >> > > > > > > > > >> >> >> > > Everyone has access to the Android emulator= . >> > > > > > > > > >> >> >> > > >> > > > > > > > > >> >> >> > > >> > > > > > > > > >> >> >> > > > In any of the above cases, developers >> > > > > > > > > >> >> >> > > > might still >> > > > > like >> > > > > > > > > >> >> >> > > > to >> > > > > > > > > >> benefit >> > > > > > > > > >> >> >> from >> > > > > > > > > >> >> >> > > > some of the major bug fixes in those 5 >> > > > > > > > > >> >> >> > > > popular >> > > > core >> > > > > > > > > >> >> >> > > > plugins >> > > > > > > > > >> >> >> mentioned >> > > > > > > > > >> >> >> > > > below. Or even if they weren't >> > > > > > > > > >> >> >> > > > specifically >> > > > looking >> > > > > > for >> > > > > > > > > >> >> >> > > > bug >> > > > > > > > > >> fixes, >> > > > > > > > > >> >> >> it >> > > > > > > > > >> >> >> > > > would be a much better experience if >> > > > > > > > > >> >> >> > > > adding or >> > > > > > updating >> > > > > > > > > >> >> >> > > > one >> > > > > > > > > of >> > > > > > > > > >> >> those >> > > > > > > > > >> >> >> > > > plugins would just work, rather than >> > > > > > > > > >> >> >> > > > failing on Android. The explanation for >> > > > > > > > > >> >> >> > > > the failure will not >> > > > be >> > > > > > > > > >> >> >> > > > obvious to many >> > > > > > > > > users, >> > > > > > > > > >> if >> > > > > > > > > >> >> >> they >> > > > > > > > > >> >> >> > > > overlooked the warning when installing >> > > > > > > > > >> >> >> > > > the plugin >> > > > or >> > > > > > if >> > > > > > > > > >> >> >> > > > they >> > > > > > > > > >> were >> > > > > > > > > >> >> >> > > > using another tool to add the plugin >> > > > > > > > > >> >> >> > > > where the >> > > > > warning >> > > > > > > > > wasn't >> > > > > > > > > >> >> >> surfaced. >> > > > > > > > > >> >> >> > > > >> > > > > > > > > >> >> >> > > >> > > > > > > > > >> >> >> > > Seems like you are describing a tooling >> > > > > > > > > >> >> >> > > problem >> > > > here. >> > > > > > If >> > > > > > > > > >> >> >> > > the >> > > > > > > > > >> tool >> > > > > > > > > >> >> >> > doesn't >> > > > > > > > > >> >> >> > > surface the warning or allows the user to >> > > > > > > > > >> >> >> > > add a >> > > > plugin >> > > > > > > > > >> >> >> > > that is >> > > > > > > > > >> >> >> > incompatible >> > > > > > > > > >> >> >> > > with the version of Cordova Android that is >> > > > > > > > > >> >> >> > > being >> > > > used >> > > > > > > > > >> >> >> > > really >> > > > > > > > > >> sounds >> > > > > > > > > >> >> >> > like a >> > > > > > > > > >> >> >> > > bug in the tooling to me. >> > > > > > > > > >> >> >> > > >> > > > > > > > > >> >> >> > > >> > > > > > > > > >> >> >> > > > Of course developers should be encouraged >> > > > > > > > > >> >> >> > > > to >> > > > upgrade >> > > > > > to >> > > > > > > > > >> >> >> > > > the >> > > > > > > > > >> latest >> > > > > > > > > >> >> >> > > > most secure highest-quality version of >> > Cordova. >> > > > But >> > > > > > the >> > > > > > > > > >> >> >> encouragement >> > > > > > > > > >> >> >> > > > does not need to be so forceful. This >> > > > > > > > > >> >> >> > > > proposed >> > > > > change >> > > > > > > > > >> >> >> > > > gives >> > > > > > > > > >> >> >> developers >> > > > > > > > > >> >> >> > > > more time to upgrade, and allows for more >> > > > > > > > > >> >> >> > > > choice >> > > > > about >> > > > > > > > > >> >> >> > > > when >> > > > > > > > > to >> > > > > > > > > >> >> >> upgrade >> > > > > > > > > >> >> >> > > > individual parts >> > > > > > > > > >> >> >> > > > (plugins) rather than limiting them to >> > > > > all-or-nothing. >> > > > > > > > > >> >> >> > > >> > > > > > > > > >> >> >> > > >> > > > > > > > > >> >> >> > > I would argue that the developer has an >> > > > > > > > > >> >> >> > > infinite >> > > > > amount >> > > > > > > > > >> >> >> > > of >> > > > > > > > > time >> > > > > > > > > >> to >> > > > > > > > > >> >> >> > > upgrade. Nothing is forcing you to upgrade >> > > > > > > > > >> >> >> > > to the >> > > > > latest >> > > > > > > > > Cordova >> > > > > > > > > >> >> >> Android >> > > > > > > > > >> >> >> > or >> > > > > > > > > >> >> >> > > Android API. They can continue to use the >> > > > > > > > > >> >> >> > > same >> > > > version >> > > > > > of >> > > > > > > > > Cordova >> > > > > > > > > >> >> >> Android >> > > > > > > > > >> >> >> > > and plugins that are currently working in >> > > > > > > > > >> >> >> > > their >> > > app. >> > > > > If >> > > > > > > > > >> >> >> > > the >> > > > > > > > > >> >> developer >> > > > > > > > > >> >> >> is >> > > > > > > > > >> >> >> > > using semver properly and we do our job >> > > > > > > > > >> >> >> > > right they shouldn't >> > > > > > > > > >> pick up >> > > > > > > > > >> >> >> > > breaking changes. For instance, if the API >> > > > > > > > > >> >> >> > > of a >> > > > plugin >> > > > > > > > > changes we >> > > > > > > > > >> >> bump >> > > > > > > > > >> >> >> > the >> > > > > > > > > >> >> >> > > major version of the plugin so users who >> > > > > > > > > >> >> >> > > setup their >> > > > > > > > > config.xml >> > > > > > > > > >> to >> > > > > > > > > >> >> >> use: >> > > > > > > > > >> >> >> > > >> > > > > > > > > >> >> >> > > > > spec=3D"^1.1.0" >> > > > /> >> > > > > > > > > >> >> >> > > >> > > > > > > > > >> >> >> > > will effectively prevent the user from >> > > > > > > > > >> >> >> > > picking up >> > > > the >> > > > > > > > > >> >> >> > > breaking >> > > > > > > > > >> >> change >> > > > > > > > > >> >> >> in >> > > > > > > > > >> >> >> > > camera version 2.0.0. >> > > > > > > > > >> >> >> > > >> > > > > > > > > >> >> >> > > Simon Mac Donald >> > > > > > > > > >> >> >> > > >> > > > > > > > > >> >> >> > > >> > > > > > > > > >> >> >> > >> > > > > > > > > >> >> >> >> > > > > > > > > >> >> >> > > > > > > > > >> >> > > > > > > > > >> > > > > > https://na01.safelinks.protection.outlook.com/?url=3Dhttp%3a%2= f% >> > > > > > 2fhi >> > > > > > .im% >> > > > > > > > > 2fsimonmacdonald&data=3D01%7c01%7cnikhilkh%40microsoft.c= om >> > > > > > %7c84fb6405edf >> > > > > > > > > >> > > > > > 74f47453208d317beb5e5%7c72f988bf86f141af91ab2d7cd011db47%7c1&s >> > > > > > data >> > > > > > =3DB8D >> > > > > > > > > uMuPF8aUU6xvIlpYTi3LJPs1HbENcHXS8JrHqEu0%3d >> > > > > > > > > >> >> >> > > B >> > > > > > > > > >> >> >> >> > > > > > > > > >> >> > > > > > KKKKKKKKKKKKKKKKKKKKKKKKKKKKKKKKKKKKKKKKKKKKKKKKKKKKKKKKKKKKKK >> > > > > > KKKK >> > > > > > K >> > > > > > > > > >> KCB >> > > > > > > > > >> >> >> > > [ X =DC=9AX K K[XZ[ >> > > > > > > > > >> >> >> > > ] ][ X =DC=9AX P =DC=99 =DD=98K \ X = K =DC=99 B =DC=88 Y ] >> > > > > > > > > >> >> >> > > [=DB=98[ >> > > [X[ >> > > > > > > > K[XZ[ >> > > > > > > > > >> >> >> > > ] Z [ =DC=99 =DD=98K \ X K =DC=99 B >> > > > > > > > > >> >> >> > > >> > > > > > > > > >> >> >> > >> > > > > > > > > >> >> >> >> > > > > > > > > >> >> > >> > > > > > > > > >> >> > >> > > > > > > > > >> >> >> > > > > > > > > >> >> > > > > > > > > >> >> > > > > > -------------------------------------------------------------- >> > > > > > ---- >> > > > > > - >> > > > > > > > > >> -- To unsubscribe, e-mail: >> > > > > > > > > >> dev-unsubscribe@cordova.apache.org >> > > > > > > > > >> > > > > > > > > >> For additional commands, e-mail: >> > > > > > > > > >> dev-help@cordova.apache.org >> > > > > > > > > >> > > > > > > > > >> >> > > > > > > > > >> > > > > > > > >> > > > > > > >> > > > > > >> > > > > >> > > > >> > > >> > >> > > --001a1136f8663fbc6b052fbe62fd--