Crosswalk plugin has already been updated (on master anyways) :) https://github.com/crosswalk-project/cordova-plugin-crosswalk-webview/pull/95
On 8 September 2016 at 14:46, Joe Bowser <bows...@gmail.com> wrote: > Hey > > So, this is exciting. Things are broken on Master if you're trying to use > Cordova with Jellybean. Apparently we forgot to add a failsafe for the > bridge to go back to APIs lower than 19, so the bridge doesn't work if > you're targeting people still on Jellybean. Unfortunately, we still have > to support Jellybean despite the fact that their jsToNativeBridge is > prompt. > > Also, I just realized that I can't send a PR to Crosswalk, since I own the > initial crosswalk repo on GitHub. (Oops!) This means that I need someone > from the Crosswalk project to adopt these changes here to get this to work > on Cordova-Android 6.0.0 > > https://github.com/infil00p/cordova-crosswalk-engine/tree/Crosswalk_Compat > > I'm tempted to nuke the thing in the near future, since the Crosswalk > Project should be managing the repository and their repository should be > the canonical source for everything Crosswalk/Cordova related and not my > prototype repo. :P > > So, is anyone using the stock webview for Jellybean, and should we spend > time fixing this issue? We changed the webview because we found that using > multiple webviews in an application was falling apart, and we needed this > change for stability so we knew which callbacks go to which instance of > webview. However, if people reviewed the patch requests, they would have > noticed that this doesn't work on API 16. :( > > I'm going to go by the assumption that we want this fixed until I hear > otherwise for now. That said, if people think it's reasonable to recommend > that people targeting Jellybean use Crosswalk, I'm cool with that as well. > > Thoughts? > > > > On Tue, Sep 6, 2016 at 11:25 AM, julio cesar sanchez < > jcesarmob...@gmail.com > > wrote: > > > Those 2 lines have minSdkVersion 14 > > > > https://github.com/apache/cordova-android/blob/master/ > > framework/AndroidManifest.xml#L22 > > https://github.com/apache/cordova-android/blob/master/ > > test/AndroidManifest.xml#L48 > > > > Maybe I looked into the wrong place and that is not being used > > > > > > > > El martes, 6 de septiembre de 2016, Joe Bowser <bows...@gmail.com > > <javascript:_e(%7B%7D,'cvml','bows...@gmail.com');>> escribió: > > > > > On Tue, Sep 6, 2016 at 11:14 AM, julio cesar sanchez < > > > jcesarmob...@gmail.com > > > > wrote: > > > > > > > Now we shouldn't forget to increase the minSdkVersion. I couldn't > find > > an > > > > issue open for this. Which API level did we say that we will support? > > 16? > > > > > > > > > > > Yes, we said 16. I already did it when bumping up to API 24, but an > > issue > > > would be nice. > > > > > > > > > > > > > > El martes, 6 de septiembre de 2016, Joe Bowser <bows...@gmail.com> > > > > escribió: > > > > > > > > > Merging the nougat_update code today. Master will now be targeting > > API > > > > 24. > > > > > > > > > > On Fri, Sep 2, 2016 at 11:21 AM, Joe Bowser <bows...@gmail.com > > > > > <javascript:;>> wrote: > > > > > > > > > > > Yeah, I already added android-24 to that line, and now the error > is > > > > even > > > > > > more explicit and it's telling me straight up to install it. I > > need > > > > > > someone to install android-24 on Travis. > > > > > > > > > > > > On Fri, Sep 2, 2016 at 11:19 AM, Jesse <purplecabb...@gmail.com > > > > > <javascript:;>> wrote: > > > > > > > > > > > >> https://github.com/apache/cordova-android/blob/master/.travi > > > s.yml#L9 > > > > > >> > > > > > >> > > > > > >> @purplecabbage > > > > > >> risingj.com > > > > > >> > > > > > >> On Thu, Sep 1, 2016 at 10:59 AM, Joe Bowser <bows...@gmail.com > > > > > <javascript:;>> wrote: > > > > > >> > > > > > >> > BTW: The nougat update has been added, but this broke our CI > > > > > >> > > > > > > >> > https://travis-ci.org/apache/cordova-android/builds/156860827 > > > > > >> > > > > > > >> > Can we get android-24 added to Travis? > > > > > >> > > > > > > >> > > > > > > >> > > > > > > >> > On Thu, Sep 1, 2016 at 8:48 AM, Joe Bowser <bows...@gmail.com > > > > > <javascript:;>> wrote: > > > > > >> > > > > > > >> > > Sounds good! > > > > > >> > > > > > > > >> > > On Thu, Sep 1, 2016 at 8:48 AM, julio cesar sanchez < > > > > > >> > > jcesarmob...@gmail.com <javascript:;>> wrote: > > > > > >> > > > > > > > >> > >> You asked for a reason to NOT do it, so if there is no > answer > > > you > > > > > >> can do > > > > > >> > >> it. > > > > > >> > >> > > > > > >> > >> > > > > > >> > >> > > > > > >> > >> 2016-09-01 17:44 GMT+02:00 Joe Bowser <bows...@gmail.com > > > > > <javascript:;>>: > > > > > >> > >> > > > > > >> > >> > BUMP!!! > > > > > >> > >> > > > > > > >> > >> > This is a major version! Are people getting my e-mails? > > > > > >> > >> > > > > > > >> > >> > On Wed, Aug 31, 2016 at 12:44 PM, Joe Bowser < > > > > bows...@gmail.com > > > > > <javascript:;>> > > > > > >> > wrote: > > > > > >> > >> > > > > > > >> > >> > > Hey > > > > > >> > >> > > > > > > > >> > >> > > This week, I'm going to start merging things that will > > > break > > > > > >> certain > > > > > >> > >> > APIs, > > > > > >> > >> > > such as the Third Party WebView. This is done so we > can > > > have > > > > > >> solid > > > > > >> > >> > > compatibility with Nougat, as well as fix some > threading > > > > issues > > > > > >> that > > > > > >> > >> > people > > > > > >> > >> > > are having when embedding CordovaWebView in native > > > projects. > > > > > >> Before > > > > > >> > >> I do > > > > > >> > >> > > this, I need to know whether there's any reason to NOT > do > > > > this. > > > > > >> > >> > > > > > > > >> > >> > > Does anyone have any objections to me starting to merge > > the > > > > > 6.0.x > > > > > >> > >> changes > > > > > >> > >> > > in and prepping for a 6.0.x release sometime in > September > > > > with > > > > > >> > Cordova > > > > > >> > >> > > targeting API 24, and having compatibility fixes? > > > > > >> > >> > > > > > > > >> > >> > > Joe > > > > > >> > >> > > > > > > > >> > >> > > > > > > >> > >> > > > > > >> > > > > > > > >> > > > > > > > >> > > > > > > >> > > > > > > > > > > > > > > > > > > > > > > > > > > >
