[GitHub] cordova-wp8 pull request: CB-8580 Fix for plugins responding with ...

2015-03-03 Thread EddyVerbruggen
Github user EddyVerbruggen closed the pull request at: https://github.com/apache/cordova-wp8/pull/77 --- 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 fea

[GitHub] cordova-wp8 pull request: CB-8580 Fix for plugins responding with ...

2015-03-03 Thread EddyVerbruggen
Github user EddyVerbruggen commented on the pull request: https://github.com/apache/cordova-wp8/pull/77#issuecomment-76902445 Ha, cool, makes a lot of sense. Work you magic! --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well.

[GitHub] cordova-lib pull request: CB-8551 npm registry integration for fet...

2015-03-03 Thread stevengill
GitHub user stevengill opened a pull request: https://github.com/apache/cordova-lib/pull/175 CB-8551 npm registry integration for fetching and removing plugins You can merge this pull request into a Git repository by running: $ git pull https://github.com/apache/cordova-lib CB

Re: [REVIEW] NPM Registry integration for fetching plugins and plugin rm

2015-03-03 Thread Steven Gill
Also, Issue: https://issues.apache.org/jira/browse/CB-8551 On Tue, Mar 3, 2015 at 12:54 AM, Steven Gill wrote: > PR: https://github.com/apache/cordova-lib/pull/175/files > > Please review, leave feedback and ask questions! >

[REVIEW] NPM Registry integration for fetching plugins and plugin rm

2015-03-03 Thread Steven Gill
PR: https://github.com/apache/cordova-lib/pull/175/files Please review, leave feedback and ask questions!

[GitHub] cordova-lib pull request: CB-8551 npm registry integration for fet...

2015-03-03 Thread TimBarham
Github user TimBarham commented on a diff in the pull request: https://github.com/apache/cordova-lib/pull/175#discussion_r25671898 --- Diff: cordova-lib/src/plugman/registry/registry.js --- @@ -218,10 +205,18 @@ module.exports = { /** * @method initSettings + *

[GitHub] cordova-lib pull request: CB-8551 npm registry integration for fet...

2015-03-03 Thread TimBarham
Github user TimBarham commented on a diff in the pull request: https://github.com/apache/cordova-lib/pull/175#discussion_r25672017 --- Diff: cordova-lib/src/plugman/registry/registry.js --- @@ -236,15 +231,22 @@ function initSettings() { module.exports.settings = r

[GitHub] cordova-lib pull request: CB-8551 npm registry integration for fet...

2015-03-03 Thread TimBarham
Github user TimBarham commented on a diff in the pull request: https://github.com/apache/cordova-lib/pull/175#discussion_r25672404 --- Diff: cordova-lib/src/plugman/registry/registry.js --- @@ -322,3 +324,102 @@ function makeRequest (method, where, what, cb_) { return

[GitHub] cordova-lib pull request: CB-8551 npm registry integration for fet...

2015-03-03 Thread TimBarham
Github user TimBarham commented on a diff in the pull request: https://github.com/apache/cordova-lib/pull/175#discussion_r25672462 --- Diff: cordova-lib/src/plugman/uninstall.js --- @@ -47,7 +47,7 @@ function uninstall(platform, project_dir, id, plugins_dir, options) { opt

[GitHub] cordova-plugin-camera pull request: CB-8498: On windows import fro...

2015-03-03 Thread Bubkastor
Github user Bubkastor commented on the pull request: https://github.com/apache/cordova-plugin-camera/pull/64#issuecomment-76925942 Image not open this uri object. --- 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 p

[GitHub] cordova-lib pull request: CB-8551 npm registry integration for fet...

2015-03-03 Thread agrieve
Github user agrieve commented on a diff in the pull request: https://github.com/apache/cordova-lib/pull/175#discussion_r25690367 --- Diff: cordova-lib/src/cordova/plugin.js --- @@ -238,7 +239,21 @@ module.exports = function plugin(command, targets, opts) { retu

[GitHub] cordova-lib pull request: CB-8551 npm registry integration for fet...

2015-03-03 Thread agrieve
Github user agrieve commented on a diff in the pull request: https://github.com/apache/cordova-lib/pull/175#discussion_r25690407 --- Diff: cordova-lib/src/plugman/fetch.js --- @@ -147,6 +146,9 @@ function fetchPlugin(plugin_src, plugins_dir, options) { checkID(options.e

[GitHub] cordova-lib pull request: CB-8551 npm registry integration for fet...

2015-03-03 Thread agrieve
Github user agrieve commented on a diff in the pull request: https://github.com/apache/cordova-lib/pull/175#discussion_r25690540 --- Diff: cordova-lib/src/plugman/registry/registry.js --- @@ -236,15 +231,22 @@ function initSettings() { module.exports.settings = rc(

