mirror of
https://github.com/nushell/nushell
synced 2024-12-27 21:43:09 +00:00
Prevent mv panic again (#6171)
Closes #6170 Signed-off-by: nibon7 <nibon7@163.com>
This commit is contained in:
parent
6b2c7a4c86
commit
cf2e9cf481
1 changed files with 6 additions and 7 deletions
|
@ -119,7 +119,7 @@ impl Command for Mv {
|
|||
return Err(ShellError::GenericError(
|
||||
format!(
|
||||
"Not possible to move {:?} to itself",
|
||||
filename.file_name().expect("Invalid file name")
|
||||
filename.file_name().unwrap_or(filename.as_os_str())
|
||||
),
|
||||
"cannot move to itself".into(),
|
||||
Some(spanned_destination.span),
|
||||
|
@ -155,18 +155,17 @@ impl Command for Mv {
|
|||
if let Err(error) = result {
|
||||
Some(Value::Error { error })
|
||||
} else if verbose {
|
||||
let val = if result.expect("Error value when unwrapping mv result") {
|
||||
format!(
|
||||
let val = match result {
|
||||
Ok(true) => format!(
|
||||
"moved {:} to {:}",
|
||||
entry.to_string_lossy(),
|
||||
destination.to_string_lossy()
|
||||
)
|
||||
} else {
|
||||
format!(
|
||||
),
|
||||
_ => format!(
|
||||
"{:} not moved to {:}",
|
||||
entry.to_string_lossy(),
|
||||
destination.to_string_lossy()
|
||||
)
|
||||
),
|
||||
};
|
||||
Some(Value::String { val, span })
|
||||
} else {
|
||||
|
|
Loading…
Reference in a new issue