Skip to content

Commit 8353012

Browse files
committed
Prepare everything for distributing miri via rustup
1 parent 67ede4c commit 8353012

File tree

5 files changed

+108
-0
lines changed

5 files changed

+108
-0
lines changed

src/bootstrap/builder.rs

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -449,6 +449,7 @@ impl<'a> Builder<'a> {
449449
dist::Rls,
450450
dist::Rustfmt,
451451
dist::Clippy,
452+
dist::Miri,
452453
dist::LlvmTools,
453454
dist::Lldb,
454455
dist::Extended,
@@ -461,6 +462,7 @@ impl<'a> Builder<'a> {
461462
install::Rls,
462463
install::Rustfmt,
463464
install::Clippy,
465+
install::Miri,
464466
install::Analysis,
465467
install::Src,
466468
install::Rustc

src/bootstrap/dist.rs

Lines changed: 84 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1275,6 +1275,90 @@ impl Step for Clippy {
12751275
}
12761276
}
12771277

1278+
#[derive(Debug, PartialOrd, Ord, Copy, Clone, Hash, PartialEq, Eq)]
1279+
pub struct Miri {
1280+
pub stage: u32,
1281+
pub target: Interned<String>,
1282+
}
1283+
1284+
impl Step for Miri {
1285+
type Output = Option<PathBuf>;
1286+
const ONLY_HOSTS: bool = true;
1287+
1288+
fn should_run(run: ShouldRun) -> ShouldRun {
1289+
run.path("miri")
1290+
}
1291+
1292+
fn make_run(run: RunConfig) {
1293+
run.builder.ensure(Miri {
1294+
stage: run.builder.top_stage,
1295+
target: run.target,
1296+
});
1297+
}
1298+
1299+
fn run(self, builder: &Builder) -> Option<PathBuf> {
1300+
let stage = self.stage;
1301+
let target = self.target;
1302+
assert!(builder.config.extended);
1303+
1304+
builder.info(&format!("Dist miri stage{} ({})", stage, target));
1305+
let src = builder.src.join("src/tools/miri");
1306+
let release_num = builder.release_num("miri");
1307+
let name = pkgname(builder, "miri");
1308+
let version = builder.miri_info.version(builder, &release_num);
1309+
1310+
let tmp = tmpdir(builder);
1311+
let image = tmp.join("miri-image");
1312+
drop(fs::remove_dir_all(&image));
1313+
builder.create_dir(&image);
1314+
1315+
// Prepare the image directory
1316+
// We expect miri to build, because we've exited this step above if tool
1317+
// state for miri isn't testing.
1318+
let miri = builder.ensure(tool::Miri {
1319+
compiler: builder.compiler(stage, builder.config.build),
1320+
target, extra_features: Vec::new()
1321+
}).or_else(|| { missing_tool("miri", builder.build.config.missing_tools); None })?;
1322+
let cargomiri = builder.ensure(tool::CargoMiri {
1323+
compiler: builder.compiler(stage, builder.config.build),
1324+
target, extra_features: Vec::new()
1325+
}).or_else(|| { missing_tool("cargo miri", builder.build.config.missing_tools); None })?;
1326+
1327+
builder.install(&miri, &image.join("bin"), 0o755);
1328+
builder.install(&cargomiri, &image.join("bin"), 0o755);
1329+
let doc = image.join("share/doc/miri");
1330+
builder.install(&src.join("README.md"), &doc, 0o644);
1331+
builder.install(&src.join("LICENSE-APACHE"), &doc, 0o644);
1332+
builder.install(&src.join("LICENSE-MIT"), &doc, 0o644);
1333+
1334+
// Prepare the overlay
1335+
let overlay = tmp.join("miri-overlay");
1336+
drop(fs::remove_dir_all(&overlay));
1337+
t!(fs::create_dir_all(&overlay));
1338+
builder.install(&src.join("README.md"), &overlay, 0o644);
1339+
builder.install(&src.join("LICENSE-APACHE"), &doc, 0o644);
1340+
builder.install(&src.join("LICENSE-MIT"), &doc, 0o644);
1341+
builder.create(&overlay.join("version"), &version);
1342+
1343+
// Generate the installer tarball
1344+
let mut cmd = rust_installer(builder);
1345+
cmd.arg("generate")
1346+
.arg("--product-name=Rust")
1347+
.arg("--rel-manifest-dir=rustlib")
1348+
.arg("--success-message=miri-ready-to-serve.")
1349+
.arg("--image-dir").arg(&image)
1350+
.arg("--work-dir").arg(&tmpdir(builder))
1351+
.arg("--output-dir").arg(&distdir(builder))
1352+
.arg("--non-installed-overlay").arg(&overlay)
1353+
.arg(format!("--package-name={}-{}", name, target))
1354+
.arg("--legacy-manifest-dirs=rustlib,cargo")
1355+
.arg("--component-name=miri-preview");
1356+
1357+
builder.run(&mut cmd);
1358+
Some(distdir(builder).join(format!("{}-{}.tar.gz", name, target)))
1359+
}
1360+
}
1361+
12781362
#[derive(Debug, PartialOrd, Ord, Copy, Clone, Hash, PartialEq, Eq)]
12791363
pub struct Rustfmt {
12801364
pub stage: u32,

src/bootstrap/install.rs

Lines changed: 11 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -32,6 +32,9 @@ pub fn install_rls(builder: &Builder, stage: u32, host: Interned<String>) {
3232
pub fn install_clippy(builder: &Builder, stage: u32, host: Interned<String>) {
3333
install_sh(builder, "clippy", "clippy", stage, Some(host));
3434
}
35+
pub fn install_miri(builder: &Builder, stage: u32, host: Interned<String>) {
36+
install_sh(builder, "miri", "miri", stage, Some(host));
37+
}
3538

3639
pub fn install_rustfmt(builder: &Builder, stage: u32, host: Interned<String>) {
3740
install_sh(builder, "rustfmt", "rustfmt", stage, Some(host));
@@ -217,6 +220,14 @@ install!((self, builder, _config),
217220
builder.info(&format!("skipping Install clippy stage{} ({})", self.stage, self.target));
218221
}
219222
};
223+
Miri, "miri", Self::should_build(_config), only_hosts: true, {
224+
if builder.ensure(dist::Clippy { stage: self.stage, target: self.target }).is_some() ||
225+
Self::should_install(builder) {
226+
install_miri(builder, self.stage, self.target);
227+
} else {
228+
builder.info(&format!("skipping Install miri stage{} ({})", self.stage, self.target));
229+
}
230+
};
220231
Rustfmt, "rustfmt", Self::should_build(_config), only_hosts: true, {
221232
if builder.ensure(dist::Rustfmt { stage: self.stage, target: self.target }).is_some() ||
222233
Self::should_install(builder) {

src/bootstrap/lib.rs

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -253,6 +253,7 @@ pub struct Build {
253253
cargo_info: channel::GitInfo,
254254
rls_info: channel::GitInfo,
255255
clippy_info: channel::GitInfo,
256+
miri_info: channel::GitInfo,
256257
rustfmt_info: channel::GitInfo,
257258
local_rebuild: bool,
258259
fail_fast: bool,
@@ -374,6 +375,7 @@ impl Build {
374375
let cargo_info = channel::GitInfo::new(&config, &src.join("src/tools/cargo"));
375376
let rls_info = channel::GitInfo::new(&config, &src.join("src/tools/rls"));
376377
let clippy_info = channel::GitInfo::new(&config, &src.join("src/tools/clippy"));
378+
let miri_info = channel::GitInfo::new(&config, &src.join("src/tools/miri"));
377379
let rustfmt_info = channel::GitInfo::new(&config, &src.join("src/tools/rustfmt"));
378380

379381
let mut build = Build {
@@ -396,6 +398,7 @@ impl Build {
396398
cargo_info,
397399
rls_info,
398400
clippy_info,
401+
miri_info,
399402
rustfmt_info,
400403
cc: HashMap::new(),
401404
cxx: HashMap::new(),

src/bootstrap/tool.rs

Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -592,6 +592,14 @@ tool_extended!((self, builder),
592592
});
593593
};
594594
Miri, miri, "src/tools/miri", "miri", {};
595+
CargoMiri, clippy, "src/tools/miri", "cargo-miri", {
596+
// Miri depends on procedural macros (serde), which requires a full host
597+
// compiler to be available, so we need to depend on that.
598+
builder.ensure(compile::Rustc {
599+
compiler: self.compiler,
600+
target: builder.config.build,
601+
});
602+
};
595603
Rls, rls, "src/tools/rls", "rls", {
596604
let clippy = builder.ensure(Clippy {
597605
compiler: self.compiler,

0 commit comments

Comments
 (0)