notifications: log directly when setting profile

Author: djc

src/cli/self_update.rs

@@ -1427,7 +1427,7 @@ mod tests {
             );
             assert_eq!(
                 for_host!(
-                    r"info: profile set to 'default'
+                    r"info: profile set to default
 info: default host triple is {0}
 "
                 ),

src/config.rs

@@ -359,7 +359,7 @@ impl<'a> Cfg<'a> {
             s.profile = Some(profile);
             Ok(())
         })?;
-        (self.notify_handler)(Notification::SetProfile(profile.as_str()));
+        info!("profile set to {}", profile.as_str());
         Ok(())
     }
 

src/notifications.rs

@@ -26,7 +26,6 @@ pub(crate) enum Notification<'a> {
     /// member, but the notification callback is already narrowed to
     /// utils::notifications by the time tar unpacking is called.
     SetDefaultBufferSize(usize),
-    SetProfile(&'a str),
     SetSelfUpdate(&'a str),
     LookingForToolchain(&'a ToolchainDesc),
     ToolchainDirectory(&'a Path),
@@ -67,8 +66,7 @@ impl Notification<'_> {
             | ReadMetadataVersion(_)
             | InstalledToolchain(_)
             | UpdateHashMatches => NotificationLevel::Debug,
-            SetProfile(_)
-            | SetSelfUpdate(_)
+            SetSelfUpdate(_)
             | UsingExistingToolchain(_)
             | UninstallingToolchain(_)
             | UninstalledToolchain(_)
@@ -102,7 +100,6 @@ impl Display for Notification<'_> {
             DownloadDataReceived(data, _) => write!(f, "received some data of size {}", data.len()),
             DownloadFinished(_) => write!(f, "download finished"),
             DownloadFailed(_) => write!(f, "download failed"),
-            SetProfile(name) => write!(f, "profile set to '{name}'"),
             SetSelfUpdate(mode) => write!(f, "auto-self-update mode set to '{mode}'"),
             LookingForToolchain(name) => write!(f, "looking for installed toolchain '{name}'"),
             ToolchainDirectory(path) => write!(f, "toolchain directory: '{}'", path.display()),

tests/suite/cli_misc.rs

@@ -1520,7 +1520,7 @@ error: override toolchain 'foo' is not installed: the +toolchain on the command
         .expect(["rustup", "+stable", "set", "profile", "minimal"])
         .await
         .with_stderr(snapbox::str![[r#"
-info: profile set to 'minimal'
+info: profile set to minimal
 
 "#]])
         .is_ok();