Skip to content

Commit c71569a

Browse files
committed
Revert "Rollup merge of rust-lang#137593 - RalfJung:subtree-sync-download-llvm, r=Mark-Simulacrum"
Looks like unfortunately the `--diff-merges` flag is a `git show`-only command, not `git rev-list`. This reverts commit 95994f9, reversing changes made to 7290b04.
1 parent ae8ab87 commit c71569a

File tree

1 file changed

+1
-9
lines changed

1 file changed

+1
-9
lines changed

Diff for: src/build_helper/src/git.rs

+1-9
Original file line numberDiff line numberDiff line change
@@ -140,7 +140,6 @@ pub fn get_closest_merge_commit(
140140
// cd \"/checkout\" && \"git\" \"merge-base\" \"origin/master\" \"HEAD\"\nexpected success, got: exit status: 1\n"
141141
// ```
142142
// Investigate and resolve this issue instead of skipping it like this.
143-
// NOTE (2025-03): this is probably caused by CI using a sparse checkout.
144143
(channel == "nightly" || !CiEnv::is_rust_lang_managed_ci_job())
145144
{
146145
git_upstream_merge_base(config, git_dir).unwrap()
@@ -151,18 +150,11 @@ pub fn get_closest_merge_commit(
151150
}
152151
};
153152

154-
// Now that rust-lang/rust is the only repo using bors, we can search the entire
155-
// history for a bors commit, not just "first parents". This is crucial to make
156-
// this logic work when the user has currently checked out a subtree sync branch.
157-
// At the same time, we use this logic in CI where only a tiny part of the history
158-
// is even checked out, making this kind of history search very fragile. It turns
159-
// out that by adding `--diff-merges=first-parent`, we get a usable reply
160-
// even for sparse checkouts: it will just return the most recent bors commit.
161153
git.args([
162154
"rev-list",
163155
&format!("--author={}", config.git_merge_commit_email),
164156
"-n1",
165-
"--diff-merges=first-parent",
157+
"--first-parent",
166158
&merge_base,
167159
]);
168160

0 commit comments

Comments
 (0)