defer loading of knex-migrator until it's needed

TryGhost · Oct 18, 2016 · 79fb9b5 · 79fb9b5
1 parent 89229ba
commit 79fb9b5
Showing 1 changed file with 5 additions and 5 deletions.
diff --git a/lib/commands/start.js b/lib/commands/start.js
@@ -1,5 +1,4 @@
-var KnexMigrator = require('knex-migrator'),
-    BaseCommand = require('./base');
+var BaseCommand = require('./base');
 
 module.exports = BaseCommand.extend({
     name: 'start',
@@ -29,7 +28,7 @@ module.exports = BaseCommand.extend({
             Promise = require('bluebird'),
             path = require('path'),
             self = this,
-            environment, config, pm, cliConfig, knexMigrator;
+            environment, config, pm, cliConfig;
 
         options = options || {};
         environment = (options.production || !options.development) ? 'production' : 'development';
@@ -46,6 +45,8 @@ module.exports = BaseCommand.extend({
         // TODO: rethink this
         return this.runCommand('config', 'paths.contentPath', path.join(process.cwd(), 'content'), {environment: environment})
             .then(function () {
+                var KnexMigrator = require('knex-migrator'),
+                    knexMigrator;
                 /**
                  * If we require any JS file from Ghost source code (for example the config), then it's always env-less!
                  *
@@ -67,8 +68,7 @@ module.exports = BaseCommand.extend({
                     .catch(function () {
                         return knexMigrator.init();
                     });
-            })
-            .then(function () {
+            }).then(function () {
                 return self.ui.run(
                     Promise.resolve(pm.start(process.cwd(), environment)),
                     'Starting Ghost instance'