diff options
author | Matan Kushner <hello@matchai.me> | 2019-09-25 17:13:58 +0900 |
---|---|---|
committer | GitHub <noreply@github.com> | 2019-09-25 17:13:58 +0900 |
commit | caaf3bc6a951022bc5a7ba4ce4fe4f5586fb33b1 (patch) | |
tree | 84b68cb57ee0ec7dbc1c446b3b9ef325368cab32 | |
parent | e6761e63ed4da3bb150c66acebe547f73bae897f (diff) |
revert: Revert the command duration in ms feature (#380) (#425)
Due to incompatibilities with the macOS date utility, we are
temporarily reverting this feature.
Refs: #c5e971a
-rw-r--r-- | docs/config/README.md | 14 | ||||
-rw-r--r-- | src/init/starship.bash | 8 | ||||
-rw-r--r-- | src/init/starship.fish | 4 | ||||
-rw-r--r-- | src/init/starship.zsh | 6 | ||||
-rw-r--r-- | src/modules/cmd_duration.rs | 38 | ||||
-rw-r--r-- | tests/testsuite/cmd_duration.rs | 16 |
6 files changed, 35 insertions, 51 deletions
diff --git a/docs/config/README.md b/docs/config/README.md index e6cb68be7..be0bce93a 100644 --- a/docs/config/README.md +++ b/docs/config/README.md @@ -212,7 +212,7 @@ use_symbol_for_status = true ## Command Duration The `cmd_duration` module shows how long the last command took to execute. -The module will be shown only if the command took longer than 2000 milliseconds (2 seconds), or +The module will be shown only if the command took longer than two seconds, or the `min_time` config value, if it exists. ::: warning Do not hook the DEBUG trap in Bash @@ -227,11 +227,11 @@ running `eval $(starship init $0)`, and then proceed as normal. ### Options -| Variable | Default | Description | -| ---------- | --------------- | ----------------------------------------------------- | -| `min_time` | `2000` | Shortest duration to show time for (in milliseconds). | -| `style` | `"bold yellow"` | The style for the module. | -| `disabled` | `false` | Disables the `cmd_duration` module. | +| Variable | Default | Description | +| ---------- | --------------- | ----------------------------------- | +| `min_time` | `2` | Shortest duration to show time for. | +| `style` | `"bold yellow"` | The style for the module. | +| `disabled` | `false` | Disables the `cmd_duration` module. | ### Example @@ -239,7 +239,7 @@ running `eval $(starship init $0)`, and then proceed as normal. # ~/.config/starship.toml [cmd_duration] -min_time = 4000 +min_time = 4 ``` ## Directory diff --git a/src/init/starship.bash b/src/init/starship.bash index bc1bf9afe..da02f6e42 100644 --- a/src/init/starship.bash +++ b/src/init/starship.bash @@ -17,7 +17,7 @@ starship_preexec() { # Avoid restarting the timer for commands in the same pipeline if [ "$PREEXEC_READY" = "true" ]; then PREEXEC_READY=false - STARSHIP_START_TIME=$(date +%s%0N) + STARSHIP_START_TIME=$(date +%s) fi } @@ -31,7 +31,7 @@ starship_precmd() { # Prepare the timer data, if needed. if [[ $STARSHIP_START_TIME ]]; then - STARSHIP_END_TIME=$(date +%s%0N) + STARSHIP_END_TIME=$(date +%s) STARSHIP_DURATION=$((STARSHIP_END_TIME - STARSHIP_START_TIME)) PS1="$(::STARSHIP:: prompt --status=$STATUS --jobs="$(jobs -p | wc -l)" --cmd-duration=$STARSHIP_DURATION)" unset STARSHIP_START_TIME @@ -65,5 +65,5 @@ else fi # Set up the start time and STARSHIP_SHELL, which controls shell-specific sequences -STARSHIP_START_TIME=$(date +%s%0N) -export STARSHIP_SHELL="bash" +STARSHIP_START_TIME=$(date +%s) +export STARSHIP_SHELL="bash"
\ No newline at end of file diff --git a/src/init/starship.fish b/src/init/starship.fish index f9409f5b5..b2e88e5ee 100644 --- a/src/init/starship.fish +++ b/src/init/starship.fish @@ -8,8 +8,8 @@ function fish_prompt set -l exit_code $status # Account for changes in variable name between v2.7 and v3.0 set -l CMD_DURATION "$CMD_DURATION$cmd_duration" - set -l starship_duration_ns {$CMD_DURATION}000000 - ::STARSHIP:: prompt --status=$exit_code --keymap=$keymap --cmd-duration=$starship_duration_ns --jobs=(count (jobs -p)) + set -l starship_duration (math --scale=0 "$CMD_DURATION / 1000") + ::STARSHIP:: prompt --status=$exit_code --keymap=$keymap --cmd-duration=$starship_duration --jobs=(count (jobs -p)) end function fish_mode_prompt; end export STARSHIP_SHELL="fish" diff --git a/src/init/starship.zsh b/src/init/starship.zsh index 7b638ccbb..fdae70e12 100644 --- a/src/init/starship.zsh +++ b/src/init/starship.zsh @@ -19,7 +19,7 @@ starship_precmd() { NUM_JOBS=$#jobstates # Compute cmd_duration, if we have a time to consume if [[ ! -z "${STARSHIP_START_TIME+1}" ]]; then - STARSHIP_END_TIME="$(date +%s%0N)" + STARSHIP_END_TIME="$(date +%s)" STARSHIP_DURATION=$((STARSHIP_END_TIME - STARSHIP_START_TIME)) PROMPT="$(::STARSHIP:: prompt --status=$STATUS --cmd-duration=$STARSHIP_DURATION --jobs="$NUM_JOBS")" unset STARSHIP_START_TIME @@ -28,7 +28,7 @@ starship_precmd() { fi } starship_preexec(){ - STARSHIP_START_TIME="$(date +%s%0N)" + STARSHIP_START_TIME="$(date +%s)" } # If precmd/preexec arrays are not already set, set them. If we don't do this, @@ -53,6 +53,6 @@ function zle-keymap-select zle reset-prompt } -STARSHIP_START_TIME="$(date +%s%0N)" +STARSHIP_START_TIME="$(date +%s)" zle -N zle-keymap-select export STARSHIP_SHELL="zsh" diff --git a/src/modules/cmd_duration.rs b/src/modules/cmd_duration.rs index fcbce5626..35a7e6858 100644 --- a/src/modules/cmd_duration.rs +++ b/src/modules/cmd_duration.rs @@ -14,10 +14,9 @@ pub fn module<'a>(context: &'a Context) -> Option<Module<'a>> { .value_of("cmd_duration") .unwrap_or("invalid_time") .parse::<u64>() - .ok()? - / 1_000_000; + .ok()?; - let signed_config_min = module.config_value_i64("min_time").unwrap_or(2000); + let signed_config_min = module.config_value_i64("min_time").unwrap_or(2); /* TODO: Once error handling is implemented, warn the user if their config min time is nonsensical */ @@ -32,39 +31,28 @@ pub fn module<'a>(context: &'a Context) -> Option<Module<'a>> { let config_min = signed_config_min as u64; let module_color = match elapsed { - time if time < config_min => module - .config_value_style("style") - .unwrap_or_else(|| Color::RGB(80, 80, 80).bold()), + time if time < config_min => return None, _ => module .config_value_style("style") .unwrap_or_else(|| Color::Yellow.bold()), }; module.set_style(module_color); - module.new_segment( - "cmd_duration", - &format!("took {}", render_time(elapsed, config_min)), - ); + module.new_segment("cmd_duration", &format!("took {}", render_time(elapsed))); module.get_prefix().set_value(""); Some(module) } // Render the time into a nice human-readable string -fn render_time(raw_milliseconds: u64, config_min: u64) -> String { +fn render_time(raw_seconds: u64) -> String { // Calculate a simple breakdown into days/hours/minutes/seconds - let (mut milliseconds, raw_seconds) = (raw_milliseconds % 1000, raw_milliseconds / 1000); let (seconds, raw_minutes) = (raw_seconds % 60, raw_seconds / 60); let (minutes, raw_hours) = (raw_minutes % 60, raw_minutes / 60); let (hours, days) = (raw_hours % 24, raw_hours / 24); - // Do not display milliseconds if command duration is less than config_min - if raw_milliseconds > config_min { - milliseconds = 0; - } - - let components = [days, hours, minutes, seconds, milliseconds]; - let suffixes = ["d", "h", "m", "s", "ms"]; + let components = [days, hours, minutes, seconds]; + let suffixes = ["d", "h", "m", "s"]; let rendered_components: Vec<String> = components .iter() @@ -87,23 +75,19 @@ mod tests { use super::*; #[test] - fn test_100ms() { - assert_eq!(render_time(100 as u64, 2000), "100ms") - } - #[test] fn test_10s() { - assert_eq!(render_time(10 * 1000 as u64, 2000), "10s") + assert_eq!(render_time(10 as u64), "10s") } #[test] fn test_90s() { - assert_eq!(render_time(90 * 1_000 as u64, 2000), "1m30s") + assert_eq!(render_time(90 as u64), "1m30s") } #[test] fn test_10110s() { - assert_eq!(render_time(10110 * 1_000 as u64, 2000), "2h48m30s") + assert_eq!(render_time(10110 as u64), "2h48m30s") } #[test] fn test_1d() { - assert_eq!(render_time(86400 * 1_000 as u64, 2000), "1d") + assert_eq!(render_time(86400 as u64), "1d") } } diff --git a/tests/testsuite/cmd_duration.rs b/tests/testsuite/cmd_duration.rs index 18e4c7c6c..287944cbd 100644 --- a/tests/testsuite/cmd_duration.rs +++ b/tests/testsuite/cmd_duration.rs @@ -6,11 +6,11 @@ use crate::common::{self, TestCommand}; #[test] fn config_blank_duration_1s() -> io::Result<()> { let output = common::render_module("cmd_duration") - .arg("--cmd-duration=1000000000") + .arg("--cmd-duration=1") .output()?; let actual = String::from_utf8(output.stdout).unwrap(); - let expected = format!("{} ", Color::RGB(80, 80, 80).bold().paint("took 1s")); + let expected = ""; assert_eq!(expected, actual); Ok(()) } @@ -18,7 +18,7 @@ fn config_blank_duration_1s() -> io::Result<()> { #[test] fn config_blank_duration_5s() -> io::Result<()> { let output = common::render_module("cmd_duration") - .arg("--cmd-duration=5000000000") + .arg("--cmd-duration=5") .output()?; let actual = String::from_utf8(output.stdout).unwrap(); @@ -32,13 +32,13 @@ fn config_5s_duration_3s() -> io::Result<()> { let output = common::render_module("cmd_duration") .use_config(toml::toml! { [cmd_duration] - min_time = 5000 + min_time = 5 }) - .arg("--cmd-duration=3000000000") + .arg("--cmd-duration=3") .output()?; let actual = String::from_utf8(output.stdout).unwrap(); - let expected = format!("{} ", Color::RGB(80, 80, 80).bold().paint("took 3s")); + let expected = ""; assert_eq!(expected, actual); Ok(()) } @@ -48,9 +48,9 @@ fn config_5s_duration_10s() -> io::Result<()> { let output = common::render_module("cmd_duration") .use_config(toml::toml! { [cmd_duration] - min_time = 5000 + min_time = 5 }) - .arg("--cmd-duration=10000000000") + .arg("--cmd-duration=10") .output()?; let actual = String::from_utf8(output.stdout).unwrap(); |