Merge pull request #8412 from julian-klode/fix-install-no-target-existing-target

install: Fix --no-target-directory with existing file

Author: sylvestre

src/uu/install/src/install.rs

@@ -607,7 +607,7 @@ fn standard(mut paths: Vec<String>, b: &Behavior) -> UResult<()> {
             return Err(InstallError::OmittingDirectory(source.clone()).into());
         }
 
-        if b.no_target_dir && target.exists() {
+        if b.no_target_dir && target.is_dir() {
             return Err(
                 InstallError::OverrideDirectoryFailed(target.clone(), source.clone()).into(),
             );

tests/by-util/test_install.rs

@@ -2082,6 +2082,20 @@ fn test_install_no_target_directory_failing_cannot_overwrite() {
     assert!(!at.dir_exists("dir/file"));
 }
 
+#[test]
+fn test_install_no_target_directory_overwrite_file() {
+    let scene = TestScenario::new(util_name!());
+    let at = &scene.fixtures;
+    let file = "file";
+    let dest = "dest";
+
+    at.touch(file);
+    scene.ucmd().arg("-T").arg(file).arg(dest).succeeds();
+    scene.ucmd().arg("-T").arg(file).arg(dest).succeeds();
+
+    assert!(!at.dir_exists("dir/file"));
+}
+
 #[test]
 fn test_install_no_target_directory_failing_omitting_directory() {
     let scene = TestScenario::new(util_name!());