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 89903186A2 for ; Tue, 22 Sep 2015 01:32:45 +0000 (UTC) Received: (qmail 92380 invoked by uid 500); 22 Sep 2015 01:32:35 -0000 Delivered-To: apmail-cordova-dev-archive@cordova.apache.org Received: (qmail 92338 invoked by uid 500); 22 Sep 2015 01:32:35 -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 92326 invoked by uid 99); 22 Sep 2015 01:32:35 -0000 Received: from Unknown (HELO spamd2-us-west.apache.org) (209.188.14.142) by apache.org (qpsmtpd/0.29) with ESMTP; Tue, 22 Sep 2015 01:32:35 +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 277FD1A5282 for ; Tue, 22 Sep 2015 01:32:35 +0000 (UTC) X-Virus-Scanned: Debian amavisd-new at spamd2-us-west.apache.org X-Spam-Flag: NO X-Spam-Score: 2.901 X-Spam-Level: ** X-Spam-Status: No, score=2.901 tagged_above=-999 required=6.31 tests=[DKIM_SIGNED=0.1, DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1, HTML_MESSAGE=3, URIBL_BLOCKED=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-eu-west.apache.org ([10.40.0.8]) by localhost (spamd2-us-west.apache.org [10.40.0.9]) (amavisd-new, port 10024) with ESMTP id cFppZFlM3HSH for ; Tue, 22 Sep 2015 01:32:23 +0000 (UTC) Received: from mail-ob0-f175.google.com (mail-ob0-f175.google.com [209.85.214.175]) by mx1-eu-west.apache.org (ASF Mail Server at mx1-eu-west.apache.org) with ESMTPS id 5BD382039F for ; Tue, 22 Sep 2015 01:32:22 +0000 (UTC) Received: by obbzf10 with SMTP id zf10so96436378obb.2 for ; Mon, 21 Sep 2015 18:32:21 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20120113; h=mime-version:references:in-reply-to:from:date:message-id:subject:to :content-type; bh=t2BQzk5tqh/NMCwHUEpjKOPmkT8Qz7oUnlaEpZRnHBo=; b=Z7NVm+OklctFdAUTFQR8VCwCx+4uwEzzkI7gXsNAC3DM/PpSlWCk3a+wdcbAiyxXfo P7v/oXQChjfH0i+qLYnrBSzOs0q9ohhsPvjRjjF7N28n16icfPmt86wq6+QH91GuFggG bHkX85nDPqQe9HVcOMVu9hwViAIGaF/VXQFQtK7mkbwgdgkk0CvGmggQ6XWFLg0CXGMx +P6Q0uADNKVKT0xNo9CJG98Rv3a4qVe4hOZr/0YMq01NuHvVrhDrAVqIniEynnEhg/Qb wlM4TzsDCLRtuSh7B6zdTO5nqK0HibunjKLZkvCcn9QcNpXAkYUNHa4b3IRbRqDYW1uo s9PA== X-Received: by 10.60.39.133 with SMTP id p5mr5982021oek.72.1442885541072; Mon, 21 Sep 2015 18:32:21 -0700 (PDT) MIME-Version: 1.0 References: In-Reply-To: From: Joe Bowser Date: Tue, 22 Sep 2015 01:32:11 +0000 Message-ID: Subject: Re: [Android] 5.0.x release branch? To: dev@cordova.apache.org Content-Type: multipart/alternative; boundary=089e013cc03c20963405204bf8ab --089e013cc03c20963405204bf8ab Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: quoted-printable On Mon, Sep 21, 2015 at 5:43 PM Nikhil Khandelwal wrote: > Can you explain why latest plugins will not be compatible with older > versions of Cordova? They won't be compatible because Cordova-Android compiles against API 22, and these plugins will require API 23 so that they can detect permissions and support Marshmellow. > Can this be avoided by any means? Only with a lot of Java reflection, and I'd rather not subject plugin developers to that, or try to hide it under the hood in some awful utility class that everyone will want to see die. I'm very much a fan of if statements because they work, and they're easy to read and debug, unlike when bad things happen to things you reflect into. Plugins that require API 23 will only work with Cordova-Android 5.0 and up. This only impacts five of our core plugins, but any plugin that requires permissions from the Android Manifest will have to be updated. If we can avoid using advanced language tricks to make the APKs compatible, we should do that. When you mean they would not be compatible - will it result in a build or > runtime failure? > > This will be a build failure, since API 22 does not have these permissions, nor does it have the code required for API 23. > For marshmallow, what is the guidance that we need to issue to the larger > Cordova plugin ecosystem? Joe you are ahead of the curve here compared to > most other plugin developers - a blot post explaining what are known > gotchas would be great. I really hope we can use our Cordova blog to > communicate these changes actively to the plugin ecosystem. This mailing > list only gets 400+ subscribers. > > There will be a blog post once 5.0 is released. We're not forcing people to upgrade to 5.0, and we will be supporting the 4.x branch for six months. This does mean we're stuck supporting 3.x, 4.x and 5.x for a brief window, but I have no control over when Marshmallow is released, only whether we want to support it or not. I think we do, but I could be wrong. At least this should be easier than the jump from 3.x to 4.x for most people, but the alternative is that your plugin just doesn't work at all on Marshmallow. We need to at least give plugin developers this option, since it'll roll out on all the Nexus devices in the next two weeks, and we'll hear more about it. > Can you re-base your cordova-android over the current master? It's hard t= o > see a diff in the current form: > https://github.com/apache/cordova-android/compare/master...infil00p:smore= s > > I had to do a merge commit to get this to happen (boo), but it should be mostly cleaned up now. It seems some style cleanup creeped into the most recent changes, but this should be a bit more readable. > -Nikhil > > -----Original Message----- > From: Joe Bowser [mailto:bowserj@gmail.com] > Sent: Monday, September 21, 2015 2:14 PM > To: dev > Subject: [Android] 5.0.x release branch? > > Hey > > In the next two weeks, Marshmallow will most likely getting released with > the brand new Nexus 6P being released from Huawei. Given that most of th= e > Nexus devices will be getting this release, we should probably release th= e > 5.0.x branch of Android soon, and get the new plugins updated. > > It should be noted that the latest plugins will not be compatible with > older versions of Cordova, which is a big deal. This is due to the use o= f > various compatibility checks to make sure they support Marshmallow and > older versions of Android. > > So, if everyone can look over the smores branches of my GitHub before I > create the 5.0.x branch and pull the changes into it, that would be aweso= me. > > > https://na01.safelinks.protection.outlook.com/?url=3Dhttps%3a%2f%2fgithub= .com%2finfil00p%2fcordova-android%2ftree%2fsmores&data=3D01%7c01%7cnikhilkh= %40microsoft.com%7c1785194b1f82494fc2d908d2c2c99f36%7c72f988bf86f141af91ab2= d7cd011db47%7c1&sdata=3D%2fPKmL8KTsz5dnC3A75yMatXLQUnfK0Nv07%2bve4PVcCE%3d > > https://na01.safelinks.protection.outlook.com/?url=3Dhttps%3a%2f%2fgithub= .com%2finfil00p%2fcordova-plugin-geolocation%2ftree%2fsmores&data=3D01%7c01= %7cnikhilkh%40microsoft.com%7c1785194b1f82494fc2d908d2c2c99f36%7c72f988bf86= f141af91ab2d7cd011db47%7c1&sdata=3Do6cLXM4f3kpUGCTlIv65ft8lKv6pc5qbeY%2bdUx= iP4bc%3d > > https://na01.safelinks.protection.outlook.com/?url=3Dhttps%3a%2f%2fgithub= .com%2finfil00p%2fcordova-plugin-camera%2ftree%2fsmores&data=3D01%7c01%7cni= khilkh%40microsoft.com%7c1785194b1f82494fc2d908d2c2c99f36%7c72f988bf86f141a= f91ab2d7cd011db47%7c1&sdata=3DkNsHIv6Uw2ITcT1ABmNq1JCmPTSigCGRb4zWC8maWpE%3= d > > https://na01.safelinks.protection.outlook.com/?url=3Dhttps%3a%2f%2fgithub= .com%2finfil00p%2fcordova-plugin-contacts%2ftree%2fsmores&data=3D01%7c01%7c= nikhilkh%40microsoft.com%7c1785194b1f82494fc2d908d2c2c99f36%7c72f988bf86f14= 1af91ab2d7cd011db47%7c1&sdata=3DrZ%2f1AALPAtUwgSXyOL1uk1b0Y%2feEmqLOdU%2fwu= a2TbLU%3d > > Work on audio is still outstanding, BUT for some reason Audio broke > recently on both Lollipop and Marshmallow. I didn't test it out on KitKa= t > or Jellybean yet, but I'm wondering whether we should keep maintaining th= is > or support the standard HTML5 audio and deal with the asset issue somehow > (which isn't straight forward). > > I hopefully want to get a 5.0.x branch happening this week if we can. > > What do people think? > > Joe > --089e013cc03c20963405204bf8ab--