uglifyjs.minify expects the code and not a path

For whatever reason that this has worked in the past, in node 14+ the behavior appears to have been changed.

The `cascade` option is not mentioned in the list of valid options and since error message were implicitly suppressed, this went by unnoticed.

Fixes #3520

diff --git a/extra/_buildCore.js b/extra/_buildCore.js
index 2338f1837bc18f21464b3f85628f1924f41a77b3..ffb511ce785aeb2cb3831a48166b2abb922b40ee 100644 (file)
--- a/extra/_buildCore.js
+++ b/extra/_buildCore.js
@@ -118,7 +118,7 @@ fs.readdirSync("./")
     .forEach(filename => {
         console.time(filename);
         {
-            let output = compiler.compile(process.cwd() + `/${filename}`);
+            let output = compiler.compile(fs.readFileSync(process.cwd() + `/${filename}`, 'utf-8'));
             fs.writeFileSync(filename.replace(/\.js$/, '.min.js'), output.code);
         }
         console.timeEnd(filename);

diff --git a/extra/compiler.js b/extra/compiler.js
index b700ebc9cfafeebdb38ce2ded6882f691eda135b..f5f18eb6ce7a2d407fae78c4b8a65d7823e16238 100644 (file)
--- a/extra/compiler.js
+++ b/extra/compiler.js
@@ -14,7 +14,6 @@ const uglifyJsConfig = {
         hoist_vars: true,
         if_return: true,
         join_vars: true,
-        cascade: true,
         /* this is basically the `--define` argument */
         global_defs: {
             COMPILER_TARGET_DEFAULT: false
@@ -30,4 +29,4 @@ module.exports = {
             filename,
             Object.assign(uglifyJsConfig, overrides));
     }
-}
\ No newline at end of file
+}