7 files changed, 80 insertions, 22 deletions

diff --git a/src/WixToolset.Core.WindowsInstaller/Bind/GenerateDatabaseCommand.cs b/src/WixToolset.Core.WindowsInstaller/Bind/GenerateDatabaseCommand.cs
index 1a77b84e..3ab3b601 100644
--- a/src/WixToolset.Core.WindowsInstaller/Bind/GenerateDatabaseCommand.cs
+++ b/src/WixToolset.Core.WindowsInstaller/Bind/GenerateDatabaseCommand.cs
@@ -334,10 +334,10 @@ namespace WixToolset.Core.WindowsInstaller.Bind
                     db.Commit();
                 }
             }
-            catch (IOException)
+            catch (IOException e)
             {
                 // TODO: this error message doesn't seem specific enough
-                throw new WixFileNotFoundException(new SourceLineNumber(this.OutputPath), this.OutputPath);
+                throw new WixException(ErrorMessages.FileNotFound(new SourceLineNumber(this.OutputPath), this.OutputPath), e);
             }
         }
 
diff --git a/src/WixToolset.Core/Bind/FileResolver.cs b/src/WixToolset.Core/Bind/FileResolver.cs
index 86075e46..01dfe36c 100644
--- a/src/WixToolset.Core/Bind/FileResolver.cs
+++ b/src/WixToolset.Core/Bind/FileResolver.cs
@@ -43,17 +43,24 @@ namespace WixToolset.Core.Bind
 
         public string Resolve(SourceLineNumber sourceLineNumbers, IntermediateTupleDefinition tupleDefinition, string source)
         {
+            var checkedPaths = new List<string>();
+
             foreach (var extension in this.LibrarianExtensions)
             {
-                var resolved = extension.Resolve(sourceLineNumbers, tupleDefinition, source);
+                var resolved = extension.ResolveFile(sourceLineNumbers, tupleDefinition, source);
+
+                if (resolved?.CheckedPaths != null)
+                {
+                    checkedPaths.AddRange(resolved.CheckedPaths);
+                }
 
-                if (null != resolved)
+                if (!String.IsNullOrEmpty(resolved?.Path))
                 {
-                    return resolved;
+                    return resolved?.Path;
                 }
             }
 
-            return this.ResolveUsingBindPaths(source, tupleDefinition, sourceLineNumbers, BindStage.Normal);
+            return this.MustResolveUsingBindPaths(source, tupleDefinition, sourceLineNumbers, BindStage.Normal, checkedPaths);
         }
 
         /// <summary>
@@ -66,24 +73,32 @@ namespace WixToolset.Core.Bind
         /// <returns>Should return a valid path for the stream to be imported.</returns>
         public string ResolveFile(string source, IntermediateTupleDefinition tupleDefinition, SourceLineNumber sourceLineNumbers, BindStage bindStage)
         {
+            var checkedPaths = new List<string>();
+
             foreach (var extension in this.ResolverExtensions)
             {
                 var resolved = extension.ResolveFile(source, tupleDefinition, sourceLineNumbers, bindStage);
 
-                if (null != resolved)
+                if (resolved?.CheckedPaths != null)
+                {
+                    checkedPaths.AddRange(resolved.CheckedPaths);
+                }
+
+                if (!String.IsNullOrEmpty(resolved?.Path))
                 {
-                    return resolved;
+                    return resolved?.Path;
                 }
             }
 
-            return this.ResolveUsingBindPaths(source, tupleDefinition, sourceLineNumbers, bindStage);
+            return this.MustResolveUsingBindPaths(source, tupleDefinition, sourceLineNumbers, bindStage, checkedPaths);
         }
 
