From 2c17f719657c46ea37e4846289e951c83409a514 Mon Sep 17 00:00:00 2001 From: Ilya Gladyshev Date: Tue, 6 Aug 2024 18:02:13 +0100 Subject: [PATCH] fix(cli): don't prompt to rebuild if --rebuild is given When running `doom sync --rebuild` after emacs upgrade the user is prompted with "Installed packages must be rebuilt. Do so now?", even though doom was explicitly instructed to rebuild. This PR removes this prompt when the `--rebuild` option is provided --- lisp/cli/sync.el | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/lisp/cli/sync.el b/lisp/cli/sync.el index e1a7e2abb..ee18ccb54 100644 --- a/lisp/cli/sync.el +++ b/lisp/cli/sync.el @@ -73,7 +73,7 @@ OPTIONS: (when (and old-host (not (equal old-host (system-name)))) (print! (warn "Your system has changed since last sync")) (setq to-rebuild t)) - (when (and to-rebuild (not (doom-cli-context-suppress-prompts-p context))) + (when (and to-rebuild (not rebuild?) (not (doom-cli-context-suppress-prompts-p context))) (cond (nobuild? (print! (warn "Packages must be rebuilt, but -B has prevented it. Skipping..."))) ((doom-cli-context-get context 'upgrading)