Closure compiler does not fail when encountering JS errors

[rmaccracken]

When our JS code has an error, such as a trailing comma, the closure compiler reports an error, but continues to process all files. In the end, the error seems to be ignored and reports that the build succeeded.

I think it makes more sense for the compiler to fail and report these errors. We have patched the following build script as follows:

file: dojo/util/build/transforms/optimizer/closure.js

change line: compiler.compile(externSourceFile, jsSourceFile, options);

to: var complierResult = compiler.compile(externSourceFile, jsSourceFile, options); if (!complierResult.success)

throw new Error("Error during Closure optimization");

It would be great to have this either as a build option or just part of the normal build.