cordova-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From julio cesar sanchez <jcesarmob...@gmail.com>
Subject Re: Fix crash on cordova-windows@5
Date Thu, 08 Feb 2018 11:57:42 GMT
The release of 6.0.0 is blocked because of a test, so a possible
5.0.1/5.1.0 will probably be blocked for the same reason.

If the 6.0.0 gets unblocked and can be released, is that good for you? or
you need the 5.0.1/5.1.0 for some reason?

2018-02-08 12:42 GMT+01:00 Chris Brody <chris.brody@gmail.com>:

> I was hoping to get this fixed for a customer. The last time I published a
> change on my fork was was outdated pretty quickly. For now I will say that
> they have to use cordova-android@4 for Windows 8.,1 until we get a new
> release with this crash resolved.
>
> Since #247 was already merged into 5.0.x I would be happiest to see it
> released. I do not see how this kind of a fix could confuse anyone. Or am I
> missing something here?
>
> On Thu, Feb 8, 2018 at 6:03 AM, Jan Piotrowski <piotrowski@gmail.com>
> wrote:
>
> > I do not think this change necessitates a 5.0.1 or 5.1 release. Cutting
> > another release before 6.0.0 will just confuse people.
> > Just fork the repo and apply the change on your fork, then use that in
> your
> > app.
> >
> > Although if anyone wants to run the release process, I of course won't
> stop
> > you. (But I will not but focus on 6.0.0).
> >
> > J
> >
> > 2018-02-08 4:44 GMT+01:00 Chris Brody <chris.brody@gmail.com>:
> >
> > > Hello team,
> > >
> > > Windows 8.1 (8.1-win) build crashes in cordova-windows@5.0.0
> (CB-12784 &
> > > CB-13175). It took multiple attempts to fix this on the cordova-windows
> > > master branch which I thought would be released 1-2 weeks ago for
> 5.1.0.
> > I
> > > discovered today that the master branch is now for cordova-windows
> 6.0.0,
> > > blocked by a CI test failure. Very frustrating since I still support a
> > > special sqlite plugin version on Windows 8.1.
> > >
> > > I raised CB-13855 with a proposed fix for cordova-windows@5 in
> > > https://github.com/apache/cordova-windows/pull/247 (tested). As stated
> > in
> > > PR #247 I did not do everything the usual way, purpose is to get this
> > fixed
> > > ASAP.
> > >
> > > Please let me know if there is anything else you need me to do to get
> > this
> > > fix into a new cordova-windows@5 release ASAP. It has been broken for
> > way
> > > too long.
> > >
> > > Thanks and best regards,
> > > Chris
> > >
> > > https://www.linkedin.com/in/chrisbrody/
> > >
> >
>

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