From 9483c11ee035e44fda053d5562c026c080c19c8e Mon Sep 17 00:00:00 2001 From: Robbert van der Helm Date: Fri, 26 Feb 2021 14:11:46 +0100 Subject: [PATCH] [yabridgectl] Reword not yet installed status The red `not installed` was causing some confusion. --- CHANGELOG.md | 5 +++++ tools/yabridgectl/src/actions.rs | 4 ++-- 2 files changed, 7 insertions(+), 2 deletions(-) diff --git a/CHANGELOG.md b/CHANGELOG.md index 728a55ab..0bc6f0b3 100644 --- a/CHANGELOG.md +++ b/CHANGELOG.md @@ -25,6 +25,11 @@ Versioning](https://semver.org/spec/v2.0.0.html). - Fixed a regression from yabridge 3.0.0 where log output would no longer include timestamps. +### yabridgectl + +- Changed the wording in `yabridgectl status` for plugins that have not been + setup yet to look less dramatic and hopefully cause less confusion. + ## [3.0.0] - 2021-02-14 ### Added diff --git a/tools/yabridgectl/src/actions.rs b/tools/yabridgectl/src/actions.rs index fca3e67b..b2af3090 100644 --- a/tools/yabridgectl/src/actions.rs +++ b/tools/yabridgectl/src/actions.rs @@ -114,8 +114,8 @@ pub fn show_status(config: &Config) -> Result<()> { println!("Could not find yabridge's files files: {}\n", err); } } - println!("installation method: {}", config.method); + for (path, search_results) in results { println!("\n{}:", path.display()); @@ -124,7 +124,7 @@ pub fn show_status(config: &Config) -> Result<()> { Some(NativeFile::Regular(_)) => "copy".green(), Some(NativeFile::Symlink(_)) => "symlink".green(), Some(NativeFile::Directory(_)) => "invalid".red(), - None => "not installed".red(), + None => "not yet installed".into(), }; println!(" {} :: {}", plugin.display(), status_str);