[GitHub] cordova-android pull request: Fix potential null pointer error iss...

2014-10-09 Thread agamemnus
GitHub user agamemnus opened a pull request: https://github.com/apache/cordova-android/pull/126 Fix potential null pointer error issue. I am writing a Google Play APK expansion library. All the libraries I have seen so far store version data inside the content provider's

[GitHub] cordova-mobile-spec pull request: [CB-7706] Failed create mobile-s...

2014-10-09 Thread MariaBukharina
GitHub user MariaBukharina opened a pull request: https://github.com/apache/cordova-mobile-spec/pull/111 [CB-7706] Failed create mobile-spec app with windows platform from master branches Removed windows subfolder from paths for windows and windows8 platforms You can merge this

[GitHub] cordova-android pull request: Fix potential null pointer error iss...

2014-10-09 Thread agrieve
Github user agrieve commented on the pull request: https://github.com/apache/cordova-android/pull/126#issuecomment-58511873 From the [JavaDoc](http://developer.android.com/reference/android/content/ContentResolver.html#openAssetFileDescriptor(android.net.Uri, java.lang.String,

[GitHub] cordova-plugin-file pull request: CB-6994 - Windows 8 does not sav...

2014-10-09 Thread SomaticIT
Github user SomaticIT commented on the pull request: https://github.com/apache/cordova-plugin-file/pull/54#issuecomment-58514355 @sgrebnov I reviewed your changes and it's OK for me. Thanks for merging. --- If your project is set up for it, you can reply to this email and have

[GitHub] cordova-android pull request: Fix potential null pointer error iss...

2014-10-09 Thread agamemnus
Github user agamemnus commented on the pull request: https://github.com/apache/cordova-android/pull/126#issuecomment-58514514 Doesn't a FileNotFoundException end the program? (sorry, not sure...) The idea is to prevent that and let everything run normally. --- If your project is set

[GitHub] cordova-coho pull request: [CB-7744] Add support for git-depth fla...

2014-10-09 Thread MariaBukharina
GitHub user MariaBukharina opened a pull request: https://github.com/apache/cordova-coho/pull/52 [CB-7744] Add support for git-depth flag to coho clone Add support for git-depth flag to coho clone to be able to check out only latest version and reduce check out time You can merge

Re: whitelist as a plugin

2014-10-09 Thread Ian Clelland
I'm running into more and more problems caused by the whitelist (today, it's because of the dual use of the internal whitelist for should be able to navigate to URL and should be able to XHR from URL) I'm going to start to pull it out, on a topic branch based off of Android 4.0.x right now. I'll

Re: whitelist as a plugin

2014-10-09 Thread Ian Clelland
CB-7747, for those following along at home. On Thu, Oct 9, 2014 at 10:38 AM, Ian Clelland iclell...@chromium.org wrote: I'm running into more and more problems caused by the whitelist (today, it's because of the dual use of the internal whitelist for should be able to navigate to URL and

[GitHub] cordova-coho pull request: [CB-7744] Add support for git-depth fla...

2014-10-09 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/cordova-coho/pull/52 --- 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 feature

Re: whitelist as a plugin

2014-10-09 Thread Joe Bowser
This should land in 4.0.x On Oct 9, 2014 7:38 AM, Ian Clelland iclell...@chromium.org wrote: I'm running into more and more problems caused by the whitelist (today, it's because of the dual use of the internal whitelist for should be able to navigate to URL and should be able to XHR from URL)

How to update cordova 3.5.0 to 3.6.3 on windows 8

2014-10-09 Thread Kartik Mehta
Hi, I want to update cordova version from command line I currently have cordova 3.5.0 want to upgrade to 3.6.3. Can you please let me know the steps to be performed on windows 8. Regards, Kartik. DISCLAIMER: This communication is confidential and privileged and is directed to and for

RE: How to update cordova 3.5.0 to 3.6.3 on windows 8

