Skip to content

Commit

Permalink
Merge branch 'master' of https://github.com/David-OConnor/pyflow
Browse files Browse the repository at this point in the history
  • Loading branch information
David-OConnor committed Nov 21, 2020
2 parents 6285091 + 93117dd commit 04d6c56
Show file tree
Hide file tree
Showing 3 changed files with 14 additions and 8 deletions.
10 changes: 7 additions & 3 deletions src/commands.rs
Expand Up @@ -3,7 +3,7 @@ use regex::Regex;
use std::{error::Error, fmt};
use std::{
path::{Path, PathBuf},
process::Command,
process::{Command, Stdio},
};

#[derive(Debug)]
Expand Down Expand Up @@ -110,8 +110,12 @@ pub fn run_python(
args: &[String],
) -> Result<(), Box<dyn Error>> {
util::set_pythonpath(lib_paths);
let output = Command::new(bin_path.join("python")).args(args).output()?;
util::check_command_output(&output, "running python");
Command::new(bin_path.join("python"))
.args(args)
.stdin(Stdio::inherit())
.stdout(Stdio::inherit())
.stderr(Stdio::inherit())
.output()?;
Ok(())
}

Expand Down
9 changes: 6 additions & 3 deletions src/dep_resolution.rs
Expand Up @@ -352,9 +352,12 @@ fn guess_graph(
d
} else {
util::abort(&format!(
"Problem getting dependency data - this is \
likely a bug in the cacheing process. Please try again in a few minutes. \
Reqs: {:#?}",
"Aborting graph creation: Problem getting dependency data\n \
Reqs: {:#?}
It's taking a long time to get dependency data - this \
usually suggests that the dependency tree is being newly \
built. Please try again in a few minutes, and if the error \
still occurs, consider opening an issue on github.",
&reqs
));
unreachable!()
Expand Down
3 changes: 1 addition & 2 deletions src/util.rs
Expand Up @@ -59,13 +59,12 @@ impl FromStr for Os {

fn from_str(s: &str) -> Result<Self, Self::Err> {
Ok(match s {
"manylinux1_i686" | "manylinux2010_i686" => Self::Linux32,
"manylinux1_i686" | "manylinux2010_i686" | "manylinux2014_i686" => Self::Linux32,
"cygwin"
| "linux"
| "linux2"
| "manylinux1_x86_64"
| "manylinux2010_x86_64"
| "manylinux2014_i686"
| "manylinux2014_aarch64"
| "manylinux2014_x86_64" => Self::Linux,
"win32" => Self::Windows32,
Expand Down

0 comments on commit 04d6c56

Please sign in to comment.