cordova-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Joe Bowser <bows...@gmail.com>
Subject Re: [Discuss] Cordova-Android 6.0.x
Date Mon, 03 Oct 2016 18:30:44 GMT
OK, I recommend starting the vote thread after the Google announcement on
Tuesday, so that I know everyone has read this.

Joe

On Thu, Sep 29, 2016 at 9:59 AM, Joe Bowser <bowserj@gmail.com> wrote:

> OK, so it's been a while.  I think we're ready for a release of this
> soon.  I just have to fix up check_reqs so it works, and then we can move
> forward with the Cordova-Android 6.0.0 release.
>
> Should we start the vote thread on Monday?
>
> On Tue, Sep 13, 2016 at 11:24 AM, Joe Bowser <bowserj@gmail.com> wrote:
>
>> 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 <darryl@dpogue.ca> 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 <bowserj@gmail.com> 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 <shazron@gmail.com> 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 <bowserj@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 <
>>> > > > jcesarmobile@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 <
>>> bowserj@gmail.com
>>> > > > > <javascript:_e(%7B%7D,'cvml','bowserj@gmail.com');>>
escribió:
>>> > > > >
>>> > > > > > On Tue, Sep 6, 2016 at 11:14 AM, julio cesar sanchez
<
>>> > > > > > jcesarmobile@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
<
>>> > bowserj@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
<
>>> bowserj@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
<
>>> > > purplecabbage@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 <
>>> > > bowserj@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 <
>>> > > bowserj@gmail.com
>>> > > > > > > > <javascript:;>> wrote:
>>> > > > > > > > >> >
>>> > > > > > > > >> > > Sounds good!
>>> > > > > > > > >> > >
>>> > > > > > > > >> > > On Thu, Sep 1, 2016 at
8:48 AM, julio cesar sanchez
>>> <
>>> > > > > > > > >> > > jcesarmobile@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 <
>>> > bowserj@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 <
>>> > > > > > > bowserj@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
>>> > > > > > > > >> > >> > >
>>> > > > > > > > >> > >> >
>>> > > > > > > > >> > >>
>>> > > > > > > > >> > >
>>> > > > > > > > >> > >
>>> > > > > > > > >> >
>>> > > > > > > > >>
>>> > > > > > > > >
>>> > > > > > > > >
>>> > > > > > > >
>>> > > > > > >
>>> > > > > >
>>> > > > >
>>> > > >
>>> > >
>>> >
>>>
>>
>>
>

Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message