[GitHub] cordova-android pull request: CB-9430 Fixes check_reqs failure whe...

2015-07-31 Thread vladimir-kotikov
GitHub user vladimir-kotikov opened a pull request: https://github.com/apache/cordova-android/pull/204 CB-9430 Fixes check_reqs failure when javac returns an extra line You can merge this pull request into a Git repository by running: $ git pull

[GitHub] cordova-lib pull request: CB-9436 Removes `require-tr` bundle tran...

2015-07-31 Thread vladimir-kotikov
GitHub user vladimir-kotikov opened a pull request: https://github.com/apache/cordova-lib/pull/275 CB-9436 Removes `require-tr` bundle transformation This PR replaces excess `require-tr` browserify transform (- 2 deps and lot of code) with `aliasify`. It also solves two

[GitHub] cordova-android pull request: CB-9430 Fixes check_reqs failure whe...

2015-07-31 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/cordova-android/pull/204 --- 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-js pull request: CB-9436 Refactor browserify to get rid of...

2015-07-31 Thread vladimir-kotikov
GitHub user vladimir-kotikov opened a pull request: https://github.com/apache/cordova-js/pull/124 CB-9436 Refactor browserify to get rid of `require-tr` transform This PR removes excess `require-tr` browserify transform and makes resultant bundle compatible with mobilespec and

[GitHub] cordova-plugin-file pull request: Make Android default persistent ...

2015-07-31 Thread macdonst
Github user macdonst commented on the pull request: https://github.com/apache/cordova-plugin-file/pull/127#issuecomment-126738049 @jcesarmobile Good point, not sure how to handle it short of telling them not to copy and stick with compatibility for their app. Also, I've

[GitHub] cordova-plugin-camera pull request: CB-9193: Add 'showLibraryButto...

2015-07-31 Thread dpolivy
Github user dpolivy commented on the pull request: https://github.com/apache/cordova-plugin-camera/pull/104#issuecomment-126746379 @infil00p @shazron @nikhilkh @stevengill Any thoughts on this discussion or possibility of a CR? --- If your project is set up for it, you can reply to

[GitHub] cordova-lib pull request: CB-9278: BugFix: Restoring multiple plat...

2015-07-31 Thread omefire
Github user omefire commented on the pull request: https://github.com/apache/cordova-lib/pull/266#issuecomment-126738584 @TimBarham , I've explored that possibility before landing with the current solution. I think messing around with a queue-based solution or the event loop is

[GitHub] cordova-windows pull request: CB-9359: Adds support for .appxbundl...

2015-07-31 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/cordova-windows/pull/102 --- 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-lib pull request: CB-9278: BugFix: Restoring multiple plat...

2015-07-31 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/cordova-lib/pull/266 --- 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: cordova platfor update DOA for cordova-android@4.1.0

2015-07-31 Thread julio cesar sanchez
+1 to Joe, this update is 4.1.0 (major version bump) because the support for API Level 10 was removed BTW, Carlos, I've seen that you changed the issue priority from Minor to Critical, I set it to Minor because there is an easy workaround, just tell the user to add the android-minSdkVersion

[GitHub] cordova-plugin-inappbrowser pull request: CB-9378 Fix InAppBrowser...

2015-07-31 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/cordova-plugin-inappbrowser/pull/109 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or

[GitHub] cordova-plugin-inappbrowser pull request: CB-9378 Fix InAppBrowser...

2015-07-31 Thread dblotsky
Github user dblotsky commented on the pull request: https://github.com/apache/cordova-plugin-inappbrowser/pull/109#issuecomment-126664717 LGTM --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not

[GitHub] cordova-lib pull request: CB-9278: BugFix: Restoring multiple plat...

2015-07-31 Thread TimBarham
Github user TimBarham commented on the pull request: https://github.com/apache/cordova-lib/pull/266#issuecomment-126670748 Made this comment offline, but decided it should be included here too... I don't think this is the right fix - I think a more general approach would serve use

[CI] Medic Update

2015-07-31 Thread Dmitry Blotsky
Hey folks, - To unsubscribe, e-mail: dev-unsubscr...@cordova.apache.org For additional commands, e-mail: dev-h...@cordova.apache.org

Re: [CI] Medic Update

2015-07-31 Thread Dmitry Blotsky
Whoop, pressed “Send” instead of “Save”. Please hold on for the real update. On Jul 31, 2015, at 7:53 PM, Dmitry Blotsky dblot...@microsoft.com wrote: Hey folks, - To unsubscribe, e-mail:

