Skip to content

Commit db732af

Browse files
author
rolsen
committed
ljswitchboard-builder: Improvements to build clarity.
maxBuffer may not need to be so large, but 1024*1024 is the current default in modern Node. I did run into 'stdout maxBuffer exceeded' while experimenting with the build.
1 parent fdb6055 commit db732af

2 files changed

Lines changed: 3 additions & 2 deletions

File tree

ljswitchboard-builder/build_scripts/empty_directory.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -12,7 +12,7 @@ exports.emptyDirectoryOrDie = function(directoryName) {
1212
var deleteDirectory = function() {
1313
var repeatExecution = false;
1414
try {
15-
console.log('Cleaning temp project files directory. Attempt:', numDeleteAttempts);
15+
console.log(`Cleaning ${directoryName}. Attempt: ${numDeleteAttempts}`);
1616
// fse.emptyDirSync(directoryName);
1717
fsex.rmrfSync(directoryName);
1818
} catch(err) {

ljswitchboard-builder/build_scripts/install_production_dependencies.js

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -100,8 +100,9 @@ function innerInstallProductionDependency(name, directory) {
100100
var exec = require('child_process').exec;
101101
exec('npm install --production', {
102102
'cwd': directory,
103+
'maxBuffer': 1024 * 1024,
103104
}, function(error, stdout, stderr) {
104-
if(DEBUG_INSTALLATION) {
105+
if(DEBUG_INSTALLATION || error) {
105106
console.log('Finished installing', name);
106107
console.log({
107108
'error': error,

0 commit comments

Comments
 (0)