diff options
| author | Bernardo Meurer Costa <bemeurer@amazon.com> | 2024-05-15 20:56:50 +0000 |
|---|---|---|
| committer | Bernardo Meurer Costa <bemeurer@amazon.com> | 2024-05-20 15:01:18 +0000 |
| commit | 4550fb52d172101115b5c31a41f7d9b171d6c10c (patch) | |
| tree | 873e8cbd553f24ebdd8f827be83a209a49b4c588 | |
| parent | d9a7868ceb960dba9c33c7801bfa1a9996ffe775 (diff) | |
| download | rust-4550fb52d172101115b5c31a41f7d9b171d6c10c.tar.gz rust-4550fb52d172101115b5c31a41f7d9b171d6c10c.zip | |
chore(bootstrap/core/build_steps/dist): add fixme for duplicated vendor logic
| -rw-r--r-- | src/bootstrap/src/core/build_steps/dist.rs | 3 |
1 files changed, 3 insertions, 0 deletions
diff --git a/src/bootstrap/src/core/build_steps/dist.rs b/src/bootstrap/src/core/build_steps/dist.rs index 1f006e1453f..2feb59d3aad 100644 --- a/src/bootstrap/src/core/build_steps/dist.rs +++ b/src/bootstrap/src/core/build_steps/dist.rs @@ -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)); |