Skip to content

Commit

Permalink
fix: Delay rming bins till right before installing them. (AztecProtoc…
Browse files Browse the repository at this point in the history
…ol#4474)

As title.
Fixes issue whereby providing a bad first argument to `aztec-up` deleted
bins and left user have to rerun install.
  • Loading branch information
charlielye authored Feb 7, 2024
1 parent 83a3136 commit fabeac8
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion aztec-up/bin/aztec-install
Original file line number Diff line number Diff line change
Expand Up @@ -90,7 +90,6 @@ fi
# Create a "hidden" `$HOME/.aztec` dir, so as not to clutter the user's cwd.
mkdir -p $AZTEC_PATH
mkdir -p $BIN_PATH
rm -f $BIN_PATH/aztec*

# Download containers from dockerhub. Tag them as latest.
function pull_container {
Expand Down Expand Up @@ -121,6 +120,7 @@ function install_bin {
}

info "Installing scripts in $BIN_PATH..."
rm -f $BIN_PATH/aztec*
install_bin .aztec-run
install_bin aztec
install_bin aztec-cli
Expand Down

0 comments on commit fabeac8

Please sign in to comment.