fix(vendor)!: vendor files with .rej/.orig suffix

This is meant to fixes #13191

As git sources and registry sources are considered immutable.
I don't think there is any reason excluding those files.
There might be a little chance local Git repositories might have those,
though that is a rare use case.

Alternatively,
we could reject all `.rej`/`.orig` files but `Cargo.toml.orig`.

Author: weihanglo

src/cargo/ops/vendor.rs

@@ -618,10 +618,6 @@ fn vendor_this(relative: &Path) -> bool {
         // Temporary Cargo files
         Some(".cargo-ok") => false,
 
-        // Skip patch-style orig/rej files. Published crates on crates.io
-        // have `Cargo.toml.orig` which we don't want to use here and
-        // otherwise these are rarely used as part of the build process.
-        Some(p) if p.ends_with(".orig") || p.ends_with(".rej") => false,
         _ => true,
     }
 }

tests/testsuite/vendor.rs

@@ -1087,13 +1087,14 @@ fn ignore_files() {
     Package::new("url", "1.4.1")
         // These will be vendored
         .file(".cargo_vcs_info.json", "")
+        .file("Cargo.toml.orig", "")
+        .file("foo.orig", "")
+        .file("foo.rej", "")
         .file("src/lib.rs", "")
         // These will not be vendored
         .file(".cargo-ok", "")
         .file(".gitattributes", "")
         .file(".gitignore", "")
-        .file("foo.orig", "")
-        .file("foo.rej", "")
         .publish();
 
     p.cargo("vendor --respect-source-config").run();
@@ -1105,6 +1106,9 @@ fn ignore_files() {
   "files": {
     ".cargo_vcs_info.json": "[..]",
     "Cargo.toml": "[..]",
+    "Cargo.toml.orig": "[..]",
+    "foo.orig": "[..]",
+    "foo.rej": "[..]",
     "src/lib.rs": "[..]"
   },
   "package": "[..]"