Skip to content

Commit

Permalink
Revert "Removed spurious recompile on integration tests (#468)" (#483)
Browse files Browse the repository at this point in the history
This reverts commit 994a17e.

This sped up compilation on nix, but rather embarrassingly broke it at
the same time. The shame.
  • Loading branch information
DavidM-D authored and ppca committed Mar 13, 2024
1 parent 1a59af2 commit 633496f
Showing 1 changed file with 1 addition and 16 deletions.
17 changes: 1 addition & 16 deletions integration-tests/src/mpc.rs
Original file line number Diff line number Diff line change
Expand Up @@ -44,22 +44,7 @@ async fn build_package(
package: &str,
target: Option<&str>,
) -> anyhow::Result<ExitStatus> {
// Do you have direnv installed?
let has_direnv = Command::new("which")
.arg("direnv")
.output()
.await
.expect("Failed to execute which command")
.status
.success();

let mut cmd = if has_direnv {
// If so use the same compiler you always use
Command::new("direnv exec cargo")
} else {
// Otherwise give up and face the pain of constant recompilation
Command::new("cargo")
};
let mut cmd = Command::new("cargo");
cmd.arg("build")
.arg("--package")
.arg(package)
Expand Down

0 comments on commit 633496f

Please sign in to comment.