Re: [DISCUSS] CI Hardware

2015-07-31 Thread Dmitry Blotsky
Ping, folks. What’s the progress? @Jesse @Carlos @Joe? Kindly, Dmitry On Jul 17, 2015, at 4:08 PM, Dmitry Blotsky dblot...@microsoft.com wrote: Hi all, As per the hangout, please respond to this thread with machines and devices you can hook up to the CI. Also please list relevant info

[CI] Medic Update

2015-07-31 Thread Dmitry Blotsky
Hey folks, TL;DR: Please contribute to the CI device fleet. Also is nobody testing on Windows? These past two week’s updates regarding CI: - Windows runs are *all* crashing; who is looking into this? - per-commit builds are still waiting on Apache Infra (INFRA-9979) - per-PR builds are in the

Buildbot: buildslave cordova-windows-slave was lost

2015-07-31 Thread buildbot
The Buildbot working for 'ASF Buildbot' has noticed that the buildslave named cordova-windows-slave went away It last disconnected at Sat Aug 1 03:08:54 2015 (buildmaster-local time) The admin on record (as reported by BUILDSLAVE:info/admin) was 'Dmitry Blotsky dblot...@microsoft.com '.

Re: Buildbot: buildslave cordova-windows-slave was lost

2015-07-31 Thread Dmitry Blotsky
Adding insult to injury. ): I’ll bring the machine back on Monday. Kindly, Dmitry On Jul 31, 2015, at 8:13 PM, build...@apache.org wrote: The Buildbot working for 'ASF Buildbot' has noticed that the buildslave named cordova-windows-slave went away It last disconnected at Sat Aug 1

[Vote] 4.1.1 Android Release

2015-07-31 Thread Steven Gill
Please review and vote on this 4.1.1 Android Release by replying to this email (and keep discussion on the DISCUSS thread) Release issue: https://issues.apache.org/jira/browse/CB-9437 The archive has been published to dist/dev:https://dist.apache.org/repos/dist/dev/cordova/CB-9437 The package

[GitHub] cordova-android pull request: CB-9172 Improved emulator deploy sta...

2015-07-31 Thread dblotsky
Github user dblotsky commented on the pull request: https://github.com/apache/cordova-android/pull/188#issuecomment-126652794 LGTM. This has been up on the staging CI for a while, and I haven't been seeing issues. Thus, letting it lose on the real CI. --- If your project is set up

[GitHub] cordova-android pull request: CB-9172 Improved emulator deploy sta...

2015-07-31 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/cordova-android/pull/188 --- 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-plugin-media pull request: CB-9238 Media plugin cannot rec...

2015-07-31 Thread dblotsky
Github user dblotsky commented on the pull request: https://github.com/apache/cordova-plugin-media/pull/58#issuecomment-126634941 Ping. Please rebase atop latest master changes, and verify that it still works. Then we can merge. --- If your project is set up for it, you can reply

[GitHub] cordova-windows pull request: CB-9359: Adds support for .appxbundl...

2015-07-31 Thread dblotsky
Github user dblotsky commented on the pull request: https://github.com/apache/cordova-windows/pull/102#issuecomment-126636672 Ping on this. Please rebase atop master and verify that tests pass. --- If your project is set up for it, you can reply to this email and have your reply

[GitHub] cordova-plugin-file pull request: Make Android default persistent ...

2015-07-31 Thread jcesarmobile
Github user jcesarmobile commented on the pull request: https://github.com/apache/cordova-plugin-file/pull/127#issuecomment-126644090 I agree with @macdonst that setting the AndroidPersistentFileLocation preference to Compatibility should be enough if the developer wants the old

[GitHub] cordova-lib pull request: CB-9278: BugFix: Restoring multiple plat...

2015-07-31 Thread dblotsky
Github user dblotsky commented on the pull request: https://github.com/apache/cordova-lib/pull/266#issuecomment-126643184 Also tried on OS X with two platforms, and it worked. +1 for adding a test case for this. --- If your project is set up for it, you can reply to this email and

[GitHub] cordova-plugin-media pull request: CB-9237 Add cdvfile:// support ...

2015-07-31 Thread dblotsky
Github user dblotsky commented on the pull request: https://github.com/apache/cordova-plugin-media/pull/57#issuecomment-126635115 Ping. Please rebase atop latest master changes, and verify that it still works. Then we can merge. --- If your project is set up for it, you can reply