Skip to content

Commit

Permalink
Rollup merge of #130336 - onur-ozkan:simple-improvement, r=albertlars…
Browse files Browse the repository at this point in the history
…an68

simplify `Build::update_existing_submodule`

`Build::update_existing_submodule` is already doing the same thing..
  • Loading branch information
workingjubilee authored Sep 16, 2024
2 parents 68758c0 + d9e560c commit 9025fe9
Showing 1 changed file with 1 addition and 5 deletions.
6 changes: 1 addition & 5 deletions src/bootstrap/src/lib.rs
Original file line number Diff line number Diff line change
Expand Up @@ -552,11 +552,7 @@ impl Build {
// Look for `submodule.$name.path = $path`
// Sample output: `submodule.src/rust-installer.path src/tools/rust-installer`
let submodule = line.split_once(' ').unwrap().1;
let path = Path::new(submodule);
// Don't update the submodule unless it's already been cloned.
if GitInfo::new(false, path).is_managed_git_subrepository() {
self.config.update_submodule(submodule);
}
self.update_existing_submodule(submodule);
}
}

Expand Down

0 comments on commit 9025fe9

Please sign in to comment.