Report preprocessor exceptions as errors. Fixes wixtoolset/issues#5881.

2 files changed, 33 insertions, 4 deletions

diff --git a/src/WixToolset.Core/CommandLine/BuildCommand.cs b/src/WixToolset.Core/CommandLine/BuildCommand.cs
index b460e48f..16c98c83 100644
--- a/src/WixToolset.Core/CommandLine/BuildCommand.cs
+++ b/src/WixToolset.Core/CommandLine/BuildCommand.cs
@@ -6,6 +6,7 @@ namespace WixToolset.Core.CommandLine
     using System.Collections.Generic;
     using System.IO;
     using System.Linq;
+    using System.Xml.Linq;
     using WixToolset.Data;
     using WixToolset.Extensibility.Data;
     using WixToolset.Extensibility.Services;
@@ -173,7 +174,16 @@ namespace WixToolset.Core.CommandLine
                 preprocessor.Platform = Platform.X86; // TODO: set this correctly
                 preprocessor.SourcePath = sourceFile.SourcePath;
                 preprocessor.Variables = this.PreprocessorVariables;
-                var document = preprocessor.Execute();
+
+                XDocument document = null;
+                try
+                {
+                    document = preprocessor.Execute();
+                }
+                catch (WixException e)
+                {
+                    this.Messaging.Write(e.Error);
+                }
 
                 if (this.Messaging.EncounteredError)
                 {
diff --git a/src/WixToolset.Core/CommandLine/CompileCommand.cs b/src/WixToolset.Core/CommandLine/CompileCommand.cs
index 6bd0f25a..ec1ce602 100644
--- a/src/WixToolset.Core/CommandLine/CompileCommand.cs
+++ b/src/WixToolset.Core/CommandLine/CompileCommand.cs
@@ -4,26 +4,31 @@ namespace WixToolset.Core.CommandLine
 {
     using System;
     using System.Collections.Generic;
+    using System.Xml.Linq;
     using WixToolset.Data;
     using WixToolset.Extensibility.Data;
+    using WixToolset.Extensibility.Services;
 
     internal class CompileCommand : ICommandLineCommand
     {
         public CompileCommand(IServiceProvider serviceProvider, IEnumerable<SourceFile> sources, IDictionary<string, string> preprocessorVariables)
         {
-            this.PreprocessorVariables = preprocessorVariables;
             this.ServiceProvider = serviceProvider;
+            this.Messaging = serviceProvider.GetService<IMessaging>();
             this.SourceFiles = sources;
+            this.PreprocessorVariables = preprocessorVariables;
         }
 
         private IServiceProvider ServiceProvider { get; }
 
-        public IEnumerable<string> IncludeSearchPaths { get; }
+        public IMessaging Messaging { get; }
 
         private IEnumerable<SourceFile> SourceFiles { get; }
 
         private IDictionary<string, string> PreprocessorVariables { get; }
 
+        public IEnumerable<string> IncludeSearchPaths { get; }
+
         public int Execute()
         {
             foreach (var sourceFile in this.SourceFiles)
@@ -33,7 +38,21 @@ namespace WixToolset.Core.CommandLine
                 preprocessor.Platform = Platform.X86; // TODO: set this correctly
                 preprocessor.SourcePath = sourceFile.SourcePath;
                 preprocessor.Variables = new Dictionary<string, string>(this.PreprocessorVariables);
-                var document = preprocessor.Execute();
+
+                XDocument document = null;
+                try
+                {
+                    document = preprocessor.Execute();
+                }
+                catch (WixException e)
+                {
+                    this.Messaging.Write(e.Error);
+                }
+
+                if (this.Messaging.EncounteredError)
+                {
+                    continue;
+                }
 
                 var compiler = new Compiler(this.ServiceProvider);
                 compiler.OutputPath = sourceFile.OutputPath;