No, I have no intention of pulling that into this release, since the corresponding Android code caused performance issues due to numerous file reads.
On Tue, Sep 13, 2016 at 11:21 AM, Darryl Pogue <[email protected]> wrote: > Just wanted to check whether this cordova-lib change was intended to get > pulled in as part of the Cordova Android 6 release: > https://github.com/apache/cordova-lib/pull/469 > > If it is, I have an additional commit to add that fixes the JS spec tests: > https://github.com/infil00p/cordova-lib/pull/1 > > > On 13 September 2016 at 10:30, Joe Bowser <[email protected]> wrote: > > > OK, so it looks like all our ducks are lined up and we should be ready > for > > a 6.0 release pretty soon. How do people feel about doing testing this > > week, and getting the vote thread going on Monday? I know the vote thread > > is only 72 hours, so I want a bit more testing on the major before we do > it > > because the PRs haven't been getting the attention that I wanted them to. > > > > What do people think? > > > > On Thu, Sep 8, 2016 at 2:54 PM, Shazron <[email protected]> wrote: > > > > > Ask Crosswalk to: > > > "To detach the fork and turn it into a standalone repository on GitHub, > > > contact GitHub Support." > > > > > > From: > > > https://help.github.com/articles/why-are-my- > > contributions-not-showing-up- > > > on-my-profile/ > > > > > > > > > > > > On Thu, Sep 8, 2016 at 2:46 PM, Joe Bowser <[email protected]> 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 < > > > > [email protected] > > > > > 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 <[email protected] > > > > > <javascript:_e(%7B%7D,'cvml','[email protected]');>> escribió: > > > > > > > > > > > On Tue, Sep 6, 2016 at 11:14 AM, julio cesar sanchez < > > > > > > [email protected] > > > > > > > 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 < > > [email protected]> > > > > > > > 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 < > [email protected] > > > > > > > > <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 < > > > [email protected] > > > > > > > > <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 < > > > [email protected] > > > > > > > > <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 < > > > [email protected] > > > > > > > > <javascript:;>> wrote: > > > > > > > > >> > > > > > > > > > >> > > Sounds good! > > > > > > > > >> > > > > > > > > > > >> > > On Thu, Sep 1, 2016 at 8:48 AM, julio cesar sanchez < > > > > > > > > >> > > [email protected] <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 < > > [email protected] > > > > > > > > <javascript:;>>: > > > > > > > > >> > >> > > > > > > > > >> > >> > BUMP!!! > > > > > > > > >> > >> > > > > > > > > > >> > >> > This is a major version! Are people getting my > > e-mails? > > > > > > > > >> > >> > > > > > > > > > >> > >> > On Wed, Aug 31, 2016 at 12:44 PM, Joe Bowser < > > > > > > > [email protected] > > > > > > > > <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 > > > > > > > > >> > >> > > > > > > > > > > >> > >> > > > > > > > > > >> > >> > > > > > > > > >> > > > > > > > > > > >> > > > > > > > > > > >> > > > > > > > > > >> > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > >
