[GitHub] cordova-plugin-media-capture pull request: CB-6978 captureImage() ...

2014-06-26 Thread mbektchiev
GitHub user mbektchiev opened a pull request: https://github.com/apache/cordova-plugin-media-capture/pull/20 CB-6978 captureImage() function fails in Android The reason is that the Intent which should write the file has no permissions in the application's cache directory by

[GitHub] cordova-lib pull request: disables CB-6976 Add support for Windows...

2014-06-26 Thread sgrebnov
GitHub user sgrebnov opened a pull request: https://github.com/apache/cordova-lib/pull/41 disables CB-6976 Add support for Windows Universal apps (Windows 8.1 and WP 8.1) We should not include this functionality until new Windows platform is released. It is better to enable this

[GitHub] cordova-cli pull request: disables CB-6976 Add support for Windows...

2014-06-26 Thread sgrebnov
GitHub user sgrebnov opened a pull request: https://github.com/apache/cordova-cli/pull/182 disables CB-6976 Add support for Windows Universal apps (Windows 8.1 and WP 8.1) We should not include this functionality until new Windows platform is released. It is better to enable this

CB-7031 --usenpm option should place downloaded package to util.libDirectory

2014-06-26 Thread Sergey Grebnov (Akvelon)
Tested -usenpm option, found out the following issue: https://issues.apache.org/jira/browse/CB-7031 Due to this there are the following two problems (if use this oprion): 1. Requirements check failed when adding new platform (if it is not cached) cordova platform add wp8 --usenpm CordovaError:

RE: [Vote] Tools Release

2014-06-26 Thread Sergey Grebnov (Akvelon)
For the Windows Universal Apps feature we need to co-release Tools and Windows platform to get it working. Since this is a 3.5.0-minor release people who already downloaded windows platform won't receive updated template and this will break functionality. I've sent the following PRs to disable

Re: [Vote] Tools Release

2014-06-26 Thread Andrew Grieve
Sergey - could you clarify what the bug here is? E.g. Is there a regression? Does this make cordova run not work? If it really does need to be applied - checking in commented-out code is not the way to go about. You should add if (version = 3.6.0) checks around things like is done here:

[GitHub] cordova-medic pull request: CB-6869 medic.json file not created fo...

2014-06-26 Thread vladimir-kotikov
Github user vladimir-kotikov closed the pull request at: https://github.com/apache/cordova-medic/pull/8 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the

Medic improvements for review

2014-06-26 Thread Sergey Grebnov
Can someone review the following PRs? They are required to properly work when slave is Windows and master is OSX (and vice versa). CB-6889, CB-6899, CB-6909 Multiple fixes in master.cfg to ensure proper work on windows slaves https://github.com/apache/cordova-medic/pull/12 CB-6846 Fixes

Chrome App Developer Tool for Mobile

2014-06-26 Thread Andrew Grieve
Chrome team has a new toy if anyone wants to try it out! Chrome App Developer Tool is a downstream version of cordova-app-harness Any mobile Chrome App or Cordova app that uses CCA plugins or core Cordova plugins should work with it. Instructions are here:

Re: [VOTE] Network information plugin release

2014-06-26 Thread Ian Clelland
Has anyone been able to verify this package? On Tue, Jun 24, 2014 at 3:55 PM, Ian Clelland iclell...@chromium.org wrote: Please review and vote on the release of this plugins release. Release issue: https://issues.apache.org/jira/browse/CB-7019 The plugins have been published to dist/dev:

Re: [VOTE] Network information plugin release

2014-06-26 Thread Andrew Grieve
+1 * verified signature hash * verified archive contents matches git hash On Tue, Jun 24, 2014 at 3:55 PM, Ian Clelland iclell...@chromium.org wrote: Please review and vote on the release of this plugins release. Release issue: https://issues.apache.org/jira/browse/CB-7019 The plugins

Device API w3c Working Group Response to Cordova feedback on Battery

2014-06-26 Thread Lisa Seacat DeLuca
Cordova Team~ This morning on the Device API's w3c Working Group call we discussed the battery implementation. I had shared with the team some of the concerns some of the Cordova team had around battery as discussed in this thread:

RE: [Vote] Tools Release

