artemist/lix-installer
0
0
Fork 0
forked from lix-project/lix-installer
Code Pull requests Activity

Use sudo and don't try to prompt on no-confirm

Browse source
This commit is contained in:
Ana Hobden 2022-11-08 12:21:02 -08:00
parent d900830f04
commit 47ae6398f8
2 changed files with 8 additions and 6 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

8
.github/workflows/ci.yml vendored
View file

@ -135,13 +135,13 @@ jobs:
with:
name: harmonic-x86_64-darwin
- name: Initial install
run: ./harmonic install darwin-multi --no-confirm
run: sudo ./harmonic install darwin-multi --no-confirm
- name: Test run
run: nix run nixpkgs#fortune
- name: Initial uninstall
run: ./harmonic uninstall --no-confirm
run: sudo ./harmonic uninstall --no-confirm
- name: Repeated install
run: ./harmonic install darwin-multi --no-confirm
run: sudo ./harmonic install darwin-multi --no-confirm
- name: Repeated uninstall
run: ./harmonic uninstall --no-confirm
run: sudo ./harmonic uninstall --no-confirm

6
src/cli/subcommand/install.rs
View file

@ -61,8 +61,10 @@ impl CommandExecute for Install {
if let Err(err) = plan.install().await {
tracing::error!("{:?}", eyre!(err));
if !interaction::confirm(plan.describe_revert(explain)).await? {
interaction::clean_exit_with_message("Okay, didn't do anything! Bye!").await;
if !no_confirm {
if !interaction::confirm(plan.describe_revert(explain)).await? {
interaction::clean_exit_with_message("Okay, didn't do anything! Bye!").await;
}
}
plan.revert().await?
}