-        private string ResolveUsingBindPaths(string source, IntermediateTupleDefinition tupleDefinition, SourceLineNumber sourceLineNumbers, BindStage bindStage)
+        private string MustResolveUsingBindPaths(string source, IntermediateTupleDefinition tupleDefinition, SourceLineNumber sourceLineNumbers, BindStage bindStage, List<string> checkedPaths)
         {
             string resolved = null;
 
             // If the file exists, we're good to go.
+            checkedPaths.Add(source);
             if (CheckFileExists(source))
             {
                 resolved = source;
@@ -121,6 +136,7 @@ namespace WixToolset.Core.Bind
                     {
                         var filePath = Path.Combine(bindPath.Path, pathWithoutSourceDir);
 
+                        checkedPaths.Add(filePath);
                         if (CheckFileExists(filePath))
                         {
                             resolved = filePath;
@@ -131,6 +147,7 @@ namespace WixToolset.Core.Bind
                     {
                         var filePath = Path.Combine(bindPath.Path, path);
 
+                        checkedPaths.Add(filePath);
                         if (CheckFileExists(filePath))
                         {
                             resolved = filePath;
@@ -141,10 +158,9 @@ namespace WixToolset.Core.Bind
 
             if (null == resolved)
             {
-                throw new WixFileNotFoundException(sourceLineNumbers, source, tupleDefinition.Name);
+                throw new WixException(ErrorMessages.FileNotFound(sourceLineNumbers, source, tupleDefinition.Name, checkedPaths));
             }
 
-            // Didn't find the file.
             return resolved;
         }
 
diff --git a/src/WixToolset.Core/Bind/ResolveFieldsCommand.cs b/src/WixToolset.Core/Bind/ResolveFieldsCommand.cs
index 0d5c3142..b7ed8a18 100644
--- a/src/WixToolset.Core/Bind/ResolveFieldsCommand.cs
+++ b/src/WixToolset.Core/Bind/ResolveFieldsCommand.cs
@@ -152,10 +152,9 @@ namespace WixToolset.Core.Bind
                                     }
 #endif
                                 }
-                                catch (WixFileNotFoundException)
+                                catch (WixException e)
                                 {
-                                    // display the error with source line information
-                                    this.Messaging.Write(ErrorMessages.FileNotFound(row.SourceLineNumbers, objectField.Path));
+                                    this.Messaging.Write(e.Error);
                                 }
                             }
 
diff --git a/src/WixToolset.Core/Bind/TransferFilesCommand.cs b/src/WixToolset.Core/Bind/TransferFilesCommand.cs
index b89d3d03..b9c54a14 100644
--- a/src/WixToolset.Core/Bind/TransferFilesCommand.cs
+++ b/src/WixToolset.Core/Bind/TransferFilesCommand.cs
@@ -63,7 +63,7 @@ namespace WixToolset.Core.Bind
                     }
                     catch (FileNotFoundException e)
                     {
-                        throw new WixFileNotFoundException(fileTransfer.SourceLineNumbers, e.FileName);
+                        throw new WixException(ErrorMessages.FileNotFound(fileTransfer.SourceLineNumbers, e.FileName));
                     }
                     catch (DirectoryNotFoundException)
                     {
diff --git a/src/WixToolset.Core/CommandLine/CommandLineParser.cs b/src/WixToolset.Core/CommandLine/CommandLineParser.cs
index 92944ab2..d518931a 100644
--- a/src/WixToolset.Core/CommandLine/CommandLineParser.cs
+++ b/src/WixToolset.Core/CommandLine/CommandLineParser.cs
@@ -22,6 +22,8 @@ namespace WixToolset.Core.CommandLine
 
     internal class CommandLineParser : ICommandLineParser
     {
+        private static readonly char[] BindPathSplit = { '=' };
+
         public CommandLineParser(IServiceProvider serviceProvider)
         {
             this.ServiceProvider = serviceProvider;
@@ -380,15 +382,15 @@ namespace WixToolset.Core.CommandLine
 
             foreach (var bindPath in bindPaths)
             {
-                var bp = BindPath.Parse(bindPath);
+                var bp = ParseBindPath(bindPath);
 
-                if (Directory.Exists(bp.Path))
+                if (File.Exists(bp.Path))
                 {
-                    result.Add(bp);
+                    this.Messaging.Write(ErrorMessages.ExpectedDirectoryGotFile("-bindpath", bp.Path));
                 }
-                else if (File.Exists(bp.Path))
+                else
                 {
-                    this.Messaging.Write(ErrorMessages.ExpectedDirectoryGotFile("-bindpath", bp.Path));
+                    result.Add(bp);
                 }
             }
 
@@ -407,5 +409,11 @@ namespace WixToolset.Core.CommandLine
 
             return false;
         }
+
+        public static BindPath ParseBindPath(string bindPath)
+        {
+            string[] namedPath = bindPath.Split(BindPathSplit, 2);
+            return (1 == namedPath.Length) ? new BindPath(namedPath[0]) : new BindPath(namedPath[0], namedPath[1]);
+        }
     }
 }
diff --git a/src/WixToolset.Core/Preprocessor.cs b/src/WixToolset.Core/Preprocessor.cs
index ac8cefe3..aca954bd 100644
--- a/src/WixToolset.Core/Preprocessor.cs
+++ b/src/WixToolset.Core/Preprocessor.cs
@@ -649,7 +649,7 @@ namespace WixToolset.Core
 
             if (null == includeFile)
             {
-                throw new WixFileNotFoundException(sourceLineNumbers, includePath, "include");
+                throw new WixException(ErrorMessages.FileNotFound(sourceLineNumbers, includePath, "include"));
             }
 
             using (XmlReader reader = XmlReader.Create(includeFile, DocumentXmlReaderSettings))
diff --git a/src/test/WixToolsetTest.CoreIntegration/MsiFixture.cs b/src/test/WixToolsetTest.CoreIntegration/MsiFixture.cs
index 7a10f71e..9d46ba98 100644
--- a/src/test/WixToolsetTest.CoreIntegration/MsiFixture.cs
+++ b/src/test/WixToolsetTest.CoreIntegration/MsiFixture.cs
@@ -171,6 +171,41 @@ namespace WixToolsetTest.CoreIntegration
         }
 
         [Fact]
