[GitHub] cordova-lib pull request: CB-9712 CLI 5.3 breaks with node 3.3.3
GitHub user daserge opened a pull request: https://github.com/apache/cordova-lib/pull/316 CB-9712 CLI 5.3 breaks with node 3.3.3 I could repro the issue using npm@3.3.6 as npm@3.3.5 seems to have issues with Cordova installation. Requires [cordova-app-hello-world#14](https://github.com/apache/cordova-app-hello-world/pull/14) changes in order for require to work. [Jira issue](https://issues.apache.org/jira/browse/CB-9712) You can merge this pull request into a Git repository by running: $ git pull https://github.com/MSOpenTech/cordova-lib CB-9712 Alternatively you can review and apply these changes as the patch at: https://github.com/apache/cordova-lib/pull/316.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 #316 commit 1661c6aa02091bd02c15f3520f904b91250e2df8 Author: daserge Date: 2015-10-06T10:40:45Z CB-9712 CLI 5.3 breaks with node 3.3.3 --- 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
Github user daserge commented on the pull request: https://github.com/apache/cordova-lib/pull/316#issuecomment-145964797 Tests are passing with the updated cordova-app-hello-world: #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-9712 CLI 5.3 breaks with node 3.3.3
Github user nikhilkh commented on the pull request: https://github.com/apache/cordova-lib/pull/316#issuecomment-146035804 LGTM. @stevengill for FYI. Next tools release will have to also release cordova-app-hello-world to make this 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-lib pull request: CB-9712 CLI 5.3 breaks with node 3.3.3
Github user stevengill commented on the pull request: https://github.com/apache/cordova-lib/pull/316#issuecomment-146042049 LGTM. We should do a new release soon! --- 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
Github user nikhilkh commented on the pull request: https://github.com/apache/cordova-lib/pull/316#issuecomment-146042904 @daserge Does the dev workflow break with this change? Does `coho npm-link` need to be modified to include linking cordova-app-hello-world for cordova committers who are linking to cloned version of the repo for this to 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-lib pull request: CB-9712 CLI 5.3 breaks with node 3.3.3
Github user daserge commented on the pull request: https://github.com/apache/cordova-lib/pull/316#issuecomment-146624899 @nikhilkh yes I think coho will need an update. Although I still have issues with local dev deployment specifically with [npm linking](https://github.com/npm/npm/issues/9766) so can't verify this for now. --- 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
Github user nikhilkh commented on the pull request: https://github.com/apache/cordova-lib/pull/316#issuecomment-146629554 Let's check this in anyway. --- 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
Github user omefire commented on the pull request: https://github.com/apache/cordova-lib/pull/316#issuecomment-146642291 I don't understand. Why would we need a new release of 'cordova-app-hello-world' for this to work ? This code doesn't seem to require anything extra as it currently stands. --- 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
Github user daserge commented on the pull request: https://github.com/apache/cordova-lib/pull/316#issuecomment-146644694 This update needs [cordova-hello-world-module to export its path](https://github.com/apache/cordova-app-hello-world/pull/14/files#diff-168726dbe96b3ce427e7fedce31bb0bcR3) as its used in `cfg.lib.www.url` assignment directly. Probably I should add a comment to this line for clearness. Alternatively we can use [require.resolve](https://nodejs.org/api/globals.html#globals_require_resolve) here instead - what do you think? --- 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
Github user nikhilkh commented on the pull request: https://github.com/apache/cordova-lib/pull/316#issuecomment-146646759 `require.resolve` seems a more elegant way to handle this. As long as that is supported in older versions of node. --- 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
Github user daserge commented on the pull request: https://github.com/apache/cordova-lib/pull/316#issuecomment-146656449 And actually the reason why this does not work with old `cordova-hello-app-world` is the fact that it did not have an entry point (index.js). --- 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
Github user omefire commented on the pull request: https://github.com/apache/cordova-lib/pull/316#issuecomment-146659921 @nikhilkh , but us using ```require.resolve``` here doesn't make it necessary to update the hello-world 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-lib pull request: CB-9712 CLI 5.3 breaks with node 3.3.3
Github user omefire commented on the pull request: https://github.com/apache/cordova-lib/pull/316#issuecomment-146660526 @daserge , Aha! that was a question I asked myself. I didn't see any entry point and assumed node.js figures it out via some weird rules. Can we also update package.json in the hello-world app to explicitly mention the entry point ? (index.js) --- 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
Github user daserge commented on the pull request: https://github.com/apache/cordova-lib/pull/316#issuecomment-146836947 @omefire, thank you for the review, I've addressed it - exposed `cordova-app-hello-world.dirname` explicitly, also included a few places with the similar path issues. --- 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
Github user vladimir-kotikov commented on the pull request: https://github.com/apache/cordova-lib/pull/316#issuecomment-146868901 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 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
Github user omefire commented on the pull request: https://github.com/apache/cordova-lib/pull/316#issuecomment-146967889 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 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
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