[GitHub] mobidev111 commented on issue #264: [WIP] Win10 only

2018-06-26 Thread GitBox
mobidev111 commented on issue #264: [WIP] Win10 only URL: https://github.com/apache/cordova-windows/pull/264#issuecomment-400563759 Windows 8 / Windows 8.1 is gone - about 7% marketshare [source](https://netmarketshare.com/operating-system-market-share.aspx?options=%7B%22filter%22%3A%7B%22%

[cordova-common] branch master updated: CB-14099 osx: Fixed Resolve Config Path for OSX

2018-06-26 Thread tripod
This is an automated email from the ASF dual-hosted git repository. tripod pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/cordova-common.git The following commit(s) were added to refs/heads/master by this push: new 7bf9627 CB-14099 osx: Fixed Resolve Con

[GitHub] brodybits commented on issue #264: [WIP] Win10 only

2018-06-26 Thread GitBox
brodybits commented on issue #264: [WIP] Win10 only URL: https://github.com/apache/cordova-windows/pull/264#issuecomment-400536290 I feel a bit ambivalent about this change right now. I continue to support legacy sqlite plugin version for pre-win10, not sure that the need has completely go

svn commit: r27750 - in /dev/cordova/CB-14174: ./ cordova-common-2.2.5.tgz cordova-common-2.2.5.tgz.asc cordova-common-2.2.5.tgz.md5 cordova-common-2.2.5.tgz.sha512

2018-06-26 Thread brodybits
Author: brodybits Date: Wed Jun 27 01:52:59 2018 New Revision: 27750 Log: CB-14174 Uploading release candidates for cordova-common (tools) release Added: dev/cordova/CB-14174/ dev/cordova/CB-14174/cordova-common-2.2.5.tgz (with props) dev/cordova/CB-14174/cordova-common-2.2.5.tgz.as

svn commit: r27749 - /dev/cordova/CB-14152/

2018-06-26 Thread brodybits
Author: brodybits Date: Wed Jun 27 01:51:33 2018 New Revision: 27749 Log: CB-14152 Removing release candidates from dist/dev Removed: dev/cordova/CB-14152/ - To unsubscribe, e-mail: commits-unsubscr...@cordova.apache.org Fo

[GitHub] raphinesse commented on issue #838: Docs should not encourage global npm installation

2018-06-26 Thread GitBox
raphinesse commented on issue #838: Docs should not encourage global npm installation URL: https://github.com/apache/cordova-docs/issues/838#issuecomment-400505667 > There probably was a reason why `sudo` is mentioned currently? @janpio I suppose because for most default setups you n

[GitHub] tripodsan commented on issue #49: Package update WIP FOR DISCUSSION [MAJOR RELEASE ITEM]

2018-06-26 Thread GitBox
tripodsan commented on issue #49: Package update WIP FOR DISCUSSION [MAJOR RELEASE ITEM] URL: https://github.com/apache/cordova-osx/pull/49#issuecomment-400503667 +1 This is an automated message from the Apache Git Service. T

[GitHub] dpogue commented on issue #838: Docs should not encourage global npm installation

2018-06-26 Thread GitBox
dpogue commented on issue #838: Docs should not encourage global npm installation URL: https://github.com/apache/cordova-docs/issues/838#issuecomment-400408050 >> And then just keep using plain cordova throughout the docs. > > Will this work on all platforms without the global cordova

[GitHub] janpio commented on issue #838: Docs should not encourage global npm installation

2018-06-26 Thread GitBox
janpio commented on issue #838: Docs should not encourage global npm installation URL: https://github.com/apache/cordova-docs/issues/838#issuecomment-400406272 There probably was a reason why `sudo` is mentioned currently? > And then just keep using plain cordova throughout the docs.

[GitHub] brodybits edited a comment on issue #838: Docs should not encourage global npm installation

2018-06-26 Thread GitBox
brodybits edited a comment on issue #838: Docs should not encourage global npm installation URL: https://github.com/apache/cordova-docs/issues/838#issuecomment-400385862 > Currently the docs say to run `sudo npm install -g` cordova which is not great +1 (+100 if you change "not grea

[GitHub] brodybits edited a comment on issue #838: Docs should not encourage global npm installation