Some thoughts/questions on the new --save feature

2015-03-03 Thread Raymond Camden
1) Is there any reason why --save isn't true by default? It would seem that in a majority of cases I'd want to save my plugins to the configuration file. I definitely see times when I would *not* want to do so, but it seems like that would be the minority of cases. 2) This is probably an edge case

[GitHub] cordova-lib pull request: CB-8551 npm registry integration for fet...

2015-03-03 Thread agrieve
Github user agrieve commented on a diff in the pull request: https://github.com/apache/cordova-lib/pull/175#discussion_r25690632 --- Diff: cordova-lib/src/plugman/registry/registry.js --- @@ -322,3 +324,102 @@ function makeRequest (method, where, what, cb_) { return r

[GitHub] cordova-lib pull request: CB-8551 npm registry integration for fet...

2015-03-03 Thread agrieve
Github user agrieve commented on a diff in the pull request: https://github.com/apache/cordova-lib/pull/175#discussion_r25691265 --- Diff: cordova-lib/src/plugman/registry/registry.js --- @@ -322,3 +324,102 @@ function makeRequest (method, where, what, cb_) { return r

[GitHub] cordova-lib pull request: CB-8551 npm registry integration for fet...

2015-03-03 Thread agrieve
Github user agrieve commented on a diff in the pull request: https://github.com/apache/cordova-lib/pull/175#discussion_r25691366 --- Diff: cordova-lib/src/plugman/registry/registry.js --- @@ -322,3 +324,102 @@ function makeRequest (method, where, what, cb_) { return r

[GitHub] cordova-lib pull request: CB-8551 npm registry integration for fet...

2015-03-03 Thread agrieve
Github user agrieve commented on a diff in the pull request: https://github.com/apache/cordova-lib/pull/175#discussion_r25691445 --- Diff: cordova-lib/package.json --- @@ -19,6 +19,7 @@ "dependencies": { "bplist-parser": "0.0.6", "cordova-js": "3.8.0", +

Re: Some thoughts/questions on the new --save feature

2015-03-03 Thread Victor Sosa
+1 on Raymond's idea, I also why --save isn't default. When talking about SCMs, the best practice for Cordova applications is to share them with no platforms nor plugins, so each time a developer checks out the project from the repo, he/she will need to add the appropriate plugins and desired platf

Re: [REVIEW] NPM Registry integration for fetching plugins and plugin rm

2015-03-03 Thread Andrew Grieve
Good stuff. Added some style comments, but logic lgtm! On Tue, Mar 3, 2015 at 3:55 AM, Steven Gill wrote: > Also, Issue: https://issues.apache.org/jira/browse/CB-8551 > > On Tue, Mar 3, 2015 at 12:54 AM, Steven Gill > wrote: > > > PR: https://github.com/apache/cordova-lib/pull/175/files > > > >

Re: [REVIEW] NPM Registry integration for fetching plugins and plugin rm

2015-03-03 Thread Darryl Pogue
Less a comment on the code and more of a question about the NPM transition from an end-user pespective: Is Cordova going to automatically discover and use plugins that are found in the node_modules folder (from package.json), or will it still keep its own plugins folder and require they be install

[GitHub] cordova-plugin-inappbrowser pull request: Inappbrowser always on t...

2015-03-03 Thread VeldMuijz
GitHub user VeldMuijz opened a pull request: https://github.com/apache/cordova-plugin-inappbrowser/pull/85 Inappbrowser always on top wp8.1 For windows 8.1 the inappbrowser can be displayed with a lower zindex than the current viewing page. Causing the inappbrowser to be opened o

Re: Some thoughts/questions on the new --save feature

2015-03-03 Thread Steven Gill
1) I would say it is not autosave by default so we follow node/npm style. You have to be explicit when saving dependencies to node projects. Example "npm install browserify --save". 2) I believe their is a --link command to point to your www for your Cordova project to use. This isn't a perfect so

Re: Some thoughts/questions on the new --save feature

2015-03-03 Thread Raymond Camden
1) I can see the logic there, but it seems like folks would want to save so much, overwhelmingly so, that it makes sense to not follow the expectation here? Also, would our "typical" user be more familiar with Node and be confused by this? 2) I know about the link too, my issue is more about what

Re: Android's new Whitelist Plugins

2015-03-03 Thread Michal Mocny
My thoughts: - The split between , , and : Like it a lot. - I think the defaults *for the plugin* are very reasonable. However, we may want to provide a default set of tags for the hello world app. A year or so ago we added a default access * whitelist and I think maybe we should continue that.

back button on IAB Android