2014-06-26 Thread Sergey Grebnov (Akvelon)
Example is 'cordova platform add windows8' will fail since updated code expects check_reqs in different folder ('\windows\bin\', instead of '\windows8\bin\' as it is in old template). There are another issues. In this situation I propose to revert the entire original commit. Thoughts? CB-6976

Re: [Vote] Tools Release

2014-06-26 Thread Mark Koudritsky
Ok, basically this means that this release would not work on windows. Canceling the vote. VOTE IS CANCELLED

[GitHub] cordova-coho pull request: Add tweaks for releasing cordova-lib

2014-06-26 Thread agrieve
Github user agrieve commented on the pull request: https://github.com/apache/cordova-coho/pull/31#issuecomment-47266828 LGTM --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature

[GitHub] cordova-blackberry pull request: CB-7036 make npm test work again ...

2014-06-26 Thread jsoref
GitHub user jsoref opened a pull request: https://github.com/apache/cordova-blackberry/pull/165 CB-7036 make npm test work again (using grunt instead of jake) Apparently there used to be support for `npm test`, it's been broken for a while because `cordova-blackberry` switched

[CANCELLED][VOTE] Tools release

2014-06-26 Thread Mark Koudritsky
Canceling this vote. This version would fail on windows without the latest windows platform version which is expected to be released with 3.6. On Thu, Jun 26, 2014 at 2:43 PM, Sergey Grebnov (Akvelon) v-seg...@microsoft.com wrote: Example is 'cordova platform add windows8' will fail since

[GitHub] cordova-blackberry pull request: CB-7036 make npm test work again ...

2014-06-26 Thread dylin
Github user dylin commented on the pull request: https://github.com/apache/cordova-blackberry/pull/165#issuecomment-47268188 r+ --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature

[GitHub] cordova-blackberry pull request: CB-7036 make npm test work again ...

2014-06-26 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/cordova-blackberry/pull/165 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the

Google Wear SDK

2014-06-26 Thread Ross Gerbasi
Just running some test with the new Google Wear SDK. We can now make android apps for wear just like any other app but it seems like we have been shorted on the WebView. Doesn't looks its actually there. I am seeing the following runtime error. *E/WebViewFactory﹕ Chromium WebView does not

Re: Google Wear SDK

2014-06-26 Thread Joe Bowser
I'm just now installing L. I'll probably mess around with it a bit, but since Protesters Cordova devs, I wasn't at IO so I didn't get any hardware. I expected the WebView to not be installed on the watch, since it takes a lot of power. Aside, I really wish there was a Chromium or WebKit based

Re: Google Wear SDK

2014-06-26 Thread Joe Bowser
On Thu, Jun 26, 2014 at 1:39 PM, Joe Bowser bows...@gmail.com wrote: I'm just now installing L. I'll probably mess around with it a bit, but since Protesters Cordova devs, I wasn't at IO so I didn't get any hardware. I expected the WebView to not be installed on the watch, since it takes a

RE: [CANCELLED][VOTE] Tools release

2014-06-26 Thread Parashuram Narasimhan (MS OPEN TECH)
Sorry for having to cancel the vote. We are working on a fix for this and will be getting the fix for this soon. -Original Message- From: Mark Koudritsky [mailto:kam...@google.com] Sent: Thursday, June 26, 2014 12:14 PM To: dev@cordova.apache.org Subject: [CANCELLED][VOTE] Tools

Re: Google Wear SDK

2014-06-26 Thread Ross Gerbasi
haha maybe a joke with some undertones of truthiness :) I ordered a LG G like a fool because I also was not there and I am addicted to gadgets. I have a Nexus 7 I am not horribly concerned about whats running on it so I think it will get the L image and see how that goes. I will also be mucking

Re: Google Wear SDK

2014-06-26 Thread Ajai Khattri
On Thu, Jun 26, 2014 at 03:34:50PM -0500, Ross Gerbasi wrote: Just running some test with the new Google Wear SDK. We can now make android apps for wear just like any other app but it seems like we have been shorted on the WebView. Doesn't looks its actually there. I am seeing the following

[GitHub] cordova-lib pull request: CB-7037 platform check doesn't warn when...

2014-06-26 Thread jsoref
GitHub user jsoref opened a pull request: https://github.com/apache/cordova-lib/pull/42 CB-7037 platform check doesn't warn when it can't install a platform cordova platform check will now report when: * it can't create a platform * it can't run a new platform version

Re: Cordova version in a created project

2014-06-26 Thread Josh Soref
https://issues.apache.org/jira/browse/CB-7037 with a pull request Victor: please try this and see if it is more helpful than the current platform check behavior. I'm open to suggestions about the wording. I'm not particularly interested in spitting out the underlying error message from things

Re: How to initialize a Cordova project using CLI if I only have the www directory of the project?

2014-06-26 Thread German Viscuso
Thank you guys! :D On Wed, Jun 25, 2014 at 5:00 PM, Max Woghiren m...@chromium.org wrote: Gorkem beat me to it. But `cordova help` has a whole bunch of info. :) On Wed, Jun 25, 2014 at 10:56 AM, Gorkem Ercan gorkem.er...@gmail.com wrote: see

[GitHub] cordova-firefoxos pull request: Update Readme

2014-06-26 Thread rodms10
GitHub user rodms10 opened a pull request: https://github.com/apache/cordova-firefoxos/pull/17 Update Readme New instructions. You can merge this pull request into a Git repository by running: $ git pull https://github.com/rodms10/cordova-firefoxos updateReadme Alternatively

[GitHub] cordova-firefoxos pull request: Update Readme

2014-06-26 Thread rodms10
Github user rodms10 commented on the pull request: https://github.com/apache/cordova-firefoxos/pull/17#issuecomment-47286219 This fixes CB-6893. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not

[GitHub] cordova-blackberry pull request: CB-6492 AppData hint is wrong for...

2014-06-26 Thread jsoref
GitHub user jsoref opened a pull request: https://github.com/apache/cordova-blackberry/pull/166 CB-6492 AppData hint is wrong for wXP Check for LOCALAPPDATA environment variable (which is what Windows Vista introduced) Otherwise, use the hard coded value for XP/older You can

platform_parser.check_requirements() not needed?

2014-06-26 Thread Andrew Grieve
Android and iOS have empty functions for check_requirements(), and I think that's the right way to go. Never really discussed it on list though, so wanted to share why I think this. Originally, the motivation was to run some checks *before* downloading the platform. It turned out that

Re: platform_parser.check_requirements() not needed?

2014-06-26 Thread Josh Soref
Andrew wrote: Android and iOS have empty functions for check_requirements(), and I think that's the right way to go. Never really discussed it on list though, so wanted to share why I think this. the motivation was to run some checks *before* downloading the platform. At one point,

Re: platform_parser.check_requirements() not needed?

2014-06-26 Thread Carlos Santana
+1 I think was the agreement last time was discussed. The goal should be to delegate platform checks to the individual platforms. Same goes for other platform commands the platform should check. For example ios create, the create command should check for the minimum requirements for create.