2014-10-09 Thread Sergey Grebnov (Akvelon)
Hi, please try the following http://cordova.apache.org/docs/en/3.6.0/guide_platforms_win8_upgrade.md.html#Upgrading%20Windows%208 1. Update cordova CLI to the latest version (npm update -g cordova) 2. cd to your app 3. Run 'cordova platform update windows8' Thx! Sergey -Original

Re: How to update cordova 3.5.0 to 3.6.3 on windows 8

2014-10-09 Thread Victor Sosa
Hi Kartik. Do you want to update CLI or platforms. Either way here how you can do it *Updating CLI* npm -g update cordova *Updating platforms* cordova platform update platform Do this update for each platform you have Source:

Re: Build signed archives using CLI

2014-10-09 Thread Frederico Galvão
I agree with pretty much everything mentioned, but as far as I know cordova is trying it's best to not depend on anything to be in /platforms/TARGET_PLATFORM/. Having said that, the package command should always expect things to be in config.xml or something like that. Also, the closer we are in

RE: Build signed archives using CLI

2014-10-09 Thread Chuck Lantz
One option here could be to build off of the idea of the res folder that is in the current samples for splashscreens and icons and introduce something like res/native. Files placed here would be put into the native project folders before_prepare and would be further enhanced by the plugin and

Re: Build signed archives using CLI

2014-10-09 Thread Jesse
I am liking all of this. Are we ready to move this to an editable plaintext doc to collaborate on? I agree that we should take advantage of as much 'by-convention' as we can, meaning things like `cordova package ios` defaults to a code sign identity of 'iPhone Developer' and signs based on

Re: List replies

2014-10-09 Thread Jesse
Yeah! INFRA fixed it, no idea why it happened ... Note: if you are replying to this email, you may still need to reply-all @purplecabbage risingj.com On Tue, Oct 7, 2014 at 2:27 PM, Jesse purplecabb...@gmail.com wrote: Damn it, did it again ... Issue created

Re: Build signed archives using CLI

