Re: How to Implement native features in Cordova

2015-01-21 Thread julio cesar sanchez
As the media plugin doesn't follow the W3C specification for media capture, it might be included in the plugin, but you should check if this is possible on other platforms too. You can file an issue on the cordova JIRA ( https://issues.apache.org/jira/browse/CB) as enhancement to the media plugin p

Re: Remove CDVJson.h from Copy Headers?

2015-01-21 Thread Shazron
Sure, let's create the 4.x branch On Wed, Jan 21, 2015 at 8:03 PM, Andrew Grieve wrote: > I've fixed InAppBrowser to call NSJSONSerializer directly, but I'm > certainly leaning towards the change to rename the symbol to be reverted > and re-applied in a 4.0.x branch. Shall I create one? > > On W

Re: Remove CDVJson.h from Copy Headers?

2015-01-21 Thread Andrew Grieve
I've fixed InAppBrowser to call NSJSONSerializer directly, but I'm certainly leaning towards the change to rename the symbol to be reverted and re-applied in a 4.0.x branch. Shall I create one? On Wed, Jan 21, 2015 at 1:40 PM, Shazron wrote: > Sounds good to me to remove it as a public header -

How to Implement native features in Cordova

2015-01-21 Thread Jeremy Plack
Hi, I'm new to Cordova and am interested in adding a native android feature. http://developer.android.com/reference/android/media/MediaMetadataRetriever.html This looks to me like it would be an improvement to the media plugin, or should this be in its own plugin? how would I go about integrati

[GitHub] cordova-lib pull request: Creating PR to test against travis / app...

2015-01-21 Thread agrieve
GitHub user agrieve opened a pull request: https://github.com/apache/cordova-lib/pull/152 Creating PR to test against travis / appveyor You can merge this pull request into a Git repository by running: $ git pull https://github.com/agrieve/cordova-lib plugin-info Alternativel

Re: Crowdin and translation process

2015-01-21 Thread Victor Sosa
Hi Andrey, sorry my late replay. Here my 2 cents. This is what the current translation process does: 1. The script downloads the latest updates to your local Got repo. 2. Adds the docs in English to CrowdIn. 3. Manually start the translation via a machine in CrowdIn 4. Once translated, manually r

RE: cordova-android 4.0 JUnit tests

2015-01-21 Thread Fu, Junwei
Thank you so much for supporting positively Crosswalk into JUnit tests. -Original Message- From: agri...@google.com [mailto:agri...@google.com] On Behalf Of Andrew Grieve Sent: Wednesday, January 21, 2015 10:46 PM To: dev Subject: Re: cordova-android 4.0 JUnit tests On Wed, Jan 21, 2015 a

Problems Bundling Large BIN files with a Plugin

2015-01-21 Thread Lyle Pratt
Howdy! I am running into an issue bundling large BIN files needed in a plugin. Mainly the problem is that Github (and most other public GIT repositories) have file size limits. It really seems like there should be a way to link to an externally hosted file in the plugin.xml, or to AT LEAST be able

RE: Cordova --list option implementation

2015-01-21 Thread Josh Soref
Oh, that problem. Yes, this is one of the areas where github is really deficient. I'm not saying I want reviewboard (I didn't like that either). What I ended up doing is this: https://github.com/w3c/mediacapture-main/pull/49#issuecomment-68760377 each version where people reviewed grows an extr

Re: Usage of jscs & JSHint in Cordova

2015-01-21 Thread Mark Koudritsky
On Wed, Jan 21, 2015 at 5:47 PM, Murat Sutunc wrote: > I can take a look at applying current rules in all subfolders soon if time > constraints were the reason. > Will be greatly appreciated. Though I would advise to start from something other than then specs folder for your own sanity :) > I s

RE: Usage of jscs & JSHint in Cordova

2015-01-21 Thread Murat Sutunc
I can take a look at applying current rules in all subfolders soon if time constraints were the reason. I suspected that real time jshinting was the reason for having jshint rules in individual files but by doing so we lose track of un-hinted files and hint standardization. It would be nice to