2015-03-03 Thread Marcel Kinard
Sorry to be absent for so long, product work here is eating me up. After a customer question, I noticed the behavior of the system Back button on the InAppBrowser on Android. In the scenario where the user navigates a couple of pages forward in the IAB, when they hit the system Back button the a

Re: Android's new Whitelist Plugins

2015-03-03 Thread Michal Mocny
..taking a look at the cli create.js script, we would need to update it a bit to actually have it import a hello-world config.xml. The lib actually ships the template config.xml right now instead of the hello-world project, which seems unnecessary given our --copy-from/--link-to import logic. Seem

Re: [REVIEW] NPM Registry integration for fetching plugins and plugin rm

2015-03-03 Thread Michal Mocny
Darryl, I believe that cordova-cli will continue to use `cordova plugin add` and not use `node_modules` at all for phase 1 rollout. It will use the `npm` library to fetch into the existing plugins/ folder. There will be no user visible change to project structure, though the plugins are being re

Re: Some thoughts/questions on the new --save feature

2015-03-03 Thread Michal Mocny
Raymond, you should just include your config.xml in the repo along with your www. It will be imported as part of --copy-from and should (I haven't tested!) auto-restore plugins with this new feature. As for default not --save, this is mostly for npm compatibility, but also because we decided to a

Re: [REVIEW] NPM Registry integration for fetching plugins and plugin rm

2015-03-03 Thread Steven Gill
Darryl, +1 to what Michal said. We are still figuring out details for phase 2. Support for package.json and installation from node_modules are two things I'd love to see supported. On Mar 3, 2015 8:54 AM, "Michal Mocny" wrote: > Darryl, > > I believe that cordova-cli will continue to use `cordov

[GitHub] cordova-plugins pull request: clarify wording on browser intent

2015-03-03 Thread cmarcelk
GitHub user cmarcelk opened a pull request: https://github.com/apache/cordova-plugins/pull/19 clarify wording on browser intent You can merge this pull request into a Git repository by running: $ git pull https://github.com/cmarcelk/cordova-plugins wording Alternatively you c

Re: Some thoughts/questions on the new --save feature

2015-03-03 Thread Michal Mocny
Yet more clarification: your git repo should look like: - www/ - index.html - ... - config.xml - ... NOT - config.xml - index.html - ... On Tue, Mar 3, 2015 at 11:59 AM, Michal Mocny wrote: > Raymond, you should just include your config.xml in the repo along with > your www. It will be i

Re: Some thoughts/questions on the new --save feature

2015-03-03 Thread Michal Mocny
Also also ;) cca applications include a default .gitignore with these contents: --- ## ## This file is the suggested default .gitignore to be used by cca applications ## # Usually, you shouldn't version control `platforms/` and `plugins/`. They are # build artefacts which change from ma

Re: Some thoughts/questions on the new --save feature

2015-03-03 Thread Raymond Camden
Thanks Michal. I forgot --copy-from was smart like that. If I remember right, it can tell if the folder is "just" WWW crap and use it like WWW crap, otherwise it merges it into a a new project. Is that a good way of saying it? On Tue, Mar 3, 2015 at 11:12 AM, Michal Mocny wrote: > Yet more clari

[GitHub] cordova-lib pull request: CB-8551 npm registry integration for fet...

2015-03-03 Thread mmocny
Github user mmocny commented on a diff in the pull request: https://github.com/apache/cordova-lib/pull/175#discussion_r25708820 --- Diff: cordova-lib/package.json --- @@ -19,6 +19,7 @@ "dependencies": { "bplist-parser": "0.0.6", "cordova-js": "3.8.0", +

[GitHub] cordova-lib pull request: CB-8551 npm registry integration for fet...

2015-03-03 Thread mmocny
Github user mmocny commented on a diff in the pull request: https://github.com/apache/cordova-lib/pull/175#discussion_r25709035 --- Diff: cordova-lib/src/cordova/plugin.js --- @@ -238,7 +239,21 @@ module.exports = function plugin(command, targets, opts) { retur

[GitHub] cordova-lib pull request: CB-8551 npm registry integration for fet...

2015-03-03 Thread mmocny
Github user mmocny commented on a diff in the pull request: https://github.com/apache/cordova-lib/pull/175#discussion_r25709236 --- Diff: cordova-lib/src/cordova/plugin.js --- @@ -238,7 +239,21 @@ module.exports = function plugin(command, targets, opts) { retur

RE: Some thoughts/questions on the new --save feature

2015-03-03 Thread Mefire O .
Michal, there is a 'mass save' feature for both platforms and plugins on the way. They will allow the saving of already installed platforms and plugins respectively : 'cordova platform save' and 'cordova plugin save'. The platform's one is currently in code review: https://github.com/apache/co

[GitHub] cordova-lib pull request: CB-8551 npm registry integration for fet...

