Skip to content

Commit e4b73e8

Browse files
committed
further streamline engine loading process and logging
1 parent bb47678 commit e4b73e8

File tree

1 file changed

+14
-5
lines changed

1 file changed

+14
-5
lines changed

builder/pattern_engines/pattern_engines.js

Lines changed: 14 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -21,13 +21,11 @@
2121
var engineNameForExtension; // generated mapping of extension to engine name
2222

2323

24-
// free private functions, for internal setup only
24+
// free "private" functions, for internal setup only
2525

2626
function findSupportedPatternEngineNames() {
2727
var foundPatternEngineNames = [];
2828

29-
console.log('patternEngines: begin diveSync ====================');
30-
3129
// find
3230
diveSync(enginesDirectory, {
3331
recursive: false,
@@ -44,7 +42,6 @@
4442
engineMatch = baseName.match(engineMatcher),
4543
foundEngineName = engineMatch[1];
4644

47-
console.log('patternEngines: FOUND ENGINE', foundEngineName);
4845
foundPatternEngineNames.push(foundEngineName);
4946
});
5047

@@ -53,13 +50,25 @@
5350

5451
// try to load all supported engines
5552
function loadAllEngines(enginesObject) {
53+
console.log('\nLoading engines...');
54+
5655
enginesObject.supportedPatternEngineNames.forEach(function (engineName) {
56+
var notLoaded = false;
57+
5758
try {
5859
enginesObject[engineName] = require('./engine_' + engineName);
5960
} catch (err) {
60-
console.log(err, 'pattern engine "' + engineName + '" not loaded. Did you install its dependency with npm?');
61+
// Handle errors loading each pattern engine. This will usually be
62+
// because the engine's renderer hasn't been installed by the end user
63+
// -- we don't include any of them (except mustache) by default as
64+
// depedencies in package.json.
65+
notLoaded = (err.code === 'MODULE_NOT_FOUND');
66+
} finally {
67+
console.log('-', engineName, 'engine:',
68+
notLoaded ? 'renderer not installed; engine disabled' : 'good to go');
6169
}
6270
});
71+
console.log('Done loading engines.\n');
6372
}
6473

6574
// produce a mapping between file extension and engine name for each of the

0 commit comments

Comments
 (0)