[DISCUSS] Mark cordova-windows 6.0.0-dev

2018-02-07 Thread Chris Brody
Hello team,

I found out today that the cordova-windows master branch is now targeting
6.0.0 due to some breaking changes ref: http://markmail.org/
message/rzfa2zoydyjq6xso

Can someone please mark this in the cordova-windows master branch?

I think this could help avoid some possible forms of confusion moving
forward.

Thanks and best regards,
Chris

https://www.linkedin.com/in/chrisbrody/


Re: [DISCUSS] Mark cordova-windows 6.0.0-dev

2018-02-07 Thread Shazron
Do you mean a README update or something else?

On Thu, Feb 8, 2018 at 10:59 AM, Chris Brody  wrote:

> Hello team,
>
> I found out today that the cordova-windows master branch is now targeting
> 6.0.0 due to some breaking changes ref: http://markmail.org/
> message/rzfa2zoydyjq6xso
>
> Can someone please mark this in the cordova-windows master branch?
>
> I think this could help avoid some possible forms of confusion moving
> forward.
>
> Thanks and best regards,
> Chris
>
> https://www.linkedin.com/in/chrisbrody/
>


Re: [DISCUSS] Cordova-Windows 6.0.0

2018-02-07 Thread Jan Piotrowski
Ok, so #235 and #246 are merged into `master` and we are down to one
failing test case https://issues.apache.org/jira/browse/CB-13834

The test is failing with Visual Studio 2017 and I can't reproduce locally,
but it happens consistently on AppVeyor CI.
If anyone here has some time, it would be really helpful if you could check
out `master` and see if you can reproduce.