2015-03-03 Thread mmocny
Github user mmocny commented on a diff in the pull request: https://github.com/apache/cordova-lib/pull/175#discussion_r25709472 --- Diff: cordova-lib/src/plugman/registry/registry.js --- @@ -218,10 +205,18 @@ module.exports = { /** * @method initSettings + * @pa

[GitHub] cordova-lib pull request: CB-8551 npm registry integration for fet...

2015-03-03 Thread mmocny
Github user mmocny commented on a diff in the pull request: https://github.com/apache/cordova-lib/pull/175#discussion_r25709825 --- Diff: cordova-lib/src/plugman/registry/registry.js --- @@ -322,3 +324,102 @@ function makeRequest (method, where, what, cb_) { return re

Re: back button on IAB Android

2015-03-03 Thread Jesse
Assuming that pull request code works, merge away. The only concern I had was that the 'hardwareback' option should default to true. @purplecabbage risingj.com On Tue, Mar 3, 2015 at 8:27 AM, Marcel Kinard wrote: > Sorry to be absent for so long, product work here is eating me up. > > After a c

Re: back button on IAB Android

2015-03-03 Thread Andrew Grieve
Makes sense to me! On Tue, Mar 3, 2015 at 11:27 AM, Marcel Kinard wrote: > Sorry to be absent for so long, product work here is eating me up. > > After a customer question, I noticed the behavior of the system Back > button on the InAppBrowser on Android. In the scenario where the user > navigat

[GitHub] cordova-lib pull request: CB-8551 npm registry integration for fet...

2015-03-03 Thread mmocny
Github user mmocny commented on a diff in the pull request: https://github.com/apache/cordova-lib/pull/175#discussion_r25710017 --- Diff: cordova-lib/src/plugman/registry/registry.js --- @@ -322,3 +324,102 @@ function makeRequest (method, where, what, cb_) { return re

Re: Android's new Whitelist Plugins

2015-03-03 Thread Andrew Grieve
Like your ideas a lot. Updating the project template makes a lot of sense. Tried to make it clear in the README, so if any part was not clear please fix it. But, the CSP tag is the more important bit, since can't actually block all requests. The only reason to even leave in there is to support p

[GitHub] cordova-lib pull request: CB-8551 npm registry integration for fet...

2015-03-03 Thread mmocny
Github user mmocny commented on the pull request: https://github.com/apache/cordova-lib/pull/175#issuecomment-76999272 Overall, impressed with the lack of significant impact. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well

Re: Android's new Whitelist Plugins

2015-03-03 Thread Michal Mocny
Ah, the bit about being mostly useful for pre-kitkat was context I was missing. Seems important to note at the start of the section. I'll update the README. On Tue, Mar 3, 2015 at 1:02 PM, Andrew Grieve wrote: > Like your ideas a lot. Updating the project template makes a lot of sense. > > Tr

Re: Some thoughts/questions on the new --save feature

2015-03-03 Thread Michal Mocny
Sweet! Ray: I think `www and config.xml crap` is a great way to say it ;) On Tue, Mar 3, 2015 at 12:53 PM, Mefire O. wrote: > Michal, there is a 'mass save' feature for both platforms and plugins on > the way. They will allow the saving of already installed platforms and > plugins respectively

Re: Android's new Whitelist Plugins

2015-03-03 Thread Michal Mocny
(Added this note: https://github.com/apache/cordova-plugins/commit/3ed17046ea7efaeccda4c4ffe82bb351e8b966f1, let me know if its inacurate). -Michal On Tue, Mar 3, 2015 at 1:06 PM, Michal Mocny wrote: > Ah, the bit about being mostly useful for pre-kitkat was context > I was missing. Seems imp

Re: Some thoughts/questions on the new --save feature

2015-03-03 Thread Gorkem Ercan
You can enable auto save by adding auto_save_plugins to be true on the .cordova/config.json file. I think this helps with the case 1 -- Gorkem On 3 Mar 2015, at 9:27, Raymond Camden wrote: 1) Is there any reason why --save isn't true by default? It would seem that in a majority of cases I'd

Re: Some thoughts/questions on the new --save feature

2015-03-03 Thread Victor Sosa
Good to know this trick. .. Thanks, Gorkem! 2015-03-03 12:14 GMT-06:00 Gorkem Ercan : > > You can enable auto save by adding auto_save_plugins to be true on the > .cordova/config.json file. I think this helps with the case 1 > -- > Gorkem > > > > On 3 Mar 2015, at 9:27, Raymond Camden wrote: > >

Re: Some thoughts/questions on the new --save feature

