From 741e3c3d8f7fa1b8f2a6360c0796ac5c808899d0 Mon Sep 17 00:00:00 2001
From: Ian Manske <ian.manske@pm.me>
Date: Fri, 12 Apr 2024 15:44:27 +0000
Subject: [PATCH] Return value instead of stream from `kill` (#12480)

# Description
The `kill` command returns a stream with a single value. This PR changes
it to simply return the value.

# User-Facing Changes
Technically a breaking change.
---
 crates/nu-command/src/platform/kill.rs | 28 ++++++++++++--------------
 1 file changed, 13 insertions(+), 15 deletions(-)

diff --git a/crates/nu-command/src/platform/kill.rs b/crates/nu-command/src/platform/kill.rs
index 012a6bf133..59486cf1ad 100644
--- a/crates/nu-command/src/platform/kill.rs
+++ b/crates/nu-command/src/platform/kill.rs
@@ -147,23 +147,21 @@ impl Command for Kill {
             });
         }
 
-        let val = String::from(
-            String::from_utf8(output.stdout)
-                .map_err(|e| ShellError::GenericError {
-                    error: "failed to convert output to string".into(),
-                    msg: e.to_string(),
-                    span: Some(call.head),
-                    help: None,
-                    inner: vec![],
-                })?
-                .trim_end(),
-        );
-        if val.is_empty() {
+        let mut output =
+            String::from_utf8(output.stdout).map_err(|e| ShellError::GenericError {
+                error: "failed to convert output to string".into(),
+                msg: e.to_string(),
+                span: Some(call.head),
+                help: None,
+                inner: vec![],
+            })?;
+
+        output.truncate(output.trim_end().len());
+
+        if output.is_empty() {
             Ok(Value::nothing(call.head).into_pipeline_data())
         } else {
-            Ok(vec![Value::string(val, call.head)]
-                .into_iter()
-                .into_pipeline_data(engine_state.ctrlc.clone()))
+            Ok(Value::string(output, call.head).into_pipeline_data())
         }
     }