2018-06-26 Thread GitBox
brodybits edited a comment on issue #838: Docs should not encourage global npm installation URL: https://github.com/apache/cordova-docs/issues/838#issuecomment-400385862 > Currently the docs say to run `sudo npm install -g` cordova which is not great +1 (+100 if you change "not grea

[GitHub] brodybits edited a comment on issue #838: Docs should not encourage global npm installation

2018-06-26 Thread GitBox
brodybits edited a comment on issue #838: Docs should not encourage global npm installation URL: https://github.com/apache/cordova-docs/issues/838#issuecomment-400385862 > Currently the docs say to run `sudo npm install -g` cordova which is not great +1 (+100 if you change "not grea

[GitHub] raphinesse commented on issue #838: Docs should not encourage global npm installation

2018-06-26 Thread GitBox
raphinesse commented on issue #838: Docs should not encourage global npm installation URL: https://github.com/apache/cordova-docs/issues/838#issuecomment-400388905 I'd suggest presenting both alternatives: - global installation (without using sudo) - Add to devDeps and use with np

[GitHub] dpogue commented on issue #838: Docs should not encourage global npm installation

2018-06-26 Thread GitBox
dpogue commented on issue #838: Docs should not encourage global npm installation URL: https://github.com/apache/cordova-docs/issues/838#issuecomment-400387262 It works perfectly for create: `npx cordova create MyProject` You can even use a specific version: `npx cordova@5.4.0 create MyP

[GitHub] brodybits commented on issue #838: Docs should not encourage global npm installation

2018-06-26 Thread GitBox
brodybits commented on issue #838: Docs should not encourage global npm installation URL: https://github.com/apache/cordova-docs/issues/838#issuecomment-400385862 > Currently the docs say to run `sudo npm install -g` cordova which is not great +1 (+100 if you change "not great" to "

[GitHub] dpogue opened a new issue #838: Docs should not encourage global npm installation

2018-06-26 Thread GitBox
dpogue opened a new issue #838: Docs should not encourage global npm installation URL: https://github.com/apache/cordova-docs/issues/838 Currently the docs say to run `sudo npm install -g cordova` which is not great from a best practices standpoint. We should never encourage running npm w

svn commit: r1834435 [1/2] - in /cordova/site/public: ./ blog/ docs/en/6.x/reference/cordova-cli/ docs/en/6.x/reference/cordova-plugin-battery-status/ docs/en/6.x/reference/cordova-plugin-camera/ docs

2018-06-26 Thread steven
Author: steven Date: Tue Jun 26 14:56:32 2018 New Revision: 1834435 URL: http://svn.apache.org/viewvc?rev=1834435&view=rev Log: Updated docs Added: cordova/site/public/news/2018/06/26/ cordova/site/public/news/2018/06/26/cordova-js-release-4.2.4.html Modified: cordova/site/public/blog

svn commit: r1834435 [2/2] - in /cordova/site/public: ./ blog/ docs/en/6.x/reference/cordova-cli/ docs/en/6.x/reference/cordova-plugin-battery-status/ docs/en/6.x/reference/cordova-plugin-camera/ docs

2018-06-26 Thread steven
Added: cordova/site/public/news/2018/06/26/cordova-js-release-4.2.4.html URL: http://svn.apache.org/viewvc/cordova/site/public/news/2018/06/26/cordova-js-release-4.2.4.html?rev=1834435&view=auto == --- cordova/site/public/

[cordova-docs] branch master updated: CB-14152 add cordova-js@4.2.4 blog post

2018-06-26 Thread brodybits
This is an automated email from the ASF dual-hosted git repository. brodybits pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/cordova-docs.git The following commit(s) were added to refs/heads/master by this push: new 9dd53f0 CB-14152 add cordova-js@4.2.4

[GitHub] brodybits closed pull request #834: CB-14152 add cordova-js@4.2.4 blog post

2018-06-26 Thread GitBox
brodybits closed pull request #834: CB-14152 add cordova-js@4.2.4 blog post URL: https://github.com/apache/cordova-docs/pull/834 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance: As this is a foreign

[GitHub] brodybits commented on issue #834: CB-14152 add cordova-js@4.2.4 blog post

2018-06-26 Thread GitBox
brodybits commented on issue #834: CB-14152 add cordova-js@4.2.4 blog post URL: https://github.com/apache/cordova-docs/pull/834#issuecomment-400324136 Thanks @dpogue for the quick response! This is an automated message from th