2015-03-03 Thread Michal Mocny
Worthy of a blogpost, Ray! On Tue, Mar 3, 2015 at 1:17 PM, Victor Sosa wrote: > Good to know this trick. .. Thanks, Gorkem! > > 2015-03-03 12:14 GMT-06:00 Gorkem Ercan : > > > > > You can enable auto save by adding auto_save_plugins to be true on the > > .cordova/config.json file. I think this h

Re: Some thoughts/questions on the new --save feature

2015-03-03 Thread Steven Gill
On Tue, Mar 3, 2015 at 8:10 AM, Raymond Camden wrote: > 1) I can see the logic there, but it seems like folks would want to > save so much, overwhelmingly so, that it makes sense to not follow the > expectation here? Also, would our "typical" user be more familiar with > Node and be confused by t

Re: Some thoughts/questions on the new --save feature

2015-03-03 Thread Tommy Williams
+1 On 4 Mar 2015 5:21 am, "Michal Mocny" wrote: > Worthy of a blogpost, Ray! > > On Tue, Mar 3, 2015 at 1:17 PM, Victor Sosa > wrote: > > > Good to know this trick. .. Thanks, Gorkem! > > > > 2015-03-03 12:14 GMT-06:00 Gorkem Ercan : > > > > > > > > You can enable auto save by adding auto_save_p

[GitHub] cordova-lib pull request: CB-8551 npm registry integration for fet...

2015-03-03 Thread stevengill
Github user stevengill commented on a diff in the pull request: https://github.com/apache/cordova-lib/pull/175#discussion_r25715497 --- Diff: cordova-lib/package.json --- @@ -19,6 +19,7 @@ "dependencies": { "bplist-parser": "0.0.6", "cordova-js": "3.8.0",

[GitHub] cordova-lib pull request: CB-8499 `cordova platform save` should s...

2015-03-03 Thread omefire
Github user omefire commented on the pull request: https://github.com/apache/cordova-lib/pull/170#issuecomment-77011164 @agrieve , I addressed your comments. could you please take another look ? --- If your project is set up for it, you can reply to this email and have your reply appe

[GitHub] cordova-lib pull request: CB-8551 npm registry integration for fet...

2015-03-03 Thread stevengill
Github user stevengill commented on a diff in the pull request: https://github.com/apache/cordova-lib/pull/175#discussion_r25715602 --- Diff: cordova-lib/src/cordova/plugin.js --- @@ -238,7 +239,21 @@ module.exports = function plugin(command, targets, opts) { r

Re: Some thoughts/questions on the new --save feature

2015-03-03 Thread Michal Mocny
We should save the phase 2 planning for a separate thread / next hangout. Many strong opinions and bikeshedding to be had, lets finish phase 1 and sit on the changes for a bit to get comfortable. -Michal On Tue, Mar 3, 2015 at 1:55 PM, Steven Gill wrote: > On Tue, Mar 3, 2015 at 8:10 AM, Raymo

[GitHub] cordova-lib pull request: CB-8551 npm registry integration for fet...

2015-03-03 Thread stevengill
Github user stevengill commented on a diff in the pull request: https://github.com/apache/cordova-lib/pull/175#discussion_r25716406 --- Diff: cordova-lib/src/plugman/registry/registry.js --- @@ -322,3 +324,102 @@ function makeRequest (method, where, what, cb_) { retur

[GitHub] cordova-lib pull request: CB-8551 npm registry integration for fet...

2015-03-03 Thread stevengill
Github user stevengill commented on a diff in the pull request: https://github.com/apache/cordova-lib/pull/175#discussion_r25716498 --- Diff: cordova-lib/src/plugman/registry/registry.js --- @@ -322,3 +324,102 @@ function makeRequest (method, where, what, cb_) { retur

[GitHub] cordova-lib pull request: CB-8551 npm registry integration for fet...

2015-03-03 Thread stevengill
Github user stevengill commented on a diff in the pull request: https://github.com/apache/cordova-lib/pull/175#discussion_r25716916 --- Diff: cordova-lib/src/plugman/registry/registry.js --- @@ -322,3 +324,102 @@ function makeRequest (method, where, what, cb_) { retur

[GitHub] cordova-lib pull request: CB-8551 npm registry integration for fet...

2015-03-03 Thread stevengill
Github user stevengill commented on a diff in the pull request: https://github.com/apache/cordova-lib/pull/175#discussion_r25717156 --- Diff: cordova-lib/src/plugman/registry/registry.js --- @@ -322,3 +324,102 @@ function makeRequest (method, where, what, cb_) { retur

[GitHub] cordova-lib pull request: CB-8551 npm registry integration for fet...

2015-03-03 Thread stevengill
Github user stevengill commented on a diff in the pull request: https://github.com/apache/cordova-lib/pull/175#discussion_r25716978 --- Diff: cordova-lib/src/plugman/registry/registry.js --- @@ -322,3 +324,102 @@ function makeRequest (method, where, what, cb_) { retur

