You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@cordova.apache.org by js...@apache.org on 2014/08/21 20:52:34 UTC
js commit: CB-7349 Tell users to run npm install
Repository: cordova-js
Updated Branches:
refs/heads/master 0b5a3a877 -> 58e9d6cd0
CB-7349 Tell users to run npm install
when browserify/jasmine-node are missing
Project: http://git-wip-us.apache.org/repos/asf/cordova-js/repo
Commit: http://git-wip-us.apache.org/repos/asf/cordova-js/commit/58e9d6cd
Tree: http://git-wip-us.apache.org/repos/asf/cordova-js/tree/58e9d6cd
Diff: http://git-wip-us.apache.org/repos/asf/cordova-js/diff/58e9d6cd
Branch: refs/heads/master
Commit: 58e9d6cd0b5ff63e6ba566a49cdedcaea897c3db
Parents: 0b5a3a8
Author: Josh Soref <js...@blackberry.com>
Authored: Wed Aug 20 19:06:35 2014 -0400
Committer: Josh Soref <js...@blackberry.com>
Committed: Thu Aug 21 14:51:15 2014 -0400
----------------------------------------------------------------------
tasks/compile-browserify.js | 7 +++++++
tasks/test.js | 7 +++++++
2 files changed, 14 insertions(+)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/cordova-js/blob/58e9d6cd/tasks/compile-browserify.js
----------------------------------------------------------------------
diff --git a/tasks/compile-browserify.js b/tasks/compile-browserify.js
index 1d28510..549fc0c 100644
--- a/tasks/compile-browserify.js
+++ b/tasks/compile-browserify.js
@@ -16,6 +16,13 @@
* specific language governing permissions and limitations
* under the License.
*/
+try {
+ require('browserify');
+} catch (e) {
+ console.error("\nbrowserify is not installed, you need to:\n\trun `npm install` from " + require('path').dirname(__dirname)+"\n");
+ process.exit(1);
+}
+
var generate = require('./lib/packager-browserify');
module.exports = function(grunt) {
http://git-wip-us.apache.org/repos/asf/cordova-js/blob/58e9d6cd/tasks/test.js
----------------------------------------------------------------------
diff --git a/tasks/test.js b/tasks/test.js
index edd6491..b1df54e 100644
--- a/tasks/test.js
+++ b/tasks/test.js
@@ -16,6 +16,13 @@
* specific language governing permissions and limitations
* under the License.
*/
+try {
+ require('jasmine-node');
+} catch (e) {
+ console.error("\njasmine-node is not installed, you need to:\n\trun `npm install` from " + require('path').dirname(__dirname)+"\n");
+ process.exit(1);
+}
+
module.exports = function(grunt) {
grunt.registerTask('_test', 'Runs test in node', function() {
var done = this.async();