Merge pull request #431 from Microsoft/users/ersciple/m104scorch2

Fix scorch.
microsoft · Jul 21, 2016 · 0b2a9c1 · 0b2a9c1
2 parents d99a96a + df35ceb
commit 0b2a9c1
Show file tree

Hide file tree

Showing 2 changed files with 1 addition and 4 deletions.
diff --git a/src/Agent.Worker/Build/TFCommandManager.cs b/src/Agent.Worker/Build/TFCommandManager.cs
@@ -48,7 +48,7 @@ public string ResolvePath(string serverPath)
             return localPath?.Trim() ?? string.Empty;
         }
 
-        public async Task ScorchAsync() => await RunCommandAsync(FormatFlags.OmitCollectionUrl, "vc", "scorch", "/diff", "/recursive", "/unmapped", SourcesDirectory);
+        public async Task ScorchAsync() => await RunCommandAsync(FormatFlags.OmitCollectionUrl, "vc", "scorch", "/diff", "/recursive", "/unmapped", "*");
 
         public async Task ShelveAsync(string shelveset, string commentFile)
         {

diff --git a/src/Agent.Worker/Build/TfsVCSourceProvider.cs b/src/Agent.Worker/Build/TfsVCSourceProvider.cs
@@ -163,9 +163,6 @@ public async Task GetSourceAsync(
                         {
                             // Scorch failed.
                             // Warn, drop the folder, and re-clone.
-                            // TODO: Fix tf.exe scorch and remove this catch handler. This catch handler was
-                            //       added because scorch fails to interpret the workspace when a root mapping
-                            //       does not exist.
                             executionContext.Warning(ex.Message);
                             existingTFWorkspace = null;
                         }