Skip to content

Commit

Permalink
Make x use the x and x.ps1 scripts
Browse files Browse the repository at this point in the history
This removes another python search from bootstrap.
  • Loading branch information
albertlarsan68 committed Dec 18, 2022
1 parent 35a99ee commit 8348e05
Showing 1 changed file with 32 additions and 40 deletions.
72 changes: 32 additions & 40 deletions src/tools/x/src/main.rs
Original file line number Diff line number Diff line change
@@ -1,51 +1,43 @@
//! Run `x.py` from any subdirectory of a rust compiler checkout.
//! Run bootstrap from any subdirectory of a rust compiler checkout.
//!
//! We prefer `exec`, to avoid adding an extra process in the process tree.
//! However, since `exec` isn't available on Windows, we indirect through
//! `exec_or_status`, which will call `exec` on unix and `status` on Windows.
//!
//! We use `python`, `python3`, or `python2` as the python interpreter to run
//! `x.py`, in that order of preference.
//! We use `powershell.exe x.ps1` on Windows, and `sh -c x` on Unix, those are
//! the ones that call `x.py`. We use `sh -c` on Unix, because it is a standard.
//! We also don't use `pwsh` on Windows, because it is not installed by default;

use std::{
env::{self, consts::EXE_EXTENSION},
io,
env, io,
path::Path,
process::{self, Command, ExitStatus},
};

const PYTHON: &str = "python";
const PYTHON2: &str = "python2";
const PYTHON3: &str = "python3";

fn python() -> &'static str {
let val = match env::var_os("PATH") {
Some(val) => val,
None => return PYTHON,
};

let mut python2 = false;
let mut python3 = false;

for dir in env::split_paths(&val) {
// `python` should always take precedence over python2 / python3 if it exists
if dir.join(PYTHON).with_extension(EXE_EXTENSION).exists() {
return PYTHON;
}
#[cfg(windows)]
fn x_command(dir: &Path) -> Command {
let mut cmd = Command::new("powershell.exe");
cmd.args([
"-NoLogo",
"-NoProfile",
"-NonInteractive",
"-ExecutionPolicy",
"RemoteSigned",
"-Command",
"./x.ps1",
])
.current_dir(dir);
cmd
}

python2 |= dir.join(PYTHON2).with_extension(EXE_EXTENSION).exists();
python3 |= dir.join(PYTHON3).with_extension(EXE_EXTENSION).exists();
}
#[cfg(unix)]
fn x_command(dir: &Path) -> Command {
Command::new(dir.join("x"))
}

// try 3 before 2
if python3 {
PYTHON3
} else if python2 {
PYTHON2
} else {
// Python was not found on path, so exit
eprintln!("Unable to find python in your PATH. Please check it is installed.");
process::exit(1);
}
#[cfg(not(any(windows, unix)))]
fn x_command(_dir: &Path) -> Command {
compile_error!("Unsupported platform");
}

#[cfg(unix)]
Expand All @@ -72,15 +64,15 @@ fn main() {
let candidate = dir.join("x.py");

if candidate.exists() {
let mut python = Command::new(python());
let mut cmd = x_command(dir);

python.arg(&candidate).args(env::args().skip(1)).current_dir(dir);
cmd.args(env::args().skip(1)).current_dir(dir);

let result = exec_or_status(&mut python);
let result = exec_or_status(&mut cmd);

match result {
Err(error) => {
eprintln!("Failed to invoke `{}`: {}", candidate.display(), error);
eprintln!("Failed to invoke `{:?}`: {}", cmd, error);
}
Ok(status) => {
process::exit(status.code().unwrap_or(1));
Expand Down

0 comments on commit 8348e05

Please sign in to comment.