Skip to content

Commit

Permalink
Merge #4409
Browse files Browse the repository at this point in the history
4409: Remove bashism from CPU check script r=marc-casperlabs a=marc-casperlabs

Small fix to simplify the given script.


Co-authored-by: Marc Brinkmann <[email protected]>
  • Loading branch information
casperlabs-bors-ng[bot] and marc-casperlabs authored Nov 20, 2023
2 parents 08d0bec + f6b043f commit 19aeefa
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 4 deletions.
2 changes: 1 addition & 1 deletion .drone.yml
Original file line number Diff line number Diff line change
Expand Up @@ -151,7 +151,7 @@ steps:
- name: check CPU features
<<: *buildenv
commands:
- bash -c ./ci/check_cpu_features.sh
- ./ci/check_cpu_features.sh

volumes:
- name: rustup
Expand Down
4 changes: 1 addition & 3 deletions ci/check_cpu_features.sh
Original file line number Diff line number Diff line change
Expand Up @@ -3,11 +3,9 @@
# Ensure that there has not been a change in CPU features used.

set -e
shopt -s expand_aliases

ROOT_DIR="$(cd "$(dirname "${BASH_SOURCE[0]}")/.." >/dev/null 2>&1 && pwd)"
cd $(dirname $0)/..

cd ${ROOT_DIR}
cargo build --release --bin casper-node
utils/dump-cpu-features.sh target/release/casper-node > current-build-cpu-features.txt
diff -u current-build-cpu-features.txt ci/cpu-features-1.4.13-release.txt
Expand Down

0 comments on commit 19aeefa

Please sign in to comment.