Avoiding using the output of the zip Task because it will shorten the path names

Author: StanleyGoldman

src/GitHub.Api/Installer/CopyHelper.cs

@@ -49,7 +49,6 @@ public static void CopyFolder(NPath fromPath, NPath toPath)
         public static void CopyFolderContents(NPath fromPath, NPath toPath)
         {
             Logger.Trace("CopyFolderContents fromPath: {0} toPath:{1}", fromPath.ToString(), toPath.ToString());
-
             toPath.DeleteContents();
             fromPath.MoveFiles(toPath, true);
         }

src/GitHub.Api/Installer/GitInstaller.cs

@@ -301,13 +301,13 @@ private GitInstallationState ExtractGit(GitInstallationState state)
                         return true;
                     });
                 unzipTask.Progress(p => Progress.UpdateProgress(40 + (long)(20 * p.Percentage), 100, unzipTask.Message));
-                var source = unzipTask.RunSynchronously();
+                unzipTask.RunSynchronously();
                 var target = state.GitInstallationPath;
                 if (unzipTask.Successful)
                 {
-                    Logger.Trace("Moving Git source:{0} target:{1}", source.ToString(), target.ToString());
+                    Logger.Trace("Moving Git source:{0} target:{1}", gitExtractPath.ToString(), target.ToString());
 
-                    CopyHelper.Copy(source, target);
+                    CopyHelper.Copy(gitExtractPath, target);
 
                     state.GitIsValid = true;
 
@@ -327,13 +327,13 @@ private GitInstallationState ExtractGit(GitInstallationState state)
                         return true;
                     });
                 unzipTask.Progress(p => Progress.UpdateProgress(60 + (long)(20 * p.Percentage), 100, unzipTask.Message));
-                var source = unzipTask.RunSynchronously();
+                unzipTask.RunSynchronously();
                 var target = state.GitLfsInstallationPath;
                 if (unzipTask.Successful)
                 {
-                    Logger.Trace("Moving GitLFS source:{0} target:{1}", source.ToString(), target.ToString());
+                    Logger.Trace("Moving GitLFS source:{0} target:{1}", gitLfsExtractPath.ToString(), target.ToString());
 
-                    CopyHelper.Copy(source, target);
+                    CopyHelper.Copy(gitLfsExtractPath, target);
 
                     state.GitLfsIsValid = true;
                 }

src/GitHub.Api/Installer/OctorunInstaller.cs

@@ -33,12 +33,13 @@ public NPath SetupOctorunIfNeeded()
 
             GrabZipFromResources();
 
-            var tempZipExtractPath = NPath.CreateTempDirectory("octorun_extract_archive_path");
+            var extractPath = NPath.CreateTempDirectory("octorun_extract_archive_path");
             var unzipTask = new UnzipTask(taskManager.Token, installDetails.ZipFile,
-                    tempZipExtractPath, sharpZipLibHelper,
+                    extractPath, sharpZipLibHelper,
                     fileSystem)
                     .Catch(e => { Logger.Error(e, "Error extracting octorun"); return true; });
-            var extractPath = unzipTask.RunSynchronously();
+            unzipTask.RunSynchronously();
+
             if (unzipTask.Successful)
                 path = MoveOctorun(extractPath.Combine("octorun"));
             return path;