Skip to content

Commit

Permalink
chore(bootstrap/core/build_steps/dist): add fixme for duplicated vend…
Browse files Browse the repository at this point in the history
…or logic
  • Loading branch information
lovesegfault committed May 15, 2024
1 parent aa63d94 commit 253d358
Showing 1 changed file with 3 additions and 0 deletions.
3 changes: 3 additions & 0 deletions src/bootstrap/src/core/build_steps/dist.rs
Original file line number Diff line number Diff line change
Expand Up @@ -1010,6 +1010,9 @@ impl Step for PlainSourceTarball {
if builder.rust_info().is_managed_git_subrepository()
|| builder.rust_info().is_from_tarball()
{
// FIXME: This code looks _very_ similar to what we have in `src/core/build_steps/vendor.rs`
// perhaps it should be removed in favor of making `dist` perform the `vendor` step?

// Ensure we have all submodules from src and other directories checked out.
for submodule in builder.get_all_submodules() {
builder.update_submodule(Path::new(submodule));
Expand Down

0 comments on commit 253d358

Please sign in to comment.