From 1493343240ce32f57b0f47b9b5783bfd932f8732 Mon Sep 17 00:00:00 2001
From: Hofer-Julian <30049909+Hofer-Julian@users.noreply.github.com>
Date: Wed, 16 Oct 2024 17:06:45 +0200
Subject: [PATCH] fix: `pixi global upgrade-all` error message (#2296)

Fixes #2293
---
 src/cli/global/upgrade.rs     |  3 ++-
 src/cli/global/upgrade_all.rs | 25 +++++++++----------------
 2 files changed, 11 insertions(+), 17 deletions(-)

diff --git a/src/cli/global/upgrade.rs b/src/cli/global/upgrade.rs
index 91ed8f31d..004d28b00 100644
--- a/src/cli/global/upgrade.rs
+++ b/src/cli/global/upgrade.rs
@@ -4,6 +4,7 @@ use rattler_conda_types::Platform;
 use crate::cli::{cli_config::ChannelsConfig, has_specs::HasSpecs};
 
 /// Upgrade specific package which is installed globally.
+/// This command has been removed, please use `pixi global update` instead
 #[derive(Parser, Debug)]
 // TODO: Uncomment as soon we implement this
 //#[clap(arg_required_else_help = true)]
@@ -29,7 +30,7 @@ impl HasSpecs for Args {
 
 pub async fn execute(_args: Args) -> miette::Result<()> {
     Err(
-        miette::miette!("You can use `pixi global update` for most use cases").wrap_err(
+        miette::miette!("You can call `pixi global update` for most use cases").wrap_err(
             "`pixi global upgrade` has been removed, and will be re-added in future releases",
         ),
     )
diff --git a/src/cli/global/upgrade_all.rs b/src/cli/global/upgrade_all.rs
index bab7c772f..14dae5927 100644
--- a/src/cli/global/upgrade_all.rs
+++ b/src/cli/global/upgrade_all.rs
@@ -1,34 +1,27 @@
 use clap::Parser;
+use pixi_config::ConfigCli;
 use rattler_conda_types::Platform;
 
-use crate::cli::{cli_config::ChannelsConfig, has_specs::HasSpecs};
+use crate::cli::cli_config::ChannelsConfig;
 
-/// Upgrade specific package which is installed globally.
+/// Upgrade all globally installed packages
+/// This command has been removed, please use `pixi global update` instead
 #[derive(Parser, Debug)]
-#[clap(arg_required_else_help = true)]
 pub struct Args {
-    /// Specifies the packages to upgrade.
-    //#[arg(required = true)]
-    pub specs: Vec<String>,
-
     #[clap(flatten)]
     channels: ChannelsConfig,
 
+    #[clap(flatten)]
+    config: ConfigCli,
+
     /// The platform to install the package for.
     #[clap(long, default_value_t = Platform::current())]
     platform: Platform,
 }
 
-impl HasSpecs for Args {
-    fn packages(&self) -> Vec<&str> {
-        self.specs.iter().map(AsRef::as_ref).collect()
-    }
-}
-
 pub async fn execute(_args: Args) -> miette::Result<()> {
     Err(
-        miette::miette!("You can use `pixi global update` for most use cases").wrap_err(
-            "`pixi global upgrade-all` has been removed, and will be re-added in future releases",
-        ),
+        miette::miette!("You can call `pixi global update` for most use cases")
+            .wrap_err("`pixi global upgrade-all` has been removed"),
     )
 }