[GitHub] liuxh0 commented on issue #277: VS Professional 2017 v15.7: MSBuild v4.0 is not supported, aborting.

2018-06-26 Thread GitBox
liuxh0 commented on issue #277: VS Professional 2017 v15.7: MSBuild v4.0 is not supported, aborting. URL: https://github.com/apache/cordova-windows/issues/277#issuecomment-400322612 Of course. I just set the env variable `MSBUILDDIR` to the path where VS is installed, in my case `C:

[GitHub] brodybits commented on a change in pull request #834: CB-14152 add cordova-js@4.2.4 blog post

2018-06-26 Thread GitBox
brodybits commented on a change in pull request #834: CB-14152 add cordova-js@4.2.4 blog post URL: https://github.com/apache/cordova-docs/pull/834#discussion_r198153467 ## File path: www/_posts/2018-06-22-cordova-js-release-4.2.4.md ## @@ -0,0 +1,24 @@ +--- +layout: post +

svn commit: r27739 - in /release/cordova/tools: cordova-js-4.2.3.tgz cordova-js-4.2.3.tgz.asc cordova-js-4.2.3.tgz.md5 cordova-js-4.2.3.tgz.sha512 cordova-js-4.2.4.tgz cordova-js-4.2.4.tgz.asc cordova

2018-06-26 Thread brodybits
Author: brodybits Date: Tue Jun 26 13:50:05 2018 New Revision: 27739 Log: CB-14152 publish cordova-js 4.2.4 (tools release) to dist Added: release/cordova/tools/cordova-js-4.2.4.tgz (with props) release/cordova/tools/cordova-js-4.2.4.tgz.asc release/cordova/tools/cordova-js-4.2.4.tg

[GitHub] janpio commented on issue #277: VS Professional 2017 v15.7: MSBuild v4.0 is not supported, aborting.

2018-06-26 Thread GitBox
janpio commented on issue #277: VS Professional 2017 v15.7: MSBuild v4.0 is not supported, aborting. URL: https://github.com/apache/cordova-windows/issues/277#issuecomment-400309409 Awesome, thanks for the update! Do you mind sharing what exactly solved the problem? Future readers m

[GitHub] liuxh0 closed issue #277: VS Professional 2017 v15.7: MSBuild v4.0 is not supported, aborting.

2018-06-26 Thread GitBox
liuxh0 closed issue #277: VS Professional 2017 v15.7: MSBuild v4.0 is not supported, aborting. URL: https://github.com/apache/cordova-windows/issues/277 This is an automated message from the Apache Git Service. To respond to

[GitHub] liuxh0 commented on issue #277: VS Professional 2017 v15.7: MSBuild v4.0 is not supported, aborting.

2018-06-26 Thread GitBox
liuxh0 commented on issue #277: VS Professional 2017 v15.7: MSBuild v4.0 is not supported, aborting. URL: https://github.com/apache/cordova-windows/issues/277#issuecomment-400306134 @janpio , you solved my problem! Thanks! T

[GitHub] janpio commented on issue #801: quicker dev build

2018-06-26 Thread GitBox
janpio commented on issue #801: quicker dev build URL: https://github.com/apache/cordova-docs/pull/801#issuecomment-400267803 See the second comment. Also I am not sure if the release of the 8.x docs has changed anything that would have to be adapted here. Feel free to drive this forward (

[cordova-plugin-geolocation] branch master updated: Add Apache Cordova issue tracker link to Readme (#107)

2018-06-26 Thread shazron
This is an automated email from the ASF dual-hosted git repository. shazron pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/cordova-plugin-geolocation.git The following commit(s) were added to refs/heads/master by this push: new 91c7313 Add Apache Cordova

[GitHub] shazron closed pull request #107: Add Apache Cordova issue tracker link to Readme

2018-06-26 Thread GitBox
shazron closed pull request #107: Add Apache Cordova issue tracker link to Readme URL: https://github.com/apache/cordova-plugin-geolocation/pull/107 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance:

[GitHub] jacquesdev opened a new pull request #107: Add Apache Cordova issue tracker link to Readme

2018-06-26 Thread GitBox
jacquesdev opened a new pull request #107: Add Apache Cordova issue tracker link to Readme URL: https://github.com/apache/cordova-plugin-geolocation/pull/107 ### What does this PR do? Updated the Readme file to include a link to the Apache Cordova issue tracker -