Re: Android's new Whitelist Plugins

2015-03-03 Thread Michal Mocny
I've filed a JIRA issue with my thoughts on how to approach this: https://issues.apache.org/jira/browse/CB-8597 On Tue, Mar 3, 2015 at 1:02 PM, Andrew Grieve wrote: > Like your ideas a lot. Updating the project template makes a lot of sense. > > Tried to make it clear in the README, so if any pa

[DISCUSS] publishing cordova-app-hello-world to npm and adding as dependency of cordova-lib

2015-03-03 Thread Michal Mocny
This came up in the context of whitelist discussion, but wanted to bring it to everyones attention. JIRA: https://issues.apache.org/jira/browse/CB-8597 We would like the default app to support installing the whitelist plugin automatically using the new plugin auto-restore feature (If you disagree

Re: [DISCUSS] publishing cordova-app-hello-world to npm and adding as dependency of cordova-lib

2015-03-03 Thread Steven Gill
+1. So on board with this. I'll take a look at your Jira issue later on today. On Tue, Mar 3, 2015 at 11:32 AM, Michal Mocny wrote: > This came up in the context of whitelist discussion, but wanted to bring it > to everyones attention. > > JIRA: https://issues.apache.org/jira/browse/CB-8597 > >

[GitHub] cordova-lib pull request: CB-8551 npm registry integration for fet...

2015-03-03 Thread stevengill
Github user stevengill commented on a diff in the pull request: https://github.com/apache/cordova-lib/pull/175#discussion_r25718379 --- Diff: cordova-lib/src/plugman/registry/registry.js --- @@ -322,3 +324,102 @@ function makeRequest (method, where, what, cb_) { retur

[GitHub] cordova-lib pull request: CB-8551 npm registry integration for fet...

2015-03-03 Thread stevengill
Github user stevengill commented on a diff in the pull request: https://github.com/apache/cordova-lib/pull/175#discussion_r25718971 --- Diff: cordova-lib/src/plugman/registry/registry.js --- @@ -236,15 +231,22 @@ function initSettings() { module.exports.settings =

[GitHub] cordova-lib pull request: CB-8551 npm registry integration for fet...

2015-03-03 Thread stevengill
Github user stevengill commented on a diff in the pull request: https://github.com/apache/cordova-lib/pull/175#discussion_r25719020 --- Diff: cordova-lib/src/plugman/registry/registry.js --- @@ -218,10 +205,18 @@ module.exports = { /** * @method initSettings + *

Re: [DISCUSS] publishing cordova-app-hello-world to npm and adding as dependency of cordova-lib

2015-03-03 Thread Jesse
+1 Does this affect the ability to use an individual platform? ie. cordova-ios/bin/create ... Will platform create scripts need to change? @purplecabbage risingj.com On Tue, Mar 3, 2015 at 11:32 AM, Michal Mocny wrote: > This came up in the context of whitelist discussion, but wanted to bring

[GitHub] cordova-lib pull request: CB-8551 npm registry integration for fet...

2015-03-03 Thread stevengill
Github user stevengill commented on a diff in the pull request: https://github.com/apache/cordova-lib/pull/175#discussion_r25719212 --- Diff: cordova-lib/src/plugman/fetch.js --- @@ -147,6 +146,9 @@ function fetchPlugin(plugin_src, plugins_dir, options) { checkID(option

Re: [DISCUSS] publishing cordova-app-hello-world to npm and adding as dependency of cordova-lib

2015-03-03 Thread Michal Mocny
Jesse, I think not. The platform scripts just copy the bundled www/ (its already included with the platform). This is only for the top-level project www/ of a cordova-cli project. I guess, though, that we could consider changing the platform create scripts to import config.xml's, since they curr

[GitHub] cordova-lib pull request: CB-8551 npm registry integration for fet...

2015-03-03 Thread stevengill
Github user stevengill commented on a diff in the pull request: https://github.com/apache/cordova-lib/pull/175#discussion_r25720506 --- Diff: cordova-lib/src/plugman/registry/registry.js --- @@ -236,15 +231,22 @@ function initSettings() { module.exports.settings =

[GitHub] cordova-lib pull request: CB-8551 npm registry integration for fet...

2015-03-03 Thread stevengill
Github user stevengill commented on a diff in the pull request: https://github.com/apache/cordova-lib/pull/175#discussion_r25720721 --- Diff: cordova-lib/src/plugman/registry/registry.js --- @@ -218,10 +205,18 @@ module.exports = { /** * @method initSettings + *

Re: [REVIEW] NPM Registry integration for fetching plugins and plugin rm

