That'd be great!

The PRs are ready: https://github.com/apache/cordova-lib/pull/437 and 
https://github.com/apache/cordova-ios/pull/221 
Although the PR for cordova-lib is still failing w/ Node 6.0, this looks like a 
setup issue (the failure reason is jshint internals) and I can't reproduce 
these failures on my machine. Also the tests for cordova-lib itself are passing 
so it should be fine.

-
Best regards, Vladimir 

-----Original Message-----
From: Steven Gill [mailto:stevengil...@gmail.com] 
Sent: Saturday, May 7, 2016 12:19 AM
To: dev@cordova.apache.org
Subject: RE: Node 6.x and Cordova CLI

Sweet. Thanks for looking into it! I can start release process for both next 
week On May 6, 2016 2:13 PM, "Vladimir Kotikov (Akvelon)" 
<v-vlk...@microsoft.com>
wrote:

> Hi list
>
> FYI, we’ve recently performed a sanity check of Cordova CLI against 
> newly released Node JS 6.0 (credits to Alex Sorokin) and found some 
> issues, caused by new Node JS version. The most critical is a failure 
> of uninstallation of plugins with custom frameworks for iOS platform - 
> JIRA
> CB-11200 [1]. The root cause is the bug in node-xcode which is already 
> fixed [2].
>
> In order to resolve the issue I think we'll need to bump node-xcode 
> version in both cordova-lib and cordova-ios and release these ASAP. 
> Also we'd probably need to update CI configs to include Node 6.x in 
> test matrix where possible. I'm going to submit a PR shortly.
>
> -
> [1] 
> https://na01.safelinks.protection.outlook.com/?url=https%3a%2f%2fissue
> s.apache.org%2fjira%2fbrowse%2fCB-11200&data=01%7c01%7cv-vlkoti%40micr
> osoft.com%7cc34ccb842dac42d6a7ac08d375f4196e%7c72f988bf86f141af91ab2d7
> cd011db47%7c1&sdata=AY47SzHGi2reQtjJRbYU%2b%2fNaKi25qdzw%2fIRW291o12E%
> 3d [2] https://github.com/alunny/node-xcode/pull/86
>
> -
> Best regards, Vladimir
>

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

Reply via email to