(Test runs of last commit to master can be seen here:
https://ci.appveyor.com/project/Humbedooh/cordova-windows/build/1.0.854)

Best,
Jan

2018-01-26 1:42 GMT+01:00 Jan Piotrowski :

> The failures in https://github.com/apache/cordova-windows/pull/235 are
> unrelated to that PR. Those are caused by the "Musts": If
> https://github.com/apache/cordova-windows/pull/246 gets merged this
> will be down to one, and the last one is
> https://issues.apache.org/jira/browse/CB-13834
>
> -J
>
> 2018-01-26 1:12 GMT+01:00 Jesse :
> > Yeah, I agree that dumping 8.1 completely would be a lot of work, let
> move
> > forward without that part.
> >
> > I think I agree with all your assessments!
> >
> > For the maybes:
> > I think CB-13817  is
> less
> > important, as it is something that even MS allows. They changed to call
> all
> > UAP => UWP but they still have docs that say UAP.
> > I would like to see https://github.com/apache/cordova-windows/pull/235
> make
> > it in, we'll need to look into the failures.
> >
> >
> >
> >
> > @purplecabbage
> > risingj.com
> >
> > On Thu, Jan 25, 2018 at 3:54 PM, Jan Piotrowski 
> > wrote:
> >
> >> Hello.
> >>
> >> After we had to scrap release 5.1.0 because of an unexpected breaking
> >> change (and broken CI and failing tests), I took the time to work a
> >> bit on this and am now planning to go forward with a 6.0.0 release of
> >> cordova-windows soonish.
> >>
> >> Current status:
> >>
> >> Must
> >> - https://github.com/apache/cordova-windows/pull/246 (of
> >> https://issues.apache.org/jira/browse/CB-13829) has to be reviewed and
> >> merged
> >> - https://issues.apache.org/jira/browse/CB-13834 has to be analyzed,
> >> fixed, reviewed and merged
> >>
> >> Maybe
> >> - https://issues.apache.org/jira/browse/CB-13817 could implement "UWP"
> >> everywhere we currently use "UAP" (which will be kept as an alias)
> >> - https://github.com/apache/cordova-windows/pull/235 should be
> >> reviewed and merged as it is a nice new feature for Windows
> >>
> >> After
> >> - Later https://github.com/apache/cordova-docs/pull/785 needs some
> >> more work so docs at least partially match reality when 6.0.0 will be
> >> released
> >>
> >> Am I missing anything?
> >>
> >> In the last release email Jesse suggested dropping 8.1 alltogether.
> >> After spending some time with the code this would a) be super nice as
> >> s much stuff could be deleted and removed but also b) pretty hard
> >> as it is all connected and interwoven. Won't be quick to do.
> >> More importantly I think we should use the release blog post to ask
> >> cordova-windows users if 8.1 is still needed for them and how they are
> >> using it in general so we get some input on how to proceed. (Some
> >> feedback from Microsoft wouldn't be the worst idea either) If no one
> >> uses it, we can do a clean 7.0.0 that removes it.
> >>
> >> Best,
> >> Jan
> >>
> >> -
> >> To unsubscribe, e-mail: dev-unsubscr...@cordova.apache.org
> >> For additional commands, e-mail: dev-h...@cordova.apache.org
> >>
> >>
>


Fix crash on cordova-windows@5

2018-02-07 Thread Chris Brody
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/


Nightly build #626 for cordova has succeeded!

2018-02-07 Thread Apache Jenkins Server
Nightly build #626 for cordova has succeeded!
The latest nightly has been published and you can try it out with 'npm i -g 
cordova@nightly'

For details check build console at 
https://builds.apache.org/job/cordova-nightly/626/consoleFull

-
Jenkins for Apache Cordova

-
To unsubscribe, e-mail: dev-unsubscr...@cordova.apache.org
For additional commands, e-mail: dev-h...@cordova.apache.org

Re: [DISCUSS] Mark cordova-windows 6.0.0-dev

2018-02-07 Thread Chris Brody
I meant something else. Change like the following commits:

   -
   
https://github.com/apache/cordova-windows/commit/34d1335187ac21f21061c638fd54364fa3afb422
   -
   
https://github.com/apache/cordova-windows/commit/ece4d0ffe16a0962c3c5d51c2289582f3d3238c9

but change version / VERSION value to 6.0.0-dev.

On Wed, Feb 7, 2018 at 10:10 PM, Shazron  wrote:

> Do you mean a README update or something else?
>
> On Thu, Feb 8, 2018 at 10:59 AM, Chris Brody 
> wrote:
>
> > Hello team,
> >
> > I found out today that the cordova-windows master branch is now targeting
> > 6.0.0 due to some breaking changes ref: http://markmail.org/
> > message/rzfa2zoydyjq6xso
> >
> > Can someone please mark this in the cordova-windows master branch?
> >
> > I think this could help avoid some possible forms of confusion moving
> > forward.
> >
> > Thanks and best regards,
> > Chris
> >
> > https://www.linkedin.com/in/chrisbrody/
> >
>


Re: [DISCUSS] Mark cordova-windows 6.0.0-dev

2018-02-07 Thread Jesse
Hi Chris,

Are you okay if we go ahead and pull in https://github.com/apache/
cordova-windows/pull/247 rebase those changes into master also, and do a
6.0.0 release ( leaving windows 8.1 support in ) ?




@purplecabbage
risingj.com

On Wed, Feb 7, 2018 at 7:18 PM, Chris Brody  wrote:

> I meant something else. Change like the following commits:
>
>-
>https://github.com/apache/cordova-windows/commit/
> 34d1335187ac21f21061c638fd54364fa3afb422
>-
>https://github.com/apache/cordova-windows/commit/
> ece4d0ffe16a0962c3c5d51c2289582f3d3238c9
>
> but change version / VERSION value to 6.0.0-dev.
>
> On Wed, Feb 7, 2018 at 10:10 PM, Shazron  wrote:
>
> > Do you mean a README update or something else?
> >
> > On Thu, Feb 8, 2018 at 10:59 AM, Chris Brody 
> > wrote:
> >
> > > Hello team,
> > >
> > > I found out today that the cordova-windows master branch is now
> targeting
> > > 6.0.0 due to some breaking changes ref: http://markmail.org/
> > > message/rzfa2zoydyjq6xso
> > >
> > > Can someone please mark this in the cordova-windows master branch?
> > >
> > > I think this could help avoid some possible forms of confusion moving
> > > forward.
> > >
> > > Thanks and best regards,
> > > Chris
> > >
> > > https://www.linkedin.com/in/chrisbrody/
> > >
> >
>