Skip to content

Commit

Permalink
cli: fixes bug where the wrong exit status is reported
Browse files Browse the repository at this point in the history
  • Loading branch information
RuvenSalamon committed Jan 25, 2023
1 parent 5f9b590 commit bf6488e
Show file tree
Hide file tree
Showing 2 changed files with 6 additions and 1 deletion.
1 change: 1 addition & 0 deletions CHANGELOG.md
Expand Up @@ -21,6 +21,7 @@ The minor version will be incremented upon a breaking change and the patch versi

- cli: Don't regenerate idl in read_all_programs(). ([#2332](https://github.com/coral-xyz/anchor/pull/2332)).
- ts: `provider.simulate` will send the transaction with `sigVerify: false` if no `signers` are present ([#2331](https://github.com/coral-xyz/anchor/pull/2331)).
- cli: Failing commands will return the correct exit status. ([#2370](https://github.com/coral-xyz/anchor/pull/2370)).

### Breaking

Expand Down
6 changes: 5 additions & 1 deletion avm/src/anchor/main.rs
Expand Up @@ -14,11 +14,15 @@ fn main() -> anyhow::Result<()> {
version
);
}
Command::new(binary_path)
let exit = Command::new(binary_path)
.args(args)
.spawn()?
.wait_with_output()
.expect("Failed to run anchor-cli");

if !exit.status.success() {
std::process::exit(exit.status.code().unwrap_or(1));
}

Ok(())
}

0 comments on commit bf6488e

Please sign in to comment.