Re: Cordova/PhoneGap Local Install

2015-01-21 Thread Staci Cooper
The use case I was initially addressing with that utility was a developer wanting multiple versions of Cordova on one machine. In that case it makes more sense to use multiple local installations than to install globally. On Wed, Jan 21, 2015 at 4:39 PM, Ross Gerbasi wrote: > Very helpful inform

Re: Cordova/PhoneGap Local Install

2015-01-21 Thread Ross Gerbasi
Very helpful information, as for why you would want to do this, I was more just bringing out the question of Is this something even worth doing? The gulp workflow looks great, I like the simplicity and control there. As mentioned there would still be shared global settings and cache, is there real

Re: Cordova/PhoneGap Local Install

2015-01-21 Thread Staci Cooper
I actually just proposed something like this a while ago: http://markmail.org/message/avdnqgylaqhwczzs . I have a utility implemented here: https://github.com/stacic/cordova-utility It's similar to grunt-cli. There was some question in that thread of

Re: Cordova/PhoneGap Local Install

2015-01-21 Thread Tommy Williams
The desire to pin the CLI is probably just legacy from when the CLI was tied more tightly to the platform versions. On 22/01/2015 8:17 am, "Michal Mocny" wrote: > ..also regarding "create" not working for current folder.. the only > requirements are that the current folder has the right directory

Re: Cordova/PhoneGap Local Install

2015-01-21 Thread Michal Mocny
..also regarding "create" not working for current folder.. the only requirements are that the current folder has the right directory structure. Namely, if you already have a www/ folder and config.xml file with a valid format, you can just "cordova platform add .." and get to work. So, you can: -

Re: Cordova/PhoneGap Local Install

2015-01-21 Thread Michal Mocny
There was also a recent mailing list thread here about a cordova-local utility someone wrote, which supports installing cordova-cli's locally within projects and calling into them automatically from a wrapper. I haven't tried it, but it could be a more similar workflow to the standard cordova-cli

Re: Usage of jscs & JSHint in Cordova

2015-01-21 Thread Mark Koudritsky
I added he jscs file in cordova-lib some time ago but didn't enforce it in npm test because that required more work in the code to get rid of the remaining style errors. Would be glad if someone invests the effort to make the code comply with the current (or modified) rules. Similar story for the

Re: Cordova/PhoneGap Local Install

2015-01-21 Thread Mark Koudritsky
The best known example of how to use local installs of command line tools is probably Grunt, a globally installed binary will always look for and invoke a local installation. I'm also experimenting with a Gulp based workflow for Cordova which uses local installs of cordova-lib and cordova-android/i

[GitHub] cordova-lib pull request: CB-8239 Add support for git urls to 'cor...

2015-01-21 Thread omefire
Github user omefire commented on a diff in the pull request: https://github.com/apache/cordova-lib/pull/148#discussion_r23330749 --- Diff: cordova-lib/spec-cordova/util.spec.js --- @@ -183,4 +184,138 @@ describe('util module', function() { expect(res.indexOf('CVS'))

[GitHub] cordova-lib pull request: CB-8239 Add support for git urls to 'cor...

2015-01-21 Thread omefire
Github user omefire commented on a diff in the pull request: https://github.com/apache/cordova-lib/pull/148#discussion_r23330721 --- Diff: cordova-lib/src/cordova/lazy_load.js --- @@ -284,4 +286,47 @@ function custom(platforms, platform) { }); } +// Returns a

Usage of jscs & JSHint in Cordova

2015-01-21 Thread Murat Sutunc
Hi, I was curious about the jscs and jshint usage in cordova repos. Seems like in cordova-lib we currently have: - jscs rules in cordova-lib\cordova-lib - individual jshint rules on some js files under cordova-lib\cordova-lib - While jscs rules are set, it's currently not plu

Cordova/PhoneGap Local Install

2015-01-21 Thread Ross Gerbasi
Hey gang, I have been poking around with trying to come up with a workflow that does not require installing cordova or phonegap globally. I haven't been able to come up with anything and I wanted to see if i am missing something, was this ever a goal, should it be a goal we work to add into the pa

