From 36cb8e8fcb8afd2965ffe26e7755eb11838264d7 Mon Sep 17 00:00:00 2001
From: Jesse Plamondon-Willard <Pathoschild@users.noreply.github.com>
Date: Sun, 7 Mar 2021 17:42:55 -0500
Subject: keep window open when installer crashes

---
 src/SMAPI.Installer/Program.cs | 25 ++++++++++++++++++++++++-
 1 file changed, 24 insertions(+), 1 deletion(-)

(limited to 'src/SMAPI.Installer')

diff --git a/src/SMAPI.Installer/Program.cs b/src/SMAPI.Installer/Program.cs
index 6c479621..d9c31dd6 100644
--- a/src/SMAPI.Installer/Program.cs
+++ b/src/SMAPI.Installer/Program.cs
@@ -3,6 +3,7 @@ using System.Diagnostics.CodeAnalysis;
 using System.IO;
 using System.IO.Compression;
 using System.Reflection;
+using System.Threading;
 
 namespace StardewModdingApi.Installer
 {
@@ -49,7 +50,15 @@ namespace StardewModdingApi.Installer
 
             // launch installer
             var installer = new InteractiveInstaller(bundleDir.FullName);
-            installer.Run(args);
+
+            try
+            {
+                installer.Run(args);
+            }
+            catch (Exception ex)
+            {
+                Program.PrintErrorAndExit($"The installer failed with an unexpected exception.\nIf you need help fixing this error, see https://smapi.io/help\n\n{ex}");
+            }
         }
 
         /*********
@@ -76,5 +85,19 @@ namespace StardewModdingApi.Installer
                 return null;
             }
         }
+
+        /// <summary>Write an error directly to the console and exit.</summary>
+        /// <param name="message">The error message to display.</param>
+        private static void PrintErrorAndExit(string message)
+        {
+            Console.ForegroundColor = ConsoleColor.Red;
+            Console.WriteLine(message);
+            Console.ResetColor();
+
+            Console.WriteLine("Game has ended. Press any key to exit.");
+            Thread.Sleep(100);
+            Console.ReadKey();
+            Environment.Exit(0);
+        }
     }
 }
-- 
cgit 


From 0ed46c09104b823a5784690fa50e7022efc12c8f Mon Sep 17 00:00:00 2001
From: Jesse Plamondon-Willard <Pathoschild@users.noreply.github.com>
Date: Sun, 7 Mar 2021 18:04:48 -0500
Subject: add descriptive error for PathTooLongException which crashes SMAPI or
 the installer

---
 src/SMAPI.Installer/InteractiveInstaller.cs | 12 ++++++++++++
 1 file changed, 12 insertions(+)

(limited to 'src/SMAPI.Installer')

diff --git a/src/SMAPI.Installer/InteractiveInstaller.cs b/src/SMAPI.Installer/InteractiveInstaller.cs
index ff74a659..3d673719 100644
--- a/src/SMAPI.Installer/InteractiveInstaller.cs
+++ b/src/SMAPI.Installer/InteractiveInstaller.cs
@@ -279,6 +279,7 @@ namespace StardewModdingApi.Installer
             /*********
             ** Step 4: validate assumptions
             *********/
+            // executable exists
             if (!File.Exists(paths.ExecutablePath))
             {
                 this.PrintError("The detected game install path doesn't contain a Stardew Valley executable.");
@@ -286,6 +287,17 @@ namespace StardewModdingApi.Installer
                 return;
             }
 
+            // game folder doesn't contain paths beyond the max limit
+            {
+                string[] tooLongPaths = PathUtilities.GetTooLongPaths(Path.Combine(paths.GamePath, "Mods")).ToArray();
+                if (tooLongPaths.Any())
+                {
+                    this.PrintError($"SMAPI can't install to the detected game folder, because some of its files exceed the maximum {context.Platform} path length.\nIf you need help fixing this error, see https://smapi.io/help\n\nAffected paths:\n   {string.Join("\n   ", tooLongPaths)}");
+                    Console.ReadLine();
+                    return;
+                }
+            }
+
 
             /*********
             ** Step 5: ask what to do
-- 
cgit