From a62b8efb741db4335a874f000840ee26d12c7b69 Mon Sep 17 00:00:00 2001
From: Chris McGee <87777443+cmcgee1024@users.noreply.github.com>
Date: Thu, 13 Mar 2025 14:52:12 -0400
Subject: [PATCH] Remove rehashing instructions when using the fish shell
 (#250)

Change Debian 12 binutils-gold virtual package to binutils

Move some progress indications under verbose flag for shorter default install log
---
 Sources/LinuxPlatform/Linux.swift |  7 +++++--
 Sources/Swiftly/Init.swift        |  5 +++--
 Sources/Swiftly/Install.swift     | 13 +++++++++++--
 3 files changed, 19 insertions(+), 6 deletions(-)

diff --git a/Sources/LinuxPlatform/Linux.swift b/Sources/LinuxPlatform/Linux.swift
index 9bb11f28..85c69f20 100644
--- a/Sources/LinuxPlatform/Linux.swift
+++ b/Sources/LinuxPlatform/Linux.swift
@@ -205,7 +205,7 @@ public struct Linux: Platform {
             ]
         case "debian12":
             [
-                "binutils-gold",
+                "binutils",
                 "libicu-dev",
                 "libcurl4-openssl-dev",
                 "libedit-dev",
@@ -392,7 +392,10 @@ public struct Linux: Platform {
     }
 
     public func verifySignature(httpClient: SwiftlyHTTPClient, archiveDownloadURL: URL, archive: URL, verbose: Bool) async throws {
-        SwiftlyCore.print("Downloading toolchain signature...")
+        if verbose {
+            SwiftlyCore.print("Downloading toolchain signature...")
+        }
+
         let sigFile = self.getTempFilePath()
         let _ = FileManager.default.createFile(atPath: sigFile.path, contents: nil)
         defer {
diff --git a/Sources/Swiftly/Init.swift b/Sources/Swiftly/Init.swift
index 5ce648bc..603c4267 100644
--- a/Sources/Swiftly/Init.swift
+++ b/Sources/Swiftly/Init.swift
@@ -86,7 +86,7 @@ internal struct Init: SwiftlyCommand {
             """)
 
             guard SwiftlyCore.promptForConfirmation(defaultBehavior: true) else {
-                throw SwiftlyError(message: "Swiftly installation has been cancelled")
+                throw SwiftlyError(message: "swiftly installation has been cancelled")
             }
         }
 
@@ -250,7 +250,8 @@ internal struct Init: SwiftlyCommand {
                 """)
             }
 
-            if pathChanged && !quietShellFollowup {
+            // Fish doesn't have path caching, so this might only be needed for bash/zsh
+            if pathChanged && !quietShellFollowup && !shell.hasSuffix("fish") {
                 SwiftlyCore.print("""
                 Your shell caches items on your path for better performance. Swiftly has added items to your path that may not get picked up right away. You can run this command to update your shell to get these items.
 
diff --git a/Sources/Swiftly/Install.swift b/Sources/Swiftly/Install.swift
index 285bcdfe..fa32a38c 100644
--- a/Sources/Swiftly/Install.swift
+++ b/Sources/Swiftly/Install.swift
@@ -103,7 +103,14 @@ struct Install: SwiftlyCommand {
             assumeYes: self.root.assumeYes
         )
 
-        if pathChanged {
+        let shell = if let s = ProcessInfo.processInfo.environment["SHELL"] {
+            s
+        } else {
+            try await Swiftly.currentPlatform.getShell()
+        }
+
+        // Fish doesn't cache its path, so this instruction is not necessary.
+        if pathChanged && !shell.hasSuffix("fish") {
             SwiftlyCore.print("""
             NOTE: We have updated some elements in your path and your shell may not yet be
             aware of the changes. You can run this command to update your shell.
@@ -271,7 +278,9 @@ struct Install: SwiftlyCommand {
                 }
             }
 
-            SwiftlyCore.print("Setting up toolchain proxies...")
+            if verbose {
+                SwiftlyCore.print("Setting up toolchain proxies...")
+            }
 
             let proxiesToCreate = Set(toolchainBinDirContents).subtracting(swiftlyBinDirContents).union(overwrite)