Re: MDM solution request

2015-01-21 Thread Hazem Saleh
Hi Vikash, You can check the Cordova documentation: http://cordova.apache.org/docs/en/4.0.0/ If you want a fast presentation about how to use Apache Cordova, you can check my presentation in the previous ApacheCon: http://www.slideshare.net/Hazems/16-9-hazemscordova There also books you can read

MDM solution request

2015-01-21 Thread vikash singh
Dear Sir, I want to develop MDM solution using apache cordova for android device. It is possible using apache cordova or not. if yes. what are the steps. Please suggest me.. Than You. -- *With Regards* *Vikash Singh* *Voice: +91 9739645074* *Email: sn.vikas...@gmail.com * *S

[GitHub] cordova-plugins pull request: CB-8337: Fix keyboardShrinksView for...

2015-01-21 Thread cjpearson
GitHub user cjpearson opened a pull request: https://github.com/apache/cordova-plugins/pull/18 CB-8337: Fix keyboardShrinksView for external keyboards, CB-8339 enable keyboardShrinksView on iOS 7.1+ Instead of listening for keyboard show and hide events, the plugin now listens for

Re: Remove CDVJson.h from Copy Headers?

2015-01-21 Thread Shazron
Sounds good to me to remove it as a public header - we never documented support for it for use. Since it is kinda public already, we should push this for 4.0 because of CDVInAppBrowser (although if we want to do it now, I suppose we could go the icky way and detect the existence of either the pre

[GitHub] cordova-plugin-geolocation pull request: Fixed returnLocationError...

2015-01-21 Thread probalroy
Github user probalroy commented on the pull request: https://github.com/apache/cordova-plugin-geolocation/pull/35#issuecomment-70896886 geolocation.getCurrentPosition giving error "position retrieval timed out" in phone gap app for ios. I am using ios version 8.1. IOS version 7.1 was

[VOTE] 3.7.0 Android Platform release

2015-01-21 Thread Joe Bowser
Please review and vote on this 3.7.0 Android Platform Release. Release issue: https://issues.apache.org/jira/browse/CB-8335 Repos ready to be released have been published to dist/dev:https://dist.apache.org/repos/dist/dev/cordova/CB-8335/ The packages were published from their corresponding git

Remove CDVJson.h from Copy Headers?

2015-01-21 Thread Andrew Grieve
RE: https://github.com/apache/cordova-ios/commit/a98144b8ad63fe714c9f6d274bc9fd98b62575e7 issue: https://issues.apache.org/jira/browse/CB-8270 We had an unprefixed extension, and prefixing it is a breaking change (e.g. CDVInAppBrowser was using it). It really is the right thing to do though... We

[GitHub] cordova-plugin-media pull request: Fixed the problem with multiple...

2015-01-21 Thread lukasT
Github user lukasT commented on the pull request: https://github.com/apache/cordova-plugin-media/pull/33#issuecomment-70873889 +1 --- 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-wp8 pull request: Use WP8 thread pool for each command, in...

2015-01-21 Thread brodybits
Github user brodybits commented on the pull request: https://github.com/apache/cordova-wp8/pull/72#issuecomment-70862219 I just sent the Individual CLA to Adobe (and forwarded a copy to your e-mail address). Glad you like it! --- If your project is set up for it, you can reply to thi

Re: Adding ability to add any platform on any OS

2015-01-21 Thread Andrew Grieve
Reviewed! All minor comments. Great job! Let me know when you've addressed them and I'll pull it in (unless anyone else wants to have a look). On Wed, Jan 21, 2015 at 9:31 AM, Vladimir Kotikov (Akvelon) < v-vlk...@microsoft.com> wrote: > Send a PR for this: https://github.com/apache/cordova-ios/p

[GitHub] cordova-ios pull request: CB-8197 Switch to nodejs for ios platfor...

2015-01-21 Thread agrieve
Github user agrieve commented on a diff in the pull request: https://github.com/apache/cordova-ios/pull/126#discussion_r23305082 --- Diff: bin/templates/scripts/cordova/lib/run.js --- @@ -0,0 +1,184 @@ +/* + Licensed to the Apache Software Foundation (ASF) under one

[GitHub] cordova-ios pull request: CB-8197 Switch to nodejs for ios platfor...

2015-01-21 Thread agrieve
Github user agrieve commented on a diff in the pull request: https://github.com/apache/cordova-ios/pull/126#discussion_r23305562 --- Diff: bin/templates/scripts/cordova/lib/run.js --- @@ -0,0 +1,184 @@ +/* + Licensed to the Apache Software Foundation (ASF) under one

[GitHub] cordova-ios pull request: CB-8197 Switch to nodejs for ios platfor...

2015-01-21 Thread agrieve
Github user agrieve commented on a diff in the pull request: https://github.com/apache/cordova-ios/pull/126#discussion_r23305406 --- Diff: bin/templates/scripts/cordova/lib/run.js --- @@ -0,0 +1,184 @@ +/* + Licensed to the Apache Software Foundation (ASF) under one

[GitHub] cordova-wp8 pull request: Use WP8 thread pool for each command, in...

2015-01-21 Thread sgrebnov
Github user sgrebnov commented on the pull request: https://github.com/apache/cordova-wp8/pull/72#issuecomment-70854793 looks great! yes, please submit Individual CLA and I'll merge this http://www.apache.org/licenses/icla.txt --- If your project is set up for it, you can reply t

[GitHub] cordova-ios pull request: CB-8197 Switch to nodejs for ios platfor...

2015-01-21 Thread agrieve
Github user agrieve commented on a diff in the pull request: https://github.com/apache/cordova-ios/pull/126#discussion_r23305293 --- Diff: bin/templates/scripts/cordova/lib/run.js --- @@ -0,0 +1,184 @@ +/* + Licensed to the Apache Software Foundation (ASF) under one

[GitHub] cordova-ios pull request: CB-8197 Switch to nodejs for ios platfor...

2015-01-21 Thread agrieve
Github user agrieve commented on a diff in the pull request: https://github.com/apache/cordova-ios/pull/126#discussion_r23305135 --- Diff: bin/templates/scripts/cordova/lib/run.js --- @@ -0,0 +1,184 @@ +/* + Licensed to the Apache Software Foundation (ASF) under one

[GitHub] cordova-ios pull request: CB-8197 Switch to nodejs for ios platfor...

2015-01-21 Thread agrieve
Github user agrieve commented on a diff in the pull request: https://github.com/apache/cordova-ios/pull/126#discussion_r23304929 --- Diff: bin/templates/scripts/cordova/lib/run.js --- @@ -0,0 +1,184 @@ +/* + Licensed to the Apache Software Foundation (ASF) under one

[GitHub] cordova-ios pull request: CB-8197 Switch to nodejs for ios platfor...

2015-01-21 Thread agrieve
Github user agrieve commented on a diff in the pull request: https://github.com/apache/cordova-ios/pull/126#discussion_r23304383 --- Diff: bin/node_modules/nopt/.npmignore --- @@ -0,0 +1 @@ +node_modules --- End diff -- NVM - I see it now :S --- If your project is

[GitHub] cordova-ios pull request: CB-8197 Switch to nodejs for ios platfor...

2015-01-21 Thread agrieve
Github user agrieve commented on a diff in the pull request: https://github.com/apache/cordova-ios/pull/126#discussion_r23304345 --- Diff: bin/node_modules/nopt/.npmignore --- @@ -0,0 +1 @@ +node_modules --- End diff -- Also - do you not need to check in the "abbrev

[GitHub] cordova-ios pull request: CB-8197 Switch to nodejs for ios platfor...

2015-01-21 Thread agrieve
Github user agrieve commented on a diff in the pull request: https://github.com/apache/cordova-ios/pull/126#discussion_r23304211 --- Diff: bin/node_modules/nopt/.npmignore --- @@ -0,0 +1 @@ +node_modules --- End diff -- Why exclude nopt's dependencies when npm pack'

[GitHub] cordova-ios pull request: CB-8197 Switch to nodejs for ios platfor...

2015-01-21 Thread agrieve
Github user agrieve commented on a diff in the pull request: https://github.com/apache/cordova-ios/pull/126#discussion_r23304037 --- Diff: bin/lib/versions.js --- @@ -85,17 +85,59 @@ exports.get_apple_xcode_version = function() { child_process.exec('xcodebuild -version', fu

[GitHub] cordova-ios pull request: CB-8197 Switch to nodejs for ios platfor...

2015-01-21 Thread agrieve
Github user agrieve commented on a diff in the pull request: https://github.com/apache/cordova-ios/pull/126#discussion_r23303870 --- Diff: bin/lib/check_reqs.js --- @@ -0,0 +1,94 @@ +/* + Licensed to the Apache Software Foundation (ASF) under one + or more co

[GitHub] cordova-ios pull request: CB-8197 Switch to nodejs for ios platfor...

2015-01-21 Thread agrieve
Github user agrieve commented on a diff in the pull request: https://github.com/apache/cordova-ios/pull/126#discussion_r23303819 --- Diff: bin/lib/versions.js --- @@ -85,17 +85,59 @@ exports.get_apple_xcode_version = function() { child_process.exec('xcodebuild -version', fu

Re: cordova-android 4.0 JUnit tests

2015-01-21 Thread Andrew Grieve
On Wed, Jan 21, 2015 at 3:30 AM, Fu, Junwei wrote: > > Junwei - do the tests pass for you? I certainly can't get most of them > > to (running on KitKat) :( > Yes, the tests passed on my Galaxy s3 device, and then I try to verify > those on KitKat, they passed partly as you said, I am glad to fix

RE: Adding ability to add any platform on any OS

2015-01-21 Thread Vladimir Kotikov (Akvelon)
Send a PR for this: https://github.com/apache/cordova-ios/pull/126 Could anyone pls. review? --- Best regards, Vladimir -Original Message- From: Vladimir Kotikov (Akvelon) [mailto:v-vlk...@microsoft.com] Sent: Tuesday, 20 January, 2015 18:32 To: dev@cordova.apache.org Subje

[GitHub] cordova-ios pull request: CB-8197 Switch to nodejs for ios platfor...

2015-01-21 Thread vladimir-kotikov
GitHub user vladimir-kotikov opened a pull request: https://github.com/apache/cordova-ios/pull/126 CB-8197 Switch to nodejs for ios platform scripts Implementation for [CB-8197](https://issues.apache.org/jira/browse/CB-8197) Part of this work is already done for create/update scr

[GitHub] cordova-wp8 pull request: Use WP8 thread pool for each command, in...

2015-01-21 Thread brodybits
GitHub user brodybits opened a pull request: https://github.com/apache/cordova-wp8/pull/72 Use WP8 thread pool for each command, instead of starting new thread As reported in [CB-7443](https://issues.apache.org/jira/browse/CB-7443), ref: https://msdn.microsoft.com/en-us/library/wind

[GitHub] cordova-windows pull request: CB-8307 - new 10yr cert and tools to...

2015-01-21 Thread sgrebnov
Github user sgrebnov commented on a diff in the pull request: https://github.com/apache/cordova-windows/pull/55#discussion_r23286596 --- Diff: package.json --- @@ -26,7 +28,8 @@ }, "devDependencies": { "jasmine-node": "1.14.5", -"rewire": "

[GitHub] cordova-windows pull request: CB-8307 - new 10yr cert and tools to...

2015-01-21 Thread sgrebnov
Github user sgrebnov commented on the pull request: https://github.com/apache/cordova-windows/pull/55#issuecomment-70806039 :+1: --- 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

RE: cordova-android 4.0 JUnit tests

2015-01-21 Thread Fu, Junwei
> Junwei - do the tests pass for you? I certainly can't get most of them > to (running on KitKat) :( Yes, the tests passed on my Galaxy s3 device, and then I try to verify those on KitKat, they passed partly as you said, I am glad to fix those issue if you don't mind. > Also - maybe leave the i