Rollup merge of rust-lang#96196 - jyn514:no-assertion, r=Mark-Simulacrum

Remove assertion that all paths in `ShouldRun` exist

This breaks on submodules (see rust-lang#96188). Disable the assertion for now until I can think of a proper
fix.

This doesn't revert any of the changes in `Step`s themselves, only what
`ShouldRun::paths` does.

src/bootstrap/builder.rs

@@ -388,11 +388,13 @@ impl<'a> ShouldRun<'a> {
  paths
  .iter()
  .map(|p| {
- assert!(
- self.builder.src.join(p).exists(),
- "`should_run.paths` should correspond to real on-disk paths - use `alias` if there is no relevant path: {}",
- p
- );
+ // FIXME(#96188): make sure this is actually a path.
+ // This currently breaks for paths within submodules.
+ //assert!(
+ // self.builder.src.join(p).exists(),
+ // "`should_run.paths` should correspond to real on-disk paths - use `alias` if there is no relevant path: {}",
+ // p
+ //);
  TaskPath { path: p.into(), kind: Some(self.kind) }
  })
  .collect(),