[GitHub] cordova-plugin-camera pull request: CB-9490 Fixed cleanup function

[sgrebnov]

Github user sgrebnov commented on a diff in the pull request:


https://github.com/apache/cordova-plugin-camera/pull/115#discussion_r41742198
  
--- Diff: src/android/CameraLauncher.java ---
@@ -386,12 +387,10 @@ private void processResultFromCamera(int destType, 
Intent intent) throws IOExcep
 FileHelper.stripFileProtocol(this.croppedUri.toString()) :
 FileHelper.stripFileProtocol(this.imageUri.toString());
 
-if (this.encodingType == JPEG) {
-try {
-//We don't support PNG, so let's not pretend we do
-exif.createInFile(sourcePath);
-exif.readExifData();
-rotate = exif.getOrientation();
+//We don't support PNG, so let's not pretend we do
--- End diff --

I don't think this change should be in scope of this PR (I know there is a 
person working on .PNG support so this will just bring merge conflict) 


---
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 is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

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

[GitHub] cordova-plugin-camera pull request: CB-9490 Fixed cleanup function

[sgrebnov]

Github user sgrebnov commented on a diff in the pull request:


https://github.com/apache/cordova-plugin-camera/pull/115#discussion_r41742092
  
--- Diff: src/android/CameraLauncher.java ---
@@ -241,6 +241,7 @@ public void takePicture(int returnType, int 
encodingType) {
  * @return a File object pointing to the temporary picture
  */
 private File createCaptureFile(int encodingType) {
+File photo = null;
--- End diff --

Looks like this variable is not used


---
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 is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

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

[GitHub] cordova-app-hello-world pull request: CB-9712 CLI 5.3 breaks with ...

[asfgit]

Github user asfgit closed the pull request at:

https://github.com/apache/cordova-app-hello-world/pull/14


---
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 is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

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

[GitHub] cordova-lib pull request: Test cb 9712

[daserge]

Github user daserge closed the pull request at:

https://github.com/apache/cordova-lib/pull/317


---
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 is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

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

[GitHub] cordova-lib pull request: CB-7015 Add support for parent and custo...

[mbektchiev]

Github user mbektchiev commented on the pull request:

https://github.com/apache/cordova-lib/pull/290#issuecomment-147354095
  
Because these dependencies are only when using gradle. For projects that 
work with ant you have no way to refer a single jar.


---
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 is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

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

[GitHub] cordova-plugin-camera pull request: CB-9490 Fixed cleanup function

[sgrebnov]

Github user sgrebnov commented on the pull request:


https://github.com/apache/cordova-plugin-camera/pull/115#issuecomment-147357893
  
Reviewed: :+1: thank you @cojomojo!
@vladimir-kotikov, I know that you are working on some other improvements 
so please merge this along with your work! 


---
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 is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

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

[GitHub] cordova-plugin-camera pull request: CB-9745 Camera plugin docs sho...

[asfgit]

Github user asfgit closed the pull request at:

https://github.com/apache/cordova-plugin-camera/pull/129


---
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 is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

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

[GitHub] cordova-lib pull request: CB-9712 CLI 5.3 breaks with node 3.3.3

[asfgit]

Github user asfgit closed the pull request at:

https://github.com/apache/cordova-lib/pull/316


---
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 is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

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

[GitHub] cordova-plugin-camera pull request: CB-9490 Fixed cleanup function

[cojomojo]

Github user cojomojo commented on the pull request:


https://github.com/apache/cordova-plugin-camera/pull/115#issuecomment-147434553
  
@sgrebnov Something got messed up when I merged in the latest before 
resubmitting. I doubt this is even valid Java right now, I'll fix it later 
today and then we should be good to go. 


---
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 is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

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

PlatformApi implementation for cordova-android

[Vladimir Kotikov (Akvelon)]

Hi, list.

I've finished with refactoring cordova-android tooling to expose PlatformApi 
contract, so platform could be used as a regular node module.
You can review pull request here: 
https://github.com/apache/cordova-android/pull/226

A few highlights on this PR:
* Platform now exposes all functionality, like build/run/create, etc. through 
Api.js.
* 'prepare' logic from cordova-lib/metadata/android_parser copied into prepare 
module.
* build script refactored and split into number of 'builders': GradleBuilder, 
AntBuilder
* All the logic that deals with Androidmanifest.xml factored into 
AndroidManifest class
* exec and spawn modules are removed in favor of cordova-common

For more details see PR.

Would love to hear any comments/feedback, if possible.

-
Best regards, Vladimir

[GitHub] cordova-lib pull request: CB-9758 Mobilespec crashes adding plugin...

[daserge]

Github user daserge commented on the pull request:

https://github.com/apache/cordova-lib/pull/318#issuecomment-147455513
  
Don't these tests do the same thing?
Or is there any state changing between tests I'm missing?


---
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 is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

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

[GitHub] cordova-plugin-camera pull request: CB-9490 Fixed cleanup function

[cojomojo]

Github user cojomojo commented on the pull request:


https://github.com/apache/cordova-plugin-camera/pull/115#issuecomment-147443350
  
@infil00p Good idea, let me know when it's merged


---
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 is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

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

[GitHub] cordova-docs pull request: Added Graded App to Showcase

[geooot]

GitHub user geooot opened a pull request:

https://github.com/apache/cordova-docs/pull/383

Added Graded App to Showcase



You can merge this pull request into a Git repository by running:

$ git pull https://github.com/geooot/cordova-docs-graded cordova-website

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/cordova-docs/pull/383.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #383


commit 2331307afecad00ee866bfdeca1e69368bd6e50c
Author: geooot 
Date:   2015-10-12T15:26:32Z

Added Graded App to featured apps




---
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 is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

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

[GitHub] cordova-plugin-camera pull request: CB-9490 Fixed cleanup function

[infil00p]

Github user infil00p commented on the pull request:


https://github.com/apache/cordova-plugin-camera/pull/115#issuecomment-147443002
  
Can this wait until after my Cordova-Android 5.0.x pull request is merged 
in? checkForDuplicateImage requires a permission on Android 6.0+ and I want to 
make sure that we don't accidentally add something without a permissions check.


---
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 is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

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

[GitHub] cordova-lib pull request: CB-9758 Mobilespec crashes adding plugin...

[daserge]

Github user daserge commented on the pull request:

https://github.com/apache/cordova-lib/pull/318#issuecomment-147445435
  
The current code results in such an entry in `mobilespec-Info.plist`:
```xml


```
This happens with mobilespec `config.xml`' [``](https://github.com/apache/cordova-mobile-spec/blob/78a865153615ae20b9d8b6305b7b1a49496bbbf7/config.xml#L55)(`scheme:*`
 if generalize) and also for `http://*` or `https://*`.


---
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 is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

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

[GitHub] cordova-lib pull request: CB-9758 Mobilespec crashes adding plugin...

[daserge]

Github user daserge commented on the pull request:

https://github.com/apache/cordova-lib/pull/318#issuecomment-147447268
  
BTW `` and `` don't crash and result in 
```xml
null

  NSExceptionAllowsInsecureHTTPLoads
  

```
and 
```xml
null

```
correspondingly.
Not sure if they work in this case though.


---
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 is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

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

[GitHub] cordova-lib pull request: CB-9782 Update create/update signatures ...

[vladimir-kotikov]

GitHub user vladimir-kotikov opened a pull request:

https://github.com/apache/cordova-lib/pull/320

CB-9782 Update create/update signatures for PlatformApi polyfill

This PR updates parameters set for `PlatformApi.create/update` methods 
according to https://github.com/apache/cordova-android/pull/226

You can merge this pull request into a Git repository by running:

$ git pull https://github.com/MSOpenTech/cordova-lib CB-9782

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/cordova-lib/pull/320.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #320


commit 4a650f9a43f4883e16bef4f75e4ae440c7785aad
Author: Vladimir Kotikov 
Date:   2015-10-09T14:55:04Z

Update create/updatePlatform Api according to android changes at 
@msopentech/cordova-android#platformApi




---
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 is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

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

[GitHub] cordova-lib pull request: CB-9758 Mobilespec crashes adding plugin...

[shazron]

Github user shazron commented on the pull request:

https://github.com/apache/cordova-lib/pull/318#issuecomment-147449612
  
Great, since you've identified the stuff that crashes, you should add tests 
for them in the jasmine tests (there are already tests for other cases).


---
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 is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

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

[GitHub] cordova-lib pull request: CB-9758 Mobilespec crashes adding plugin...

[shazron]

Github user shazron commented on the pull request:

https://github.com/apache/cordova-lib/pull/318#issuecomment-147454277
  
No, its correct. The two tags are different.


---
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 is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

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

[GitHub] cordova-lib pull request: CB-9758 Mobilespec crashes adding plugin...

[shazron]

Github user shazron commented on the pull request:

https://github.com/apache/cordova-lib/pull/318#issuecomment-147457505
  
They do the same thing, almost. The only thing different is access tags do 
not handle wildcards in the schemes. I've opted to keep them separate for 
completeness, and ease of maintenance. 

Ideally the allow-navigation test should be part of the whitelist plugin, 
since it is only part of that. 


---
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 is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

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

cordova-vm.apache.org http server issue

[Alexander Sorokin (Akvelon)]

Hi guys,

Currently many file-transfer plugin tests are failing on all platforms because 
they try to download from:

http://cordova-vm.apache.org:5000

which seem to be offline.

Any ideas on how we can fix it?

Thanks,
Alexander Sorokin

[GitHub] cordova-android pull request: CB-9782 Implements PlatformApi contr...

[infil00p]

Github user infil00p commented on the pull request:

https://github.com/apache/cordova-android/pull/226#issuecomment-147458901
  
Why is all of element-tree and the other npm dependencies checked in?  That 
seems very odd, and makes it harder to figure out what's going on.


---
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 is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

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

[GitHub] cordova-plugin-camera pull request: CB-9490 Fixed cleanup function

[cojomojo]

Github user cojomojo commented on the pull request:


https://github.com/apache/cordova-plugin-camera/pull/115#issuecomment-147473478
  
@sgrebnov @vladimir-kotikov OK, I fixed the issue caused by a bad merge. 
This PR is ready once we see how @infil00p 's PR for Cordova-Android 5.0.x  and 
the permissions issue he mentioned affects things.


---
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 is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

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

[GitHub] cordova-browser pull request: Update to use new 'express' implemen...

[asfgit]

Github user asfgit closed the pull request at:

https://github.com/apache/cordova-browser/pull/21


---
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 is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

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

Apache Cordova Course

[Parashuram N]

Hi,

I will be teaching a course on "Creating responsive and performant Hybrid 
Mobile Apps using Apache Cordova" at the HTML5DevConf. 
http://html5devconf.com/training.html#parashuram-cordova
Here is what I plan to talk about - 
https://github.com/panarasi/cordova-tutorial. Course contents - 
https://github.com/panarasi/cordova-tutorial/wiki
Would love to hear any feedback you folks may have. Are there any important 
Cordova topics I am missing. Would there to topics that folks would like to 
hear about ?

RE: Cordova Face to Face Meeting

[Parashuram N]

The Cordova face to face is less than 2 days away !! 

If you have not yet registered, please add your name here - 
https://docs.google.com/forms/d/1peBvvVXocf8_Z-JzB12HwRd83YtFkMPITkpsLutqRxg/viewform.
 This is just to get an idea about the number of people attending. 

LIVE BROADCAST: 
We will also have the morning session broadcast live - so even if you are not 
able to attend, you should be able to participate in the conversation. As we 
break into smaller groups, each group can set up a Google hangout to keep the 
channel open with remote participants. I will send out the links tomorrow on 
this mailing list. Also watch out for the links on our Slack group. 

LOGISTICS:
Address: Building 33, 16070 NE 36th Way, Redmond, WA 98052. The room is called 
RAINIER. 

- The meeting starts at 10, but folks will start showing up by 8:30. Breakfast 
and lunch will be served on both days. 
- I also heard that a few folks were planning to informally get together at 
8:30 on the first day to talk about Cloud Services for Cordova. This is NOT an 
Apache Cordova discussion, but if you would like to hear what people talk about 
Cloud services, you could join them. 
- If you are staying at the Sliver Cloud Inn, note that it is still a decent 
walk to the nearest Microsoft Office building. If would be better to get ask if 
the hotel provides Shuttles to Microsoft Buildings. If you are taking a 
shuttle, please note that you need to have a Microsoft employee accompany you. 
Ping me (on slack or send me an email at panar...@microsoft.com) and I can come 
over to pick you up. 

OTHERS: 
- We could also plan to get drinks on the evening of the first day. Any 
preferences/suggestions ? 
- The tentative agenda is here - 
https://github.com/cordova/cordova-discuss/blob/master/proposals/Face2Face-Oct15.md.
 Please add any topics you would talk about here to help better organized.

CONTACTS:
If you have any trouble locating the place or getting here, send me an email at 
panar...@microsoft.com.


Looking forward to a productive 2 days !!  

-Original Message-
From: Martijn Walraven [mailto:martijnwalra...@meteor.com] 
Sent: Sunday, October 11, 2015 9:04 AM
To: dev@cordova.apache.org
Subject: Re: Cordova Face to Face Meeting

I created a PR with some additions 
(https://na01.safelinks.protection.outlook.com/?url=https%3a%2f%2fgithub.com%2fcordova%2fcordova-discuss%2fpull%2f18=01%7c01%7cpanarasi%40microsoft.com%7c953f3df1954a4c96d00f08d2d2559714%7c72f988bf86f141af91ab2d7cd011db47%7c1=oLOO9sBbdrjkKbsB9jvT4LguXJygVVjp4arH%2fX%2bPcq4%3d).
 Hope this makes sense.

Really looking forward to meeting everyone and discussing some of these topics. 
I’ll also try to make it to the breakfast on Wednesday.

> On 08 Oct 2015, at 19:39, Nikhil Khandelwal  wrote:
> 
> Thanks, Martijn! This is a great outline. I merged most of what you 
> mentioned here: 
> https://na01.safelinks.protection.outlook.com/?url=https%3a%2f%2fgithu
> b.com%2fcordova%2fcordova-discuss%2fblob%2fmaster%2fproposals%2fFace2F
> ace-Oct15.md=01%7c01%7cpanarasi%40microsoft.com%7c953f3df1954a4c9
> 6d00f08d2d2559714%7c72f988bf86f141af91ab2d7cd011db47%7c1=TzV5ZET
> CxGHKCKU8SAPa8P0hd8oe5jB4aHxizWrnoKw%3d 
>  ub.com%2fcordova%2fcordova-discuss%2fblob%2fmaster%2fproposals%2fFace2
> Face-Oct15.md=01%7c01%7cpanarasi%40microsoft.com%7c953f3df1954a4c
> 96d00f08d2d2559714%7c72f988bf86f141af91ab2d7cd011db47%7c1=TzV5ZE
> TCxGHKCKU8SAPa8P0hd8oe5jB4aHxizWrnoKw%3d>
> 
> Feel free to send a PR if I missed out anything. It will be great if others 
> can add or +1 areas of their interest for the Face2Face.
> 
> -Nikhil
> 
> -Original Message-
> From: Martijn Walraven [mailto:martijnwalra...@meteor.com 
> ]
> Sent: Monday, October 5, 2015 6:35 AM
> To: dev@cordova.apache.org 
> Subject: Re: Cordova Face to Face Meeting
> 
> I’m new here, and I don’t know how these meetings usually work, but I thought 
> it might make sense to structure the sessions around questions instead of 
> smaller topics. This way, we can get a more open discussion going and talk 
> about these topics within a larger context.
> 
> Some of the questions I would be interested in discussing are:
> 
> - How can we improve the development experience?
>  - What can we learn from the React Native developer experience?
>- Replacing 'white screen of death' with an error reporting screen
>- Adding a development/debugger menu
>  - How can we make platform installation easier?
>- Improve requirements checking and error messages
>- Install SDKs with one command (TACO)
> 
> - How can we improve the plugin ecosystem?
>  - Plugin dependencies and version constraints
>  - Validated platforms and plugins (TACO)
>  - Improving native dependencies (CocoaPods for iOS?)
>  - What can we learn from the React Native plugin API?
> 
> - How can we better 

[GitHub] cordova-lib pull request: Update 'serve' to use 'express' implemen...

[asfgit]

Github user asfgit closed the pull request at:

https://github.com/apache/cordova-lib/pull/319


---
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 is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

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

[GitHub] cordova-lib pull request: Install and link cordova-serve before ru...

[TimBarham]

GitHub user TimBarham opened a pull request:

https://github.com/apache/cordova-lib/pull/321

Install and link cordova-serve before running cordova-lib tests.



You can merge this pull request into a Git repository by running:

$ git pull https://github.com/MSOpenTech/cordova-lib install-cordova-serve

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/cordova-lib/pull/321.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #321


commit 2bf248a07eb0299d30314503e6572cfe0df7e01b
Author: Tim Barham 
Date:   2015-10-12T20:37:01Z

Install and link cordova-serve before running cordova-lib tests.




---
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 is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

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

[GitHub] cordova-plugin-splashscreen pull request: fix link to splash and i...

[JhonataRibeiro]

GitHub user JhonataRibeiro opened a pull request:

https://github.com/apache/cordova-plugin-splashscreen/pull/63

fix link to splash and icon doc



You can merge this pull request into a Git repository by running:

$ git pull https://github.com/JhonataRibeiro/cordova-plugin-splashscreen 
master

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/cordova-plugin-splashscreen/pull/63.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #63


commit 1741512ca1f87de6dd2eea131ba1a979a93b5993
Author: Jhonata Ribeiro 
Date:   2015-10-12T19:28:38Z

fix link to splash and icon doc




---
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 is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

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

[GitHub] cordova-ios pull request: CB-9719 set allow_non_modular_includes t...

[shazron]

Github user shazron commented on the pull request:

https://github.com/apache/cordova-ios/pull/171#issuecomment-147561614
  
I fixed the test failure above and ran the tests myself, they pass.


---
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 is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

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

[GitHub] cordova-docs pull request: Please add One Verse BIBLE app

[ozexpert]

GitHub user ozexpert opened a pull request:

https://github.com/apache/cordova-docs/pull/384

Please add One Verse BIBLE app

Hi, I've built the first BIBLE app using Cordova (so far I know).
Please take a look and add my beautifully crafted Cordova app on the 
official cordova website!

Thank you,
Young
- CodeInFaith

You can merge this pull request into a Git repository by running:

$ git pull https://github.com/ozexpert/cordova-docs cordova-website

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/cordova-docs/pull/384.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #384


commit dcdb064b8028b4867478472a05b64943c276bce1
Author: Young Park 
Date:   2015-10-13T01:43:47Z

added One Verse Bible app




---
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 is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

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

[GitHub] cordova-docs pull request: Buildr added to the showcase

[dblotsky]

Github user dblotsky commented on the pull request:

https://github.com/apache/cordova-docs/pull/378#issuecomment-147530872
  
Merged. Thanks for the PR! Please feel free to close it and delete the 
branch.


---
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 is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

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

Re: cordova platform

[Shazron]

Hi Joerg,
Makes sense - users can add the ios platform on Windows even if not
supported. Please file a feature request in issues.cordova.io, thanks!


On Thu, Oct 8, 2015 at 1:08 PM, Joerg Holz  wrote:
> Hi,
>
> after calling cordova platform on my MacBook Pro I just get this list of 
> platforms:
>
> amazon-fireos, android, blackberry10, browser, firefoxos, ios, osx, webos
>
> Is this depending of the OS of the developer machine? I think it would be 
> better to show the full list of platforms.
>
> Joerg
>
> 
> Jörg Holz | +49-175-640 35 80
> h...@hamburg.de
>
> RELAUNCHED:
> http://www.workflow-management.net
> 
>

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

[GitHub] cordova-docs pull request: Keepe App Added to Showcase

[dblotsky]

Github user dblotsky commented on the pull request:

https://github.com/apache/cordova-docs/pull/376#issuecomment-147530905
  
Merged. Thanks for the PR! Please feel free to close it and delete the 
branch.


---
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 is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

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

Re: Cordova website redesign

[Raymond Camden]

+1 here too.

On Fri, Oct 9, 2015 at 1:46 PM, Jesse  wrote:
> This sounds more like consensus!
> I too am +1 on changing the typeface.
>
> Cheers,
>   Jesse
>
>> On Oct 8, 2015, at 11:02 AM, Parashuram N  wrote:
>>
>> +1 on the typeface - I think it does send the “modern-ness” message.
>>
>>
>>
>>
>>> On 10/8/15, 10:39 AM, "Carlos Santana"  wrote:
>>>
>>> + 1 on the typeface
>>> leave the logo as it is today (3d, gray, etc..)
>>>
 On Thu, Oct 8, 2015 at 12:26 AM Steven Gill  wrote:

 +1
> On Oct 7, 2015 9:15 PM, "Brian LeRoux"  wrote:
>
> Well said, and ya updated typeface looks slick. 
>> On Wed, Oct 7, 2015 at 6:10 PM Ryan J. Salva 
> wrote:
>
>> I'm personally a big fan of Ben's proposal. The typography really
 stands
>> out and gives the brand a strong, updated and modern presence. I know
>> there's a lot of pride in the heritage of the logo, and for that
 reason I
>> wouldn't dream of losing our beloved robot. However, the typography is
 a
>> different story.
>>
>> Is it absolutely necessary? Well... that really depends on your
>> perspective. From talking with many of you and other developers at
>> conferences and events over the past year, I've heard a repeating theme
>> that Cordova has a public-perception problem. A typography update won't
>> change that perception by itself, but it will demonstrate that Cordova
>> isn't staying still... that the project is evolving, growing and
> improving.
>>
>> 
>> rjs
>>
>> Ryan J. Salva  |  Principal Program Manager Lead
>> Visual Studio Tools for Apache Cordova
>> rsa...@microsoft.com
>> 206 612 5079 mobile
>>
>>
>>
>> -Original Message-
>> From: Tommy-Carlos Williams [mailto:to...@devgeeks.org]
>> Sent: Wednesday, October 7, 2015 12:21 PM
>> To: dev@cordova.apache.org
>> Subject: Re: Cordova website redesign
>>
>> Yeah... I kinda fell into the "not sure, wait and see what others
> thought"
>> category on this one.
>>
>> I would also like more discussion unless everyone really has no
 opinion.
>>
>>> On 8 Oct 2015, at 05:42, Jesse  wrote:
>>>
>>> I would suggest that the lack of response on the logo change means we
>> don't need it.
>>> I think there should at least be some discussion before moving on it.
>>>
 On Oct 7, 2015, at 11:31 AM, Brad Gashler 
> wrote:

 Hi Everyone,

 I wanted to follow up on the branding changes proposed by Ben Sperry
>> (his email is included below).

 I’m planning on implementing his changes, since no objections were
>> raised.  This includes the new logo, with its modern typeface, as shown
> in
>> picture A below.  Also, we may replace the background image with the
> simple
>> gradient as shown in picture B below.  Please let me know if there are
> any
>> concerns replacing this background image.

 Thanks,

 Brad Gashler


 A) New logo only



 B) New logo and new gradient background




 -Original Message-
 From: Ben Sperry [mailto:b...@ionic.io]
 Sent: Thursday, September 24, 2015 1:17 PM
 To: dev@cordova.apache.org
 Subject: Re: Cordova website redesign

 I really like where this update is going, and what it signals to the
 community: Cordova is a legitimate, modern tech solution that's
 still
>> very much alive and growing. The website look/feel has a lot of
> confidence,
>> love it!

 I'm curious what thoughts are around a Cordova logo update itself. I
 put together a version that I thought might support the evolution of
 Cordova into a more modern looking brand. Would love feedback on
 this
 early logo update mockup: regular
 <
 https://na01.safelinks.protection.outlook.com/?url=http%3a%2f%2fioni
 https://na01.safelinks.protection.outlook.com/?url=c-io-assets.s3.amazonaws.com=01%7c01%7cpanarasi%40microsoft.com%7c60f8e63b1c9a46d5e95208d2d00785b0%7c72f988bf86f141af91ab2d7cd011db47%7c1=r7JctK6wY1QhZ2GZNlumefajI4vAGiseq9LrehDxEPQ%3d
 %2fcordova%2fcordova-logo-3d-proposal.png
 =01%7c01%7cbradga%40microsoft.com
 %7c46d14102d64c4e5de06408d2c51d
 409d%7c72f988bf86f141af91ab2d7cd011db47%7c1=oKVaCThqWbhTgak3efA
 aM%2f3f1d1wGyqazXPdboZjsAk%3d>
 / flat
 <
 https://na01.safelinks.protection.outlook.com/?url=http%3a%2f%2fioni
 

[GitHub] cordova-ios pull request: CB-8789 Asset Catalog support

[asfgit]

Github user asfgit closed the pull request at:

https://github.com/apache/cordova-ios/pull/148


---
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 is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

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

[GitHub] cordova-plugin-media pull request: Streaming & Background threadin...

[ghenry22]

Github user ghenry22 commented on the pull request:


https://github.com/apache/cordova-plugin-media/pull/62#issuecomment-147566101
  
@dts - thanks for the suggestion, I have implemented this and hopefully 
also fixed for playback from local files as well.

@purplecabbage - updated with implementation for stop() for compliance as 
requested.

incremented the version number so that I can see which version is actually 
installed.  if you install from my branch it should show version 1.0.3-dev when 
you run cordova plugin list.


---
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 is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

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

[GitHub] cordova-docs pull request: Added Graded App to Showcase

[geooot]

Github user geooot closed the pull request at:

https://github.com/apache/cordova-docs/pull/383


---
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 is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

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

[GitHub] cordova-docs pull request: Added Graded App to Showcase

[dblotsky]

Github user dblotsky commented on the pull request:

https://github.com/apache/cordova-docs/pull/383#issuecomment-147574304
  
Thank you for the PR! It's merged now, and you may safely close it and 
delete the branch.


---
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 is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

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

[GitHub] cordova-cli pull request: CB-9784 Remove CLI logger levels prefixe...

[shazron]

Github user shazron commented on the pull request:

https://github.com/apache/cordova-cli/pull/224#issuecomment-147543581
  
+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
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

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

[GitHub] cordova-plugin-media pull request: Streaming & Background threadin...

[dts]

Github user dts commented on the pull request:


https://github.com/apache/cordova-plugin-media/pull/62#issuecomment-147547878
  
Good stuff - only downside is it currently always reports '0' as the 
duration of the resource.


---
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 is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

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

[GitHub] cordova-cli pull request: CB-9784 Remove CLI logger levels prefixe...

[gorkem]

Github user gorkem commented on the pull request:

https://github.com/apache/cordova-cli/pull/224#issuecomment-147553471
  
+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
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

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

[GitHub] cordova-cli pull request: CB-9784 Remove CLI logger levels prefixe...

[nikhilkh]

Github user nikhilkh commented on a diff in the pull request:

https://github.com/apache/cordova-cli/pull/224#discussion_r41816178
  
--- Diff: src/logger.js ---
@@ -59,13 +58,12 @@ function formatError(error, isVerbose) {
 logger.log = function (logLevel, message) {
 if (this.levels[logLevel] >= this.levels[this.logLevel]) {
 var isVerbose = this.logLevel === 'verbose';
-var prefix = this.prefixes[logLevel] ? this.prefixes[logLevel] + 
': ' : '';
 
 if(message instanceof Error) {
 message = formatError(message, isVerbose);
 }
 
-message = prefix + message + EOL;
+message = message + EOL;
 
 if (!this.cursor) {
 this.output.write(message);
--- End diff --

I just realized that we should write error messages to stderr instead of 
output - is that correct?


---
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 is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

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

[GitHub] cordova-ios pull request: add JIRA issue tracker link

[asfgit]

Github user asfgit closed the pull request at:

https://github.com/apache/cordova-ios/pull/161


---
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 is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

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

RE: [DISCUSS] CORDOVA, LIVERELOAD & GESTURES SYNC

[Mefire O .]

Thanks for the feedback, Carlos.

- I think the issue you are raising is an important one. I'm planning on 
re-examining it and maybe adding the ability to handle multiple IP addresses in 
a future version.
   For now, I'll be focusing on basic scenarios and making sure those work well 
as advertised.

- I think we are seeing some pattern around some of Cordova's plugins being 
'tooling plugins' instead of 'runtime plugins', and for those tooling plugins, 
whose goal
  Is not to end up being packaged and ran inside a cordova app, it makes a lot 
of sense to have the capability to install dependencies from package.json.
  I think cordova itself should be installing those dependencies while doing 
the plugin install, instead of this being left to plugins themselves.

- I've added a PR to allow for the capability of ignoring and not watching some 
files : https://github.com/omefire/cordova-plugin-livereload/pull/33

- Like you said, Let's also sync up Face to Face at the Meetup on Oct 14 and 
15: How should Cordova handle plugin dependencies and LiveReload.
  I'll add it to the calendar.


Thanks,

-Original Message-
From: Carlos Santana [mailto:csantan...@gmail.com] 
Sent: Friday, October 9, 2015 6:21 AM
To: dev@cordova.apache.org
Subject: Re: [DISCUSS] CORDOVA AND LIVERELOAD

+10, you cracked one of the big problems Omar.
watch project www, but serve/livereload platform www

#Multiple External IP addresses for livereload on device I see you use a 
browsersync method bs.options.getIn(['urls', 'external']);

But those this handle the use case when there are multiple ip adresses.

This is something we hit here at work with our CLI, and we handle in the same 
form as Ionic CLI.
To reproduce usually I hit when I'm at home and connected to work using VPN. 
then my system has two external ips, but only one of them my iphone can reach.
9.x.x.x (work vpn)
10.x.x.x (home wifi)
and my iphone can only connect to 10.x.x.x This is not a check you have to do 
all them time, you can save last value selected, and next time if that ip is 
still valid is a good chance that's the one you want to use again avoiding 
asking the user every time.
https://na01.safelinks.protection.outlook.com/?url=https%3a%2f%2fgithub.com%2fdriftyco%2fionic-app-lib%2fblob%2fe9d17cd81fb5fea29229b7bfa19dce765361f243%2flib%2fserve.js%23L843=01%7c01%7commenjik%40microsoft.com%7c4922aee79f33467a390208d2d0ac7e9e%7c72f988bf86f141af91ab2d7cd011db47%7c1=i2jdC3TPVDRAG99BcqaYcQ4uvYhemZkW4laMh9PJUtw%3d


#NPM dependencies

after_plugin_add dependenciesInstall

I don't know how I feel about a plugin automatically running npm install on an 
users to satisfy plugin hook nodejs scripts

one way is to allow user to install npm modules in the root of the cordova 
project, node will walk up the directory tree and find the dependency there.

Some of those dependencies you can get a reference from cordova-lib with
context.requireCordovaModule()

So no need to have them in package.json

The reason I bring this things up is because we at work we are shipping some 
cordova plugins, and phase the dilema of npm dependencies for the plugin hooks 
that we use.
1. QA, we want to use the version of the dependency we tested with (if we use 
requireCordovaModule we might not get the same version) 2. Legal, we need to 
clear with legal team and notices any 3rd party dependency so user is aware of 
any 3rd party software we are using 3. Node/NPM not installed, believe it or 
not we shipped node and in our product we provide a cli that takes care of 
nodejs, and cordova-cli for the user. user only need to download our cli and 
set it in their path.

So for now what we do is to bundle the dependencies with the plugin, same 
solution we have today for platforms the cordova platforms bundle the npm 
dependencies with the git repo.

I would like to discuss livereload and npm dependences topic in F2F with folks 
that are interested on this.


On Thu, Oct 8, 2015 at 12:38 PM Carlos Santana  wrote:

> Hi Omar
> improvements are good direction.
>
> How do I specify which files to watch or ignore?
>
> Maybe by default it watches every file in www/**/*.* But allow 
> --livereload to take a value of glob/match ="www/js/*.*"
>
> Another use case is the one that the browsersync plugin offers is to 
> specify ignore by default watch everything but except 
> --ignore=lib/**/*.* 
> https://na01.safelinks.protection.outlook.com/?url=https%3a%2f%2fgithu
> b.com%2fnparashuram%2fcordova-plugin-browsersync%23ignoring-files
> =01%7c01%7commenjik%40microsoft.com%7c4922aee79f33467a390208d2d0ac7e9e
> %7c72f988bf86f141af91ab2d7cd011db47%7c1=iTt9V%2fsL%2bSHyH5UTlI30
> k9E6xfB5dNiSLrLPwmaF2qs%3d
>
>
>
> On Thu, Oct 8, 2015 at 1:26 AM Mefire O.  wrote:
>
>> Thanks for the feedback, it was received.
>>
>> As follow up actions, I've done the following changes:
>>
>> - Changed things into a plugin
>> - Added support for multiplatform support: `cordova run 

[GitHub] cordova-cli pull request: CB-9784 Remove CLI logger levels prefixe...

[TimBarham]

Github user TimBarham commented on the pull request:

https://github.com/apache/cordova-cli/pull/224#issuecomment-147536916
  
:+1: :smile: 


---
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 is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

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

[GitHub] cordova-android pull request: CB-9782 Implements PlatformApi contr...

[nikhilkh]

Github user nikhilkh commented on a diff in the pull request:

https://github.com/apache/cordova-android/pull/226#discussion_r41816386
  
--- Diff: bin/templates/cordova/Api.js ---
@@ -0,0 +1,506 @@
+/**
+Licensed to the Apache Software Foundation (ASF) under one
+or more contributor license agreements.  See the NOTICE file
+distributed with this work for additional information
+regarding copyright ownership.  The ASF licenses this file
+to you under the Apache License, Version 2.0 (the
+"License"); you may not use this file except in compliance
+with the License.  You may obtain a copy of the License at
+
+http://www.apache.org/licenses/LICENSE-2.0
+
+Unless required by applicable law or agreed to in writing,
+software distributed under the License is distributed on an
+"AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
+KIND, either express or implied.  See the License for the
+specific language governing permissions and limitations
+under the License.
+*/
+
+var Q = require('q');
+var fs = require('fs');
+var path = require('path');
+var shell = require('shelljs');
+
+var CordovaError = require('cordova-common').CordovaError;
+var PlatformJson = require('cordova-common').PlatformJson;
+var ActionStack = require('cordova-common').ActionStack;
+var AndroidProject = require('./lib/AndroidProject');
+var PlatformMunger = 
require('cordova-common').ConfigChanges.PlatformMunger;
+var PluginInfoProvider = require('cordova-common').PluginInfoProvider;
+
+var pluginHandlers = require('./lib/pluginHandlers');
+
+var PLATFORM = 'android';
+var GENERIC_EVENTS = new (require('events').EventEmitter)()
+.on('verbose', function (message) {
+if (process.argv.indexOf('-d') >= 0 || 
process.argv.indexOf('--verbose') >= 0)
+console.log(message);
+})
+.on('log', console.log)
+.on('warn', console.warn)
+.on('error', function (error) {
+if (process.argv.indexOf('-d') >= 0 || 
process.argv.indexOf('--verbose') >= 0) {
+console.error((error && error.message) || error);
--- End diff --

Should this be flipped? We should `stack` for verbose output and not 
otherwise? Also, does the message not show when we show stack? If I understand 
correctly, this is only for legacy scenarios - correct? Otherwise, the cli 
should eventually be doing the logging - correct?



---
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 is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

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

[GitHub] cordova-android pull request: CB-9782 Implements PlatformApi contr...

[nikhilkh]

Github user nikhilkh commented on a diff in the pull request:

https://github.com/apache/cordova-android/pull/226#discussion_r41816415
  
--- Diff: bin/templates/cordova/Api.js ---
@@ -0,0 +1,506 @@
+/**
+Licensed to the Apache Software Foundation (ASF) under one
+or more contributor license agreements.  See the NOTICE file
+distributed with this work for additional information
+regarding copyright ownership.  The ASF licenses this file
+to you under the Apache License, Version 2.0 (the
+"License"); you may not use this file except in compliance
+with the License.  You may obtain a copy of the License at
+
+http://www.apache.org/licenses/LICENSE-2.0
+
+Unless required by applicable law or agreed to in writing,
+software distributed under the License is distributed on an
+"AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
+KIND, either express or implied.  See the License for the
+specific language governing permissions and limitations
+under the License.
+*/
+
+var Q = require('q');
+var fs = require('fs');
+var path = require('path');
+var shell = require('shelljs');
+
+var CordovaError = require('cordova-common').CordovaError;
+var PlatformJson = require('cordova-common').PlatformJson;
+var ActionStack = require('cordova-common').ActionStack;
+var AndroidProject = require('./lib/AndroidProject');
+var PlatformMunger = 
require('cordova-common').ConfigChanges.PlatformMunger;
+var PluginInfoProvider = require('cordova-common').PluginInfoProvider;
+
+var pluginHandlers = require('./lib/pluginHandlers');
+
+var PLATFORM = 'android';
+var GENERIC_EVENTS = new (require('events').EventEmitter)()
+.on('verbose', function (message) {
+if (process.argv.indexOf('-d') >= 0 || 
process.argv.indexOf('--verbose') >= 0)
+console.log(message);
+})
+.on('log', console.log)
+.on('warn', console.warn)
+.on('error', function (error) {
+if (process.argv.indexOf('-d') >= 0 || 
process.argv.indexOf('--verbose') >= 0) {
+console.error((error && error.message) || error);
--- End diff --

Or cordova-lib needs to do the logging not to break existing users of 
cordova-lib.


---
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 is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

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

[GitHub] cordova-plugin-file-transfer pull request: CB-8721 - Better header...

[nikhilkh]

Github user nikhilkh commented on the pull request:


https://github.com/apache/cordova-plugin-file-transfer/pull/54#issuecomment-147560508
  
I think this change is not relevant anymore because of this commit: 
https://github.com/apache/cordova-plugin-file-transfer/commit/e8ca110a3026b74276f762a81bb10c4a04ade4fe


---
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 is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

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

[GitHub] cordova-docs pull request: Added Graded App to Showcase

[geooot]

Github user geooot commented on the pull request:

https://github.com/apache/cordova-docs/pull/383#issuecomment-147570978
  
Ok will do!


---
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 is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

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

Re: cordova-vm.apache.org http server issue

[Shazron]

Is this still an issue? I'm not quite sure who has access to this. I
believe Steve might.

On Mon, Oct 12, 2015 at 7:09 AM, Alexander Sorokin (Akvelon)
 wrote:
> Hi guys,
>
> Currently many file-transfer plugin tests are failing on all platforms 
> because they try to download from:
>
> http://cordova-vm.apache.org:5000
>
> which seem to be offline.
>
> Any ideas on how we can fix it?
>
> Thanks,
> Alexander Sorokin

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

[GitHub] cordova-cli pull request: CB-9784 Remove CLI logger levels prefixe...

[daserge]

GitHub user daserge opened a pull request:

https://github.com/apache/cordova-cli/pull/224

CB-9784 Remove CLI logger levels prefixes

`EventEmitter` usage provides info on an event level so the prefixing may 
be superfluous taking into account we have coloring distinction for event 
levels.

The change is proposed by @TimBarham.

[Jira issue](https://issues.apache.org/jira/browse/CB-9784)

You can merge this pull request into a Git repository by running:

$ git pull https://github.com/MSOpenTech/cordova-cli CB-9784

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/cordova-cli/pull/224.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #224


commit 96f811712db0cb6e6d844925eecee2b9183d2a62
Author: daserge 
Date:   2015-10-12T22:10:03Z

CB-9784 Remove CLI logger levels prefixes




---
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 is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

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

[GitHub] cordova-cli pull request: CB-9784 Remove CLI logger levels prefixe...

[nikhilkh]

Github user nikhilkh commented on the pull request:

https://github.com/apache/cordova-cli/pull/224#issuecomment-147556117
  
We should remove this: 
https://github.com/apache/cordova-cli/pull/224/files#diff-567859a4cc8b910de0454da79959bde6R53
 - I know a lot of downstream tools currently use Error: prefix to determine 
something is an error vs not.


---
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 is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

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

[GitHub] cordova-ios pull request: CB-9552 Updating linked platform removes...

[asfgit]

Github user asfgit closed the pull request at:

https://github.com/apache/cordova-ios/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 feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

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

[GitHub] cordova-lib pull request: CB-9758 Mobilespec crashes adding plugin...

[shazron]

Github user shazron commented on the pull request:

https://github.com/apache/cordova-lib/pull/318#issuecomment-147529842
  
looks good to me. Thanks @daserge!


---
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 is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

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

Re: Question on Contributing

[Shazron]

Hi Michael,
Regardless of the CCLA[1], you would need to file an iCLA[2] as well.
Please read: http://www.apache.org/dev/new-committers-guide.html#cla

[1] https://www.apache.org/licenses/cla-corporate.txt
[2] https://www.apache.org/licenses/icla.txt


On Thu, Oct 8, 2015 at 12:08 PM, Michael Zimmerman
 wrote:
> Hi, I am currently working on a proposal for my place of employment to 
> separate our front-end from our Grails backend. One of the requirements I was 
> tasked with was to ensure an SSO implementation functioned with Cordova, and 
> unfortunately the InAppBrowserPlugin had issues with this on Android.
>
> I went ahead and added a few utility functions to allow the JS to drive 
> sharing cookies between Android and the Cordova web view.
>
> I started to look at the contributing documentation and requirements and saw 
> a clause for Companies. Would this code I am contributing fall under this 
> clause? We are not explicitly using this for work, but I did develop it 
> because of a possible direction we may be going.
>
>
> The changes can be found at 
> https://github.com/Sinistralis/cordova-plugin-inappbrowser/commit/32bcda6e69be39e2cbef3f582ac696873bb0b2bf
>
>
> I’m still rather new to contributing to open source so sorry if this is a 
> silly question, I am hoping to contribute more in the future, but I want to 
> make sure I am following all the legalities and whatnot.
>
>
> Michael Zimmerman | Software Engineer
> Turning Technologies 255 West Federal Street Youngstown OH 44503
> Direct: 330-599-4921 | Main: 330-746-3015 | Toll Free: 866-746-3015 | Fax: 
> 330-884-6065
> mzimmer...@turningtechnologies.com | www.TurningTechnologies.com 
> 
>
> [Turning Technologies]
> Stay Connected - @TurningTech  ·   Find us 
> on 
> Facebook
> This email message, including any attached files, is for the sole use of the 
> intended recipients(s) and may contain confidential and/or privileged 
> information. Any unauthorized review, use, disclosure or distribution is 
> prohibited. If you are not the intended recipient or you wish to discontinue 
> receiving any future e-mails from the vendor featured in this message, please 
> send an email 
> request.
> Turning Green - Please consider the environment before printing this email

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

CB-9784 Remove CLI logger levels prefixes

[Sergey Shakhnazarov (Akvelon)]

Hello guys,

I wanted to discuss the change[1], which regards to removing the CLI logger 
level prefixes:
EventEmitter usage provides info on an event level so the prefixing may be 
superfluous taking into account we have coloring distinction for event levels.

What do you think?

[1]: https://github.com/apache/cordova-cli/pull/224

Best regards,
Sergey Shakhnazarov.

[GitHub] cordova-docs pull request: Added Graded App to Showcase

[dblotsky]

Github user dblotsky commented on the pull request:

https://github.com/apache/cordova-docs/pull/383#issuecomment-147569864
  
Thank you for the PR! Sorry for the trouble: could you please submit an 
image that is smaller than 128KiB?


---
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 is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

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

[GitHub] cordova-docs pull request: Added Clever Baby to the app showcase

[bymodude]

Github user bymodude closed the pull request at:

https://github.com/apache/cordova-docs/pull/377


---
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 is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

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

[GitHub] cordova-plugin-media pull request: Streaming & Background threadin...

[Alkashi]

Github user Alkashi commented on the pull request:


https://github.com/apache/cordova-plugin-media/pull/62#issuecomment-147611994
  
OK so I was not completely crazy. Many thanks to @ghenry22 and others.


---
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 is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

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

[GitHub] cordova-lib pull request: CB-9758 Mobilespec crashes adding plugin...

[daserge]

Github user daserge commented on the pull request:

https://github.com/apache/cordova-lib/pull/318#issuecomment-147529120
  
@shazron, could you please review again?


---
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 is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

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

[GitHub] cordova-ios pull request: CB-9685 A fix for the magnifying glass p...

[asfgit]

Github user asfgit closed the pull request at:

https://github.com/apache/cordova-ios/pull/164


---
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 is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

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

Re: Cordova website redesign

[Jesse]

These need to be updated as well :
http://cordova.apache.org/artwork/



My team is hiring!
@purplecabbage
risingj.com

On Mon, Oct 12, 2015 at 7:20 PM, Dmitry Blotsky 
wrote:

> The new typeface and a few other minor visual tweaks are up!
>
> Kindly,
> Dmitry
>
> > On Oct 12, 2015, at 3:16 PM, Raymond Camden 
> wrote:
> >
> > +1 here too.
> >
> > On Fri, Oct 9, 2015 at 1:46 PM, Jesse  wrote:
> >> This sounds more like consensus!
> >> I too am +1 on changing the typeface.
> >>
> >> Cheers,
> >>  Jesse
> >>
> >>> On Oct 8, 2015, at 11:02 AM, Parashuram N 
> wrote:
> >>>
> >>> +1 on the typeface - I think it does send the “modern-ness” message.
> >>>
> >>>
> >>>
> >>>
>  On 10/8/15, 10:39 AM, "Carlos Santana"  wrote:
> 
>  + 1 on the typeface
>  leave the logo as it is today (3d, gray, etc..)
> 
> > On Thu, Oct 8, 2015 at 12:26 AM Steven Gill 
> wrote:
> >
> > +1
> >> On Oct 7, 2015 9:15 PM, "Brian LeRoux"  wrote:
> >>
> >> Well said, and ya updated typeface looks slick. 
> >>> On Wed, Oct 7, 2015 at 6:10 PM Ryan J. Salva  >
> >> wrote:
> >>
> >>> I'm personally a big fan of Ben's proposal. The typography really
> > stands
> >>> out and gives the brand a strong, updated and modern presence. I
> know
> >>> there's a lot of pride in the heritage of the logo, and for that
> > reason I
> >>> wouldn't dream of losing our beloved robot. However, the
> typography is
> > a
> >>> different story.
> >>>
> >>> Is it absolutely necessary? Well... that really depends on your
> >>> perspective. From talking with many of you and other developers at
> >>> conferences and events over the past year, I've heard a repeating
> theme
> >>> that Cordova has a public-perception problem. A typography update
> won't
> >>> change that perception by itself, but it will demonstrate that
> Cordova
> >>> isn't staying still... that the project is evolving, growing and
> >> improving.
> >>>
> >>> 
> >>> rjs
> >>>
> >>> Ryan J. Salva  |  Principal Program Manager Lead
> >>> Visual Studio Tools for Apache Cordova
> >>> rsa...@microsoft.com
> >>> 206 612 5079 mobile
> >>>
> >>>
> >>>
> >>> -Original Message-
> >>> From: Tommy-Carlos Williams [mailto:to...@devgeeks.org]
> >>> Sent: Wednesday, October 7, 2015 12:21 PM
> >>> To: dev@cordova.apache.org
> >>> Subject: Re: Cordova website redesign
> >>>
> >>> Yeah... I kinda fell into the "not sure, wait and see what others
> >> thought"
> >>> category on this one.
> >>>
> >>> I would also like more discussion unless everyone really has no
> > opinion.
> >>>
>  On 8 Oct 2015, at 05:42, Jesse  wrote:
> 
>  I would suggest that the lack of response on the logo change
> means we
> >>> don't need it.
>  I think there should at least be some discussion before moving on
> it.
> 
> > On Oct 7, 2015, at 11:31 AM, Brad Gashler 
> >> wrote:
> >
> > Hi Everyone,
> >
> > I wanted to follow up on the branding changes proposed by Ben
> Sperry
> >>> (his email is included below).
> >
> > I’m planning on implementing his changes, since no objections
> were
> >>> raised.  This includes the new logo, with its modern typeface, as
> shown
> >> in
> >>> picture A below.  Also, we may replace the background image with
> the
> >> simple
> >>> gradient as shown in picture B below.  Please let me know if there
> are
> >> any
> >>> concerns replacing this background image.
> >
> > Thanks,
> >
> > Brad Gashler
> >
> >
> > A) New logo only
> >
> >
> >
> > B) New logo and new gradient background
> >
> >
> >
> >
> > -Original Message-
> > From: Ben Sperry [mailto:b...@ionic.io]
> > Sent: Thursday, September 24, 2015 1:17 PM
> > To: dev@cordova.apache.org
> > Subject: Re: Cordova website redesign
> >
> > I really like where this update is going, and what it signals to
> the
> > community: Cordova is a legitimate, modern tech solution that's
> > still
> >>> very much alive and growing. The website look/feel has a lot of
> >> confidence,
> >>> love it!
> >
> > I'm curious what thoughts are around a Cordova logo update
> itself. I
> > put together a version that I thought might support the
> evolution of
> > Cordova into a more modern looking brand. Would love feedback on
> > this
> > early logo update 

[GitHub] cordova-ios pull request: CB-9719 set allow_non_modular_includes t...

[asfgit]

Github user asfgit closed the pull request at:

https://github.com/apache/cordova-ios/pull/171


---
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 is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

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

[GitHub] cordova-osx pull request: CB-9581 OSX WebView does not send cookie...

[asfgit]

Github user asfgit closed the pull request at:

https://github.com/apache/cordova-osx/pull/23


---
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 is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

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

Re: Cordova website redesign

[Dmitry Blotsky]

The new typeface and a few other minor visual tweaks are up!

Kindly,
Dmitry

> On Oct 12, 2015, at 3:16 PM, Raymond Camden  wrote:
> 
> +1 here too.
> 
> On Fri, Oct 9, 2015 at 1:46 PM, Jesse  wrote:
>> This sounds more like consensus!
>> I too am +1 on changing the typeface.
>> 
>> Cheers,
>>  Jesse
>> 
>>> On Oct 8, 2015, at 11:02 AM, Parashuram N  wrote:
>>> 
>>> +1 on the typeface - I think it does send the “modern-ness” message.
>>> 
>>> 
>>> 
>>> 
 On 10/8/15, 10:39 AM, "Carlos Santana"  wrote:
 
 + 1 on the typeface
 leave the logo as it is today (3d, gray, etc..)
 
> On Thu, Oct 8, 2015 at 12:26 AM Steven Gill  
> wrote:
> 
> +1
>> On Oct 7, 2015 9:15 PM, "Brian LeRoux"  wrote:
>> 
>> Well said, and ya updated typeface looks slick. 
>>> On Wed, Oct 7, 2015 at 6:10 PM Ryan J. Salva 
>> wrote:
>> 
>>> I'm personally a big fan of Ben's proposal. The typography really
> stands
>>> out and gives the brand a strong, updated and modern presence. I know
>>> there's a lot of pride in the heritage of the logo, and for that
> reason I
>>> wouldn't dream of losing our beloved robot. However, the typography is
> a
>>> different story.
>>> 
>>> Is it absolutely necessary? Well... that really depends on your
>>> perspective. From talking with many of you and other developers at
>>> conferences and events over the past year, I've heard a repeating theme
>>> that Cordova has a public-perception problem. A typography update won't
>>> change that perception by itself, but it will demonstrate that Cordova
>>> isn't staying still... that the project is evolving, growing and
>> improving.
>>> 
>>> 
>>> rjs
>>> 
>>> Ryan J. Salva  |  Principal Program Manager Lead
>>> Visual Studio Tools for Apache Cordova
>>> rsa...@microsoft.com
>>> 206 612 5079 mobile
>>> 
>>> 
>>> 
>>> -Original Message-
>>> From: Tommy-Carlos Williams [mailto:to...@devgeeks.org]
>>> Sent: Wednesday, October 7, 2015 12:21 PM
>>> To: dev@cordova.apache.org
>>> Subject: Re: Cordova website redesign
>>> 
>>> Yeah... I kinda fell into the "not sure, wait and see what others
>> thought"
>>> category on this one.
>>> 
>>> I would also like more discussion unless everyone really has no
> opinion.
>>> 
 On 8 Oct 2015, at 05:42, Jesse  wrote:
 
 I would suggest that the lack of response on the logo change means we
>>> don't need it.
 I think there should at least be some discussion before moving on it.
 
> On Oct 7, 2015, at 11:31 AM, Brad Gashler 
>> wrote:
> 
> Hi Everyone,
> 
> I wanted to follow up on the branding changes proposed by Ben Sperry
>>> (his email is included below).
> 
> I’m planning on implementing his changes, since no objections were
>>> raised.  This includes the new logo, with its modern typeface, as shown
>> in
>>> picture A below.  Also, we may replace the background image with the
>> simple
>>> gradient as shown in picture B below.  Please let me know if there are
>> any
>>> concerns replacing this background image.
> 
> Thanks,
> 
> Brad Gashler
> 
> 
> A) New logo only
> 
> 
> 
> B) New logo and new gradient background
> 
> 
> 
> 
> -Original Message-
> From: Ben Sperry [mailto:b...@ionic.io]
> Sent: Thursday, September 24, 2015 1:17 PM
> To: dev@cordova.apache.org
> Subject: Re: Cordova website redesign
> 
> I really like where this update is going, and what it signals to the
> community: Cordova is a legitimate, modern tech solution that's
> still
>>> very much alive and growing. The website look/feel has a lot of
>> confidence,
>>> love it!
> 
> I'm curious what thoughts are around a Cordova logo update itself. I
> put together a version that I thought might support the evolution of
> Cordova into a more modern looking brand. Would love feedback on
> this
> early logo update mockup: regular
> <
> https://na01.safelinks.protection.outlook.com/?url=http%3a%2f%2fioni
> https://na01.safelinks.protection.outlook.com/?url=c-io-assets.s3.amazonaws.com=01%7c01%7cpanarasi%40microsoft.com%7c60f8e63b1c9a46d5e95208d2d00785b0%7c72f988bf86f141af91ab2d7cd011db47%7c1=r7JctK6wY1QhZ2GZNlumefajI4vAGiseq9LrehDxEPQ%3d
> %2fcordova%2fcordova-logo-3d-proposal.png
> =01%7c01%7cbradga%40microsoft.com
> 

[GitHub] cordova-osx pull request: CB-9692 OSX: cordova.exec is not availab...

[asfgit]

Github user asfgit closed the pull request at:

https://github.com/apache/cordova-osx/pull/24


---
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 is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

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

[GitHub] cordova-ios pull request: CB-9500 Added no sign argument to iOS bu...

[shazron]

Github user shazron commented on the pull request:

https://github.com/apache/cordova-ios/pull/152#issuecomment-147590435
  
@nikhilkh any objections to this getting pulled in?


---
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 is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

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

[GitHub] cordova-ios pull request: CB-9500 Added no sign argument to iOS bu...

[shazron]

Github user shazron commented on the pull request:

https://github.com/apache/cordova-ios/pull/152#issuecomment-147599790
  
Hi @njtman I couldn't find your name (from the patch's From: field) in the 
Apache CLA list: http://people.apache.org/committer-index.html

Once you have filed one, I can proceed with merging this in. Instructions 
here: https://www.apache.org/licenses/icla.txt


---
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 is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

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

Re: Cordova website redesign

[Ryan J. Salva]

Yup. We're on it. Should have an update this week.

Sent from Outlook




On Mon, Oct 12, 2015 at 8:53 PM -0700, "Jesse" 
> wrote:

These need to be updated as well :
https://na01.safelinks.protection.outlook.com/?url=http%3a%2f%2fcordova.apache.org%2fartwork%2f=01%7c01%7crsalva%40microsoft.com%7c306f2f80f7fb4aae07bf08d2d381e0d1%7c72f988bf86f141af91ab2d7cd011db47%7c1=w3J0dvoHePnSkK47DxEjN2F4yu47I%2fz8cZzLXnA0C2Y%3d



My team is hiring!
@purplecabbage
https://na01.safelinks.protection.outlook.com/?url=risingj.com=01%7c01%7crsalva%40microsoft.com%7c306f2f80f7fb4aae07bf08d2d381e0d1%7c72f988bf86f141af91ab2d7cd011db47%7c1=rDTjIDETsdSaiiZnIvfUMkI6FxgXO68iVs52vsg5uec%3d

On Mon, Oct 12, 2015 at 7:20 PM, Dmitry Blotsky 
wrote:

> The new typeface and a few other minor visual tweaks are up!
>
> Kindly,
> Dmitry
>
> > On Oct 12, 2015, at 3:16 PM, Raymond Camden 
> wrote:
> >
> > +1 here too.
> >
> > On Fri, Oct 9, 2015 at 1:46 PM, Jesse  wrote:
> >> This sounds more like consensus!
> >> I too am +1 on changing the typeface.
> >>
> >> Cheers,
> >>  Jesse
> >>
> >>> On Oct 8, 2015, at 11:02 AM, Parashuram N 
> wrote:
> >>>
> >>> +1 on the typeface - I think it does send the “modern-ness” message.
> >>>
> >>>
> >>>
> >>>
>  On 10/8/15, 10:39 AM, "Carlos Santana"  wrote:
> 
>  + 1 on the typeface
>  leave the logo as it is today (3d, gray, etc..)
> 
> > On Thu, Oct 8, 2015 at 12:26 AM Steven Gill 
> wrote:
> >
> > +1
> >> On Oct 7, 2015 9:15 PM, "Brian LeRoux"  wrote:
> >>
> >> Well said, and ya updated typeface looks slick. 
> >>> On Wed, Oct 7, 2015 at 6:10 PM Ryan J. Salva  >
> >> wrote:
> >>
> >>> I'm personally a big fan of Ben's proposal. The typography really
> > stands
> >>> out and gives the brand a strong, updated and modern presence. I
> know
> >>> there's a lot of pride in the heritage of the logo, and for that
> > reason I
> >>> wouldn't dream of losing our beloved robot. However, the
> typography is
> > a
> >>> different story.
> >>>
> >>> Is it absolutely necessary? Well... that really depends on your
> >>> perspective. From talking with many of you and other developers at
> >>> conferences and events over the past year, I've heard a repeating
> theme
> >>> that Cordova has a public-perception problem. A typography update
> won't
> >>> change that perception by itself, but it will demonstrate that
> Cordova
> >>> isn't staying still... that the project is evolving, growing and
> >> improving.
> >>>
> >>> 
> >>> rjs
> >>>
> >>> Ryan J. Salva  |  Principal Program Manager Lead
> >>> Visual Studio Tools for Apache Cordova
> >>> rsa...@microsoft.com
> >>> 206 612 5079 mobile
> >>>
> >>>
> >>>
> >>> -Original Message-
> >>> From: Tommy-Carlos Williams [mailto:to...@devgeeks.org]
> >>> Sent: Wednesday, October 7, 2015 12:21 PM
> >>> To: dev@cordova.apache.org
> >>> Subject: Re: Cordova website redesign
> >>>
> >>> Yeah... I kinda fell into the "not sure, wait and see what others
> >> thought"
> >>> category on this one.
> >>>
> >>> I would also like more discussion unless everyone really has no
> > opinion.
> >>>
>  On 8 Oct 2015, at 05:42, Jesse  wrote:
> 
>  I would suggest that the lack of response on the logo change
> means we
> >>> don't need it.
>  I think there should at least be some discussion before moving on
> it.
> 
> > On Oct 7, 2015, at 11:31 AM, Brad Gashler 
> >> wrote:
> >
> > Hi Everyone,
> >
> > I wanted to follow up on the branding changes proposed by Ben
> Sperry
> >>> (his email is included below).
> >
> > I’m planning on implementing his changes, since no objections
> were
> >>> raised.  This includes the new logo, with its modern typeface, as
> shown
> >> in
> >>> picture A below.  Also, we may replace the background image with
> the
> >> simple
> >>> gradient as shown in picture B below.  Please let me know if there
> are
> >> any
> >>> concerns replacing this background image.
> >
> > Thanks,
> >
> > Brad Gashler
> >
> >
> > A) New logo only
> >
> >
> >
> > B) New logo and new gradient background
> >
> >
> >
> >
> > -Original Message-
> > From: Ben Sperry [mailto:b...@ionic.io]
> > Sent: Thursday, September 24, 2015 1:17 PM
> > To: dev@cordova.apache.org
> > 

[GitHub] cordova-ios pull request: CB-9500 Added no sign argument to iOS bu...

[nikhilkh]

Github user nikhilkh commented on the pull request:

https://github.com/apache/cordova-ios/pull/152#issuecomment-147592877
  
This looks good. Let's get it in!


---
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 is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

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