2015-03-03 Thread Steven Gill
Thanks for all of the feedback so far! I will spend some time this afternoon making some changes. On Tue, Mar 3, 2015 at 9:13 AM, Steven Gill wrote: > Darryl, > > +1 to what Michal said. We are still figuring out details for phase 2. > Support for package.json and installation from node_modules

Re: [DISCUSS] publishing cordova-app-hello-world to npm and adding as dependency of cordova-lib

2015-03-03 Thread Michal Mocny
Pushed first attempt to remote branches: - https://github.com/Apache/cordova-cli/tree/copy-from-app-hello-world - https://github.com/Apache/cordova-app-hello-world/tree/copy-from-app-hello-world Cleaning up cordova-lib is strictly required, though I think we should still take the opportunity to do

[GitHub] cordova-lib pull request: CB-8551 npm registry integration for fet...

2015-03-03 Thread mmocny
Github user mmocny commented on a diff in the pull request: https://github.com/apache/cordova-lib/pull/175#discussion_r25722381 --- Diff: cordova-lib/src/plugman/registry/registry.js --- @@ -322,3 +324,102 @@ function makeRequest (method, where, what, cb_) { return re

[GitHub] cordova-lib pull request: CB-8551 npm registry integration for fet...

2015-03-03 Thread agrieve
Github user agrieve commented on a diff in the pull request: https://github.com/apache/cordova-lib/pull/175#discussion_r25725351 --- Diff: cordova-lib/src/plugman/registry/registry.js --- @@ -236,15 +231,22 @@ function initSettings() { module.exports.settings = rc(

[GitHub] cordova-plugin-test-framework pull request: CB-8385 Ensure plugin-...

2015-03-03 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/cordova-plugin-test-framework/pull/10 --- 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

[GitHub] cordova-plugin-camera pull request: CB-8599 fix threading issue wi...

2015-03-03 Thread muratsu
GitHub user muratsu opened a pull request: https://github.com/apache/cordova-plugin-camera/pull/72 CB-8599 fix threading issue with cameraPicker Fixes the issue where cameraPicker is trying to modify the main thread UI from a background thread. You can merge this pull request into

[CLI] Default target when not specified for "build", "compile"?

2015-03-03 Thread Shazron
What is the expected behaviour here? `cordova build` My expectation is that it would build for both device and emulator. In iOS pre-3.8.0 it does, but now it only builds for emulator only after the node.js script port, which may or may not be a bug depending on consensus. The online docs and cli

[DISCUSS] Docs release

2015-03-03 Thread Steven Gill
Been a while. Thoughts? What needs to be updated? Should we wait for Android 4.0.0 release?

Re: [DISCUSS] Docs release

2015-03-03 Thread Victor Sosa
Hi Steven. Just want to point out that Russian docs are not completely translated yet https://crowdin.com/project/cordova This is mainly because Andrey requested holding back the automated translation while he is working on the validation framework for the documentation. As an FYI, I'll review wh

Re: [DISCUSS] Docs release

2015-03-03 Thread Steven Gill
Sounds good. Keep me in the loop and let me know when you all are ready to release. On Tue, Mar 3, 2015 at 1:56 PM, Victor Sosa wrote: > Hi Steven. > Just want to point out that Russian docs are not completely translated yet > https://crowdin.com/project/cordova > > This is mainly because Andr

[GitHub] cordova-plugin-camera pull request: CB-8599 fix threading issue wi...

2015-03-03 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/cordova-plugin-camera/pull/72 --- 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 f

[GitHub] cordova-plugin-camera pull request: add try ... catch for getting ...

2015-03-03 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/cordova-plugin-camera/pull/63 --- 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 f

Re: [CLI] Default target when not specified for "build", "compile"?

2015-03-03 Thread Michal Mocny
Imho, it should do what: "cordova run" would do. I would think it would run once for each platform, defaulting to device if attached. On Tue, Mar 3, 2015 at 4:37 PM, Shazron wrote: > What is the expected behaviour here? > `cordova build` > > My expectation is that it would build for both device

[GitHub] cordova-plugin-camera pull request: Fix for geolocation/exif data

2015-03-03 Thread purplecabbage
Github user purplecabbage commented on the pull request: https://github.com/apache/cordova-plugin-camera/pull/71#issuecomment-77058089 This looks good. Can you fix the white-space additions? Also, I don't see you listed on https://people.apache.org/committer-index.html C

Re: [CLI] Default target when not specified for "build", "compile"?

2015-03-03 Thread Shazron
Currently `cordova run` on iOS defaults to emulator as well, like the compile/build commands. It's probably safe to say we never specified what happens when no target is specified (thus right now the behaviour is not consistent for all platforms) -- and this is a new "feature request". On Tue, M

[GitHub] cordova-lib pull request: CB-8551 npm registry integration for fet...

