Return exit code

Author: Yui T

src/compiler/tsc.ts

@@ -208,32 +208,32 @@ module ts {
         // setting up localization, report them and quit.
         if (commandLine.errors.length > 0) {
             reportDiagnostics(commandLine.errors);
-            return sys.exit(1);
+            return sys.exit(EmitReturnStatus.CompilerOptionsErrors);
         }
 
         if (commandLine.options.version) {
             reportDiagnostic(createCompilerDiagnostic(Diagnostics.Version_0, version));
-            return sys.exit(0);
+            return sys.exit(EmitReturnStatus.CompilerOptionsErrors);
         }
 
         if (commandLine.options.help || commandLine.filenames.length === 0) {
             printVersion();
             printHelp();
-            return sys.exit(0);
+            return sys.exit(EmitReturnStatus.CompilerOptionsErrors);
         }
 
         var defaultCompilerHost = createCompilerHost(commandLine.options);
 
         if (commandLine.options.watch) {
             if (!sys.watchFile) {
                 reportDiagnostic(createCompilerDiagnostic(Diagnostics.The_current_host_does_not_support_the_0_option, "--watch"));
-                return sys.exit(1);
+                return sys.exit(EmitReturnStatus.CompilerOptionsErrors);
             }
 
             watchProgram(commandLine, defaultCompilerHost);
         }
         else {
-            var result = compile(commandLine, defaultCompilerHost).errors.length > 0 ? 1 : 0;
+            var result = compile(commandLine, defaultCompilerHost).exitStatus
             return sys.exit(result);
         }
     }
@@ -328,23 +328,28 @@ module ts {
 
     function compile(commandLine: ParsedCommandLine, compilerHost: CompilerHost) {
         var parseStart = new Date().getTime();
-        var program = createProgram(commandLine.filenames, commandLine.options, compilerHost);
+        var compilerOptions = commandLine.options;
+        var program = createProgram(commandLine.filenames, compilerOptions, compilerHost);
 
         var bindStart = new Date().getTime();
-        var errors = program.getDiagnostics();
-        if (errors.length) {
+        var syntacticErrors = program.getDiagnostics();
+        var emitErrors: Diagnostic[];
+        var semanticErrors: Diagnostic[];
+        var errors: Diagnostic[];
+
+        if (syntacticErrors.length) {
             var checkStart = bindStart;
             var emitStart = bindStart;
             var reportStart = bindStart;
         }
         else {
             var checker = program.getTypeChecker(/*fullTypeCheckMode*/ true);
             var checkStart = new Date().getTime();
-            var semanticErrors = checker.getDiagnostics();
+            semanticErrors = checker.getDiagnostics();
             var emitStart = new Date().getTime();
-            var emitErrors = checker.emitFiles().errors;
+            emitErrors = checker.emitFiles().errors;
             var reportStart = new Date().getTime();
-            errors = concatenate(semanticErrors, emitErrors);
+            errors = concatenate(syntacticErrors, concatenate(semanticErrors, emitErrors));
         }
 
         reportDiagnostics(errors);
@@ -366,7 +371,21 @@ module ts {
             reportTimeStatistic("Total time", reportStart - parseStart);
         }
 
-        return { program: program, errors: errors };
+        // Check types of diagnostics and return associated exit code 
+        if (syntacticErrors.length > 0) {
+            return { program: program, exitStatus: EmitReturnStatus.AllOutputGenerationSkipped };
+        } else if (semanticErrors.length > 0 && !compilerOptions.declaration) {
+            // No '-d' is specified; javascript file is generated with semantic errors
+            return { program: program, exitStatus: EmitReturnStatus.JSGeneratedWithSemanticErrors };
+        } else if (semanticErrors.length > 0 && compilerOptions.declaration) {
+            // '-d' is specified; javascript file will be emitted with semantic errors but declaration file will be skipped
+            return { program: program, exitStatus: EmitReturnStatus.DeclarationGenerationSkipped };
+        } else if (emitErrors.length > 0 && compilerOptions.declaration) {
+            return { program: program, exitStatus: EmitReturnStatus.EmitErrorsEncountered };
+        } else {
+            // There is no error message
+            return { program: program, exitStatus: EmitReturnStatus.Succeeded };
+        }
 
     }
 

src/compiler/types.ts

@@ -606,7 +606,8 @@ module ts {
         AllOutputGenerationSkipped = 1,     // No .js generated because of syntax errors, or compiler options errors, nothing generated
         JSGeneratedWithSemanticErrors = 2,  // .js and .map generated with semantic errors
         DeclarationGenerationSkipped = 3,   // .d.ts generation skipped because of semantic errors or declaration emitter specific errors; Output .js with semantic errors
-        EmitErrorsEncountered = 4           // Emitter errors occurred during emitting process
+        EmitErrorsEncountered = 4,          // Emitter errors occurred during emitting process
+        CompilerOptionsErrors = 5,          // Errors occurred in parsing compiler options
     }
 
     export interface EmitResult {