Skip to content

Commit

Permalink
[fix] Don't localize runlist
Browse files Browse the repository at this point in the history
Localizing runlist caused weird behavior, for example:

  1. Template was changed and system was published
  2. `quill configure <system>` was ran
  3. File was templated but `configure` script wasn't ran
  • Loading branch information
mmalecki committed Aug 8, 2013
1 parent 22a818d commit 04e0833
Showing 1 changed file with 0 additions and 27 deletions.
27 changes: 0 additions & 27 deletions lib/runlist.js
Expand Up @@ -140,33 +140,6 @@ var runlist = module.exports = function (options, callback) {
return list;
};

//
// ### function localize (options)
// #### @runlist {Array} List of systems to create runlist for.
// #### @installed {Object} List of systems to create runlist for.
// #### @log {Logger} Winston logger to write to.
//
// Localizes the specified `runlist` removing any systems already installed.
//
runlist.localize = function (runlist, installed, log) {
if (!installed) {
return runlist;
}

//
// Reduce the list for anything already installed
//
return runlist.map(function (system) {
if (installed[system.name] && installed[system.name].system) {
log.info('Already installed: ' + system.name.magenta);
}

return !installed[system.name] || !installed[system.name].system
? system
: null;
}).filter(Boolean);
};

//
// ### function filter (runlist, script, callback)
// #### @options {Object} Options for filtering the runlist.
Expand Down

0 comments on commit 04e0833

Please sign in to comment.