From e431a6bb63072c18aa1d8ae32d6df12368353606 Mon Sep 17 00:00:00 2001 From: Nelson Osacky Date: Thu, 26 Mar 2026 15:22:17 +0100 Subject: [PATCH] fix(build): Ungate ProgressBarMode::Response from managed feature The `Response` variant was gated behind `#[cfg(not(feature = "managed"))]` because its only consumer (`download_with_progress`) was also managed-gated. The new `download_installable_build` method (used by `build download`) references this variant without a managed gate, breaking the Docker build which compiles with `--features managed`. The progress bar has nothing to do with managed mode, so remove the feature gate from the variant and simplify the `response()` method. Co-Authored-By: Claude Opus 4.6 (1M context) --- src/utils/progress.rs | 9 +-------- 1 file changed, 1 insertion(+), 8 deletions(-) diff --git a/src/utils/progress.rs b/src/utils/progress.rs index 3efd79b3f9..17fb3b0aa3 100644 --- a/src/utils/progress.rs +++ b/src/utils/progress.rs @@ -128,7 +128,6 @@ impl ProgressBar { #[derive(Clone)] pub enum ProgressBarMode { Disabled, - #[cfg(not(feature = "managed"))] Response, Shared((Arc, u64, usize, Arc>>)), } @@ -141,12 +140,6 @@ impl ProgressBarMode { /// Returns whether a progress bar should be displayed during download. pub fn response(&self) -> bool { - #[cfg(not(feature = "managed"))] - let rv = matches!(*self, ProgressBarMode::Response); - - #[cfg(feature = "managed")] - let rv = false; - - rv + matches!(*self, ProgressBarMode::Response) } }