+        public void CanFailBuildMissingFile()
+        {
+            var folder = TestData.Get(@"TestData\SingleFile");
+
+            using (var fs = new DisposableFileSystem())
+            {
+                var baseFolder = fs.GetFolder();
+                var intermediateFolder = Path.Combine(baseFolder, "obj");
+
+                var result = WixRunner.Execute(new[]
+                {
+                    "build",
+                    Path.Combine(folder, "Package.wxs"),
+                    Path.Combine(folder, "PackageComponents.wxs"),
+                    "-loc", Path.Combine(folder, "Package.en-us.wxl"),
+                    "-bindpath", Path.Combine(folder, "does-not-exist"),
+                    "-bindpath", Path.Combine(folder, "also-does-not-exist"),
+                    "-intermediateFolder", intermediateFolder,
+                    "-o", Path.Combine(baseFolder, @"bin\test.msi")
+                }, out var messages);
+                Assert.Equal(103, result);
+
+                var error = messages.Single(m => m.Level == MessageLevel.Error);
+                var errorMessage = error.ToString();
+                var checkedPaths = errorMessage.Substring(errorMessage.IndexOf(':') + 1).Split(new[] { ',' }).Select(s => s.Trim()).ToArray();
+                Assert.Equal(new[]
+                {
+                    "test.txt",
+                    Path.Combine(folder, "does-not-exist", "test.txt"),
+                    Path.Combine(folder, "also-does-not-exist", "test.txt"),
+                }, checkedPaths);
+            }
+        }
+
+        [Fact]
         public void CanLoadPdbGeneratedByBuild()
         {
             var folder = TestData.Get(@"TestData\MultiFileCompressed");