Skip to content

Commit

Permalink
Merge #692: CI: Revert cc pin in ASAN job
Browse files Browse the repository at this point in the history
7b6a13b CI: Revert cc pin in ASAN job (Tobin C. Harding)

Pull request description:

  Revert commit: 92778ef

  We can't use git to revert it because the code has now moved from `test.sh` to `_test.sh`.

  I don't remember the problem but lets just use CI to see if its fixed.

ACKs for top commit:
  sanket1729:
    utACK 7b6a13b
  apoelstra:
    ACK 7b6a13b

Tree-SHA512: d804c73152b3d8b14e8f13e64066c33e2dfbdef8b184d55672638df6b468a6f636e632c5e1a0b09e617534aaf466b1c719c6df16952aaf8a51cb2659bed1d0ef
  • Loading branch information
apoelstra committed Mar 27, 2024
2 parents d5f8b2a + 7b6a13b commit 1674406
Showing 1 changed file with 0 additions and 5 deletions.
5 changes: 0 additions & 5 deletions contrib/_test.sh
Original file line number Diff line number Diff line change
Expand Up @@ -101,11 +101,6 @@ if [ "$DO_ASAN" = true ]; then
RUSTFLAGS='-Zsanitizer=memory -Zsanitizer-memory-track-origins -Cforce-frame-pointers=yes -Cllvm-args=-msan-eager-checks=0' \
cargo test --lib --all --features="$FEATURES" -Zbuild-std --target x86_64-unknown-linux-gnu

pushd "$REPO_DIR/no_std_test" > /dev/null || exit 1
# See https://github.com/rust-bitcoin/rust-secp256k1/pull/641#issuecomment-1671598914
cargo update -p cc --precise 1.0.79
popd > /dev/null || exit 1

cargo run --release --manifest-path=./no_std_test/Cargo.toml | grep -q "Verified Successfully"
cargo run --release --features=alloc --manifest-path=./no_std_test/Cargo.toml | grep -q "Verified alloc Successfully"
fi
Expand Down

0 comments on commit 1674406

Please sign in to comment.