2015-03-03 Thread TimBarham
Github user TimBarham commented on a diff in the pull request: https://github.com/apache/cordova-lib/pull/175#discussion_r25737174 --- Diff: cordova-lib/src/plugman/registry/registry.js --- @@ -236,15 +231,22 @@ function initSettings() { module.exports.settings = r

RE: [CLI] Default target when not specified for "build", "compile"?

2015-03-03 Thread Josh Soref
Fwiw, BlackBerry 10 is building both (tested w/ cordova 4.3.1-dev and blackberry10 3.7.0): $ cordova -v 4.3.1-dev (cordova-lib@4.2.1-dev) $ cordova platform update blackberry10 Updating blackberry10 project... BlackBerry10 project is now at version 3.7.0 $ cordova build Running command: "platform

[GitHub] cordova-lib pull request:

2015-03-03 Thread jsoref
Github user jsoref commented on the pull request: https://github.com/apache/cordova-lib/commit/4cf298cab1af6b02ddb9ee96df597bfe5ef5b1d3#commitcomment-10018781 In cordova-lib/src/plugman/registry/registry.js: In cordova-lib/src/plugman/registry/registry.js on line 341: below you

[GitHub] cordova-lib pull request: CB-8551 npm registry integration for fet...

2015-03-03 Thread jsoref
Github user jsoref commented on a diff in the pull request: https://github.com/apache/cordova-lib/pull/175#discussion_r25737449 --- Diff: cordova-lib/src/plugman/registry/registry.js --- @@ -236,15 +231,22 @@ function initSettings() { module.exports.settings = rc('

Re: [CLI] Default target when not specified for "build", "compile"?

2015-03-03 Thread Shazron
Defaulting to device might be problematic on iOS: sure we can detect if a device is attached (using iso-deploy), but the build might fail because a signing cert/provisioning profile is not valid (I think the only way to check is to just build, I might be wrong). The only "guaranteed" way it will a

[GitHub] cordova-lib pull request:

2015-03-03 Thread jsoref
Github user jsoref commented on the pull request: https://github.com/apache/cordova-lib/commit/4cf298cab1af6b02ddb9ee96df597bfe5ef5b1d3#commitcomment-10018789 In cordova-lib/src/plugman/registry/registry.js: In cordova-lib/src/plugman/registry/registry.js on line 344: `to for`

[GitHub] cordova-lib pull request:

2015-03-03 Thread jsoref
Github user jsoref commented on the pull request: https://github.com/apache/cordova-lib/commit/049548a5994d4efcd85560c81526bcd0cd4c3ed5#commitcomment-10018768 In cordova-lib/src/plugman/registry/registry.js: In cordova-lib/src/plugman/registry/registry.js on line 163: you proba

[GitHub] cordova-lib pull request:

2015-03-03 Thread jsoref
Github user jsoref commented on the pull request: https://github.com/apache/cordova-lib/commit/7db3df134aba1bb5b5c1ea547ff919d621c30575#commitcomment-10018820 In cordova-lib/src/plugman/registry/registry.js: In cordova-lib/src/plugman/registry/registry.js on line 381: it might

[GitHub] cordova-lib pull request:

2015-03-03 Thread jsoref
Github user jsoref commented on the pull request: https://github.com/apache/cordova-lib/commit/38a0afbdaef58dd2bc9cff88c76b0e382d835592#commitcomment-10018840 In cordova-lib/src/cordova/plugin.js: In cordova-lib/src/cordova/plugin.js on line 185: `changes` -> `changed` --- If

[GitHub] cordova-lib pull request:

2015-03-03 Thread jsoref
Github user jsoref commented on the pull request: https://github.com/apache/cordova-lib/commit/d49159b783909bee31858eb92279253698c9ec9d#commitcomment-10018875 In cordova-lib/src/cordova/plugin.js: In cordova-lib/src/cordova/plugin.js on line 321: if we're changing lines, it'd b

[GitHub] cordova-lib pull request: CB-8551 npm registry integration for fet...

2015-03-03 Thread stevengill
Github user stevengill commented on a diff in the pull request: https://github.com/apache/cordova-lib/pull/175#discussion_r25738050 --- Diff: cordova-lib/src/plugman/registry/registry.js --- @@ -236,15 +231,22 @@ function initSettings() { module.exports.settings =

[GitHub] cordova-lib pull request: CB-8551 npm registry integration for fet...

2015-03-03 Thread stevengill
Github user stevengill commented on a diff in the pull request: https://github.com/apache/cordova-lib/pull/175#discussion_r25738089 --- Diff: cordova-lib/src/plugman/registry/registry.js --- @@ -236,15 +231,22 @@ function initSettings() { module.exports.settings =

  1   2   >