remove check for destination is link

Author: cre4ture

src/uu/install/src/install.rs

@@ -748,12 +748,17 @@ fn perform_backup(to: &Path, b: &Behavior) -> UResult<Option<PathBuf>> {
 /// Returns an empty Result or an error in case of failure.
 ///
 fn copy_file(from: &Path, to: &Path) -> UResult<()> {
-    let dest_meta_res = fs::symlink_metadata(to);
-    if let Ok(dest_meta) = dest_meta_res {
-        if dest_meta.is_symlink() {
-            // fs::copy fails if destination is a invalid symlink
-            // so lets just remove all symlinks at destination before copy
-            fs::remove_file(to)?;
+    // fs::copy fails if destination is a invalid symlink.
+    // so lets just remove all existing files at destination before copy.
+    match fs::remove_file(to) {
+        Ok(()) => {}
+        Err(e) if e.kind() == std::io::ErrorKind::NotFound => { /* expected */ }
+        Err(e) => {
+            show_error!(
+                "Failed to remove existing file {}. Error: {:?}",
+                to.display(),
+                e
+            );
         }
     }