2014-10-09 Thread Andrew Grieve
The prompting is actually pretty appropriate here since passwords are involved I think. I think also that keys will often not be checked into source control, but maybe the best way to support that is to allow multiple ways of specifying things (e.g. default to convention, allow override via

Re: whitelist as a plugin

2014-10-09 Thread Andrew Grieve
It's technically a breaking change, so agree 4.0.x makes sense. On Thu, Oct 9, 2014 at 12:09 PM, Joe Bowser bows...@gmail.com wrote: This should land in 4.0.x On Oct 9, 2014 7:38 AM, Ian Clelland iclell...@chromium.org wrote: I'm running into more and more problems caused by the whitelist

Re: Independent platform release summary

2014-10-09 Thread Michal Mocny
On Wed, Oct 8, 2014 at 4:12 PM, Michal Mocny mmo...@chromium.org wrote: Thanks everyone for participation in what was a long and grueling discussion. Summary of current proposal: - Cad-ver is dead. - Everything moves Sem-ver, with platforms continuing from current versions and diverging

Re: Build signed archives using CLI

2014-10-09 Thread Shazron
Liking Subhag's proposal. Agree with Jesse on using conventions as a default plus config.xml -- with overrides/env-vars possible. The only caveat for including info in the config.xml is, the config.xml data is copied into the iOS platform and will be included in the .app bundle, and will leak

[GitHub] cordova-plugin-file pull request: [fxos] update __format__ to matc...

2014-10-09 Thread rodms10
GitHub user rodms10 opened a pull request: https://github.com/apache/cordova-plugin-file/pull/85 [fxos] update __format__ to match pathsPrefix Noticed this while implementing new functionality on PG dev app. The filesystem prefix should match what we have in

Re: Build signed archives using CLI

2014-10-09 Thread Frederico Galvão
The ant target that is executed when I run cordova build android --release with the appropriate .properties in place already prompts me for the keystore passwords (there are 2). The itunes deployment done through xCode archiver also asks for a password once. I think prompting for passwords in the

RE: Independent platform release summary

2014-10-09 Thread Treggiari, Leo
I’ll have to admit that this seems a bit weird. That is, independent versions of the CLI and platforms, with a “Cordova release” named “something” – e.g. a date? Imagine a user wants to know whether the new whitelist entry in config.xml is supported in the versions of CLI and platforms that

Re: Build signed archives using CLI

2014-10-09 Thread Jesse
Yeah, I agree, the password prompt makes sense, I relax my position against prompting. .. and config.xml may not be the best, I was just hoping to avoid another configuration file discussion. I believe MS already has the answers to all these questions since Visual Studio can do all of this, so

RE: Build signed archives using CLI

2014-10-09 Thread Subhag Oak
Adding to what Shazron said, isn't config.xml supposed to be considered as app-wide settings/properties? Typically packaging information is per platform and hence in my opinion, should be decoupled from config settings. Jesse, I am working on a documentation that I will share out soon for the

[GitHub] cordova-plugin-file pull request: [fxos] update __format__ to matc...

2014-10-09 Thread zalun
Github user zalun commented on the pull request: https://github.com/apache/cordova-plugin-file/pull/85#issuecomment-58570309 :) 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

[GitHub] cordova-plugin-file pull request: [fxos] update __format__ to matc...

2014-10-09 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/cordova-plugin-file/pull/85 --- 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

RE: Independent platform release summary

2014-10-09 Thread Chuck Lantz
Okay - so - there's a pretty nasty CLI blocker bug right now. Plugins with dependencies don't install (this affects all platforms). In my opinion, we need to get a CLI release out really soon. Are we closed on this topic, or do we need to look at doing the old process to get this out the

Re: Independent platform release summary

2014-10-09 Thread Steven Gill
I think vladimir fixed the bug. We just need to release now. Only thing holding back the release now is consensus on the version of the cli. It seemed like most people were leaning toward 10.0.0. Should I move forward with that? I would just have to branch + pin deps Leo the documentation

RE: Build signed archives using CLI

2014-10-09 Thread Subhag Oak
Here is the link to the proposal: https://docs.google.com/document/d/1tJQ9OoGrrMhZcLI3mg46rGzAfbiQu9PuNBL1auAMGFM/edit?usp=sharing Jump on it people :) Subhag Oak  |  Senior Program Manager Visual Studio, Client Tools s...@microsoft.com 425 707 5598 office -Original Message- From:

[GitHub] cordova-coho pull request: Added npm rc publish step

2014-10-09 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/cordova-coho/pull/49 --- 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 feature

Re: Independent platform release summary

2014-10-09 Thread Terence M. Bandoian
How about 1.0.0 for the CLI version? -Terence On 10/9/2014 4:13 PM, Steven Gill wrote: I think vladimir fixed the bug. We just need to release now. Only thing holding back the release now is consensus on the version of the cli. It seemed like most people were leaning toward 10.0.0. Should I

Re: Independent platform release summary

2014-10-09 Thread Shazron
We're not going backwards. What happens when we hit 3.0.0, an already re-leased version? On Thu, Oct 9, 2014 at 2:27 PM, Terence M. Bandoian tere...@tmbsw.com wrote: How about 1.0.0 for the CLI version? -Terence On 10/9/2014 4:13 PM, Steven Gill wrote: I think vladimir fixed the bug. We

Re: Independent platform release summary

2014-10-09 Thread Steven Gill
Unfortunately, that would cause more confusion. I am opposed to lowering the current version. It would cause havoc once the version worked it's way back up to 3. On Thursday, October 9, 2014, Terence M. Bandoian tere...@tmbsw.com wrote: How about 1.0.0 for the CLI version? -Terence On

RE: Independent platform release summary

2014-10-09 Thread Chuck Lantz
Yeah agreed - Vladimir squashed the bug and what was at once point to be called 3.7.0 has been mainly waiting on a version number. Personally I am fine with 10.0.0 or 5.0.0 - Either send the message that platform versions are divorced from the CLI from a versioning perspective (though behavior

Re: Independent platform release summary

2014-10-09 Thread Parashuram Narasimhan (MS OPEN TECH)
Lets start with a vote for 10.0.0 ? And if someone feels strongly about calling it something the vote could be cancelled !! On 10/9/14, 2:41 PM, Chuck Lantz cla...@microsoft.com wrote: Yeah agreed - Vladimir squashed the bug and what was at once point to be called 3.7.0 has been mainly waiting

Re: Independent platform release summary

2014-10-09 Thread Steven Gill
I don't think a vote is necessary. I'd hate to see us resort to voting to solve problems. Voting should be a last resort if consensus is split. I don't see that in this scenario. I propose we bumb the version up to 10.0.0. On Thu, Oct 9, 2014 at 2:45 PM, Parashuram Narasimhan (MS OPEN TECH)

Re: Independent platform release summary

2014-10-09 Thread Jesse
+1 to not voting ;) , it implies we will wait 72 hours before moving on. How about if anyone is completely against 10.0.0 they voice it here, in the next couple hours, otherwise we move forward. @purplecabbage risingj.com On Thu, Oct 9, 2014 at 2:52 PM, Steven Gill stevengil...@gmail.com wrote:

[GitHub] cordova-plugin-geolocation pull request: CB-7754 - [Geolocation] S...

2014-10-09 Thread shazron
GitHub user shazron opened a pull request: https://github.com/apache/cordova-plugin-geolocation/pull/28 CB-7754 - [Geolocation] Support Background Location tracking You can merge this pull request into a Git repository by running: $ git pull

RE: Independent platform release summary

2014-10-09 Thread Chuck Lantz
+1 -Chuck -Original Message- From: Jesse [mailto:purplecabb...@gmail.com] Sent: Thursday, October 9, 2014 2:55 PM To: dev@cordova.apache.org Subject: Re: Independent platform release summary +1 to not voting ;) , it implies we will wait 72 hours before moving on. How about if anyone

[GitHub] cordova-plugin-geolocation pull request: CB-7754 - [Geolocation] S...

2014-10-09 Thread shazron
Github user shazron commented on the pull request: https://github.com/apache/cordova-plugin-geolocation/pull/28#issuecomment-58585067 Adding it to this repo because it logically belongs here. Either here, or the cordova-plugins repo. I would not expect this sub-plugin to be

Re: Independent platform release summary

2014-10-09 Thread Terence M. Bandoian
Oh, well, scratch that idea. I didn't think about that and wasn't trying to be smart. It's just that you're establishing a new way of looking at and doing things and there needs to be a really clear distinction between CLI and the platforms. Seems very significant to me. Huge even.

Re: Independent platform release summary

2014-10-09 Thread Parashuram Narasimhan (MS OPEN TECH)
I meant tag and start the vote for the next release :) On 10/9/14, 3:01 PM, Chuck Lantz cla...@microsoft.com wrote: +1 -Chuck -Original Message- From: Jesse [mailto:purplecabb...@gmail.com] Sent: Thursday, October 9, 2014 2:55 PM To: dev@cordova.apache.org Subject: Re: Independent

[GitHub] cordova-plugin-camera pull request: A __bridge_retained cast incre...

2014-10-09 Thread krisselden
GitHub user krisselden opened a pull request: https://github.com/apache/cordova-plugin-camera/pull/51 A __bridge_retained cast increments the count and requires a CFRelease, ... ...this should just be __bridge a cast since it isn’t being held onto by this method. You can merge

[GitHub] cordova-plugin-camera pull request: A __bridge_retained cast incre...

2014-10-09 Thread XiMnet
Github user XiMnet commented on the pull request: https://github.com/apache/cordova-plugin-camera/pull/51#issuecomment-58593280 Hi, nbsp; Please be informed that we will be away from 10 Oct ndash; 12 Oct (Fri ndash; Sun) for company retreat. nbsp;

[GitHub] cordova-plugin-camera pull request: A __bridge_retained cast incre...

2014-10-09 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/cordova-plugin-camera/pull/51 --- 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

[GitHub] cordova-ios pull request: CB-7493 Adds test-build command to packa...

2014-10-09 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/cordova-ios/pull/103 --- 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 feature