From 80c4bd5065768a29ef8e8bde049396f3d5609ecd Mon Sep 17 00:00:00 2001 From: PandaMimo <116785@kwc-culemborg.nl> Date: Mon, 10 Nov 2025 13:44:17 +0100 Subject: [PATCH] ref: comment run_config_update (#1448) Co-authored-by: Milo Co-authored-by: Gideon <87426140+GideonBear@users.noreply.github.com> --- src/steps/os/linux.rs | 2 ++ 1 file changed, 2 insertions(+) diff --git a/src/steps/os/linux.rs b/src/steps/os/linux.rs index c89fa5be..c5fefe51 100644 --- a/src/steps/os/linux.rs +++ b/src/steps/os/linux.rs @@ -1030,6 +1030,7 @@ pub fn run_dkp_pacman_update(ctx: &ExecutionContext) -> Result<()> { } pub fn run_config_update(ctx: &ExecutionContext) -> Result<()> { + // The `config_update` step always requests user input, so when running with `--yes` we need to skip the step entirely if ctx.config().yes(Step::ConfigUpdate) { return Err(SkipStep(t!("Skipped in --yes").to_string()).into()); } @@ -1039,6 +1040,7 @@ pub fn run_config_update(ctx: &ExecutionContext) -> Result<()> { let sudo = ctx.require_sudo()?; sudo.execute(ctx, etc_update)?.status_checked()?; } else if let Ok(pacdiff) = require("pacdiff") { + // When `DIFFPROG` is unset, `pacdiff` uses `vim` by default if std::env::var("DIFFPROG").is_err() { require("vim")?; }