Page MenuHomePhabricator
Differential D11663

Install dependencies for Chronik: Rust and Corrosion
ClosedPublic
Actions

Authored by tobias_ruck on Jun 23 2022, 21:47.

Details

Reviewers
Fabien
Group Reviewers
Restricted Project
Commits
rABC0a3e7327bde7: Install dependencies for Chronik: Rust and Corrosion
Summary
Test Plan

On a Linux machine with docker:

  1. docker run -i -t bitcoinabc/bitcoin-abc-dev
  2. Run the commands added in this diff -> installs Rust and Corrosion successfully
  3. source $HOME/.cargo/env
  4. cargo --version results in version 1.61
  5. Test against experimental Chronik impl on GitHub: git clone https://github.com/raipay/bitcoin-abc.git && cd bitcoin-abc && git checkout chronik && mkdir build && cd build && cmake .. -GNinja -> Successfully run Corrosion and calls Rust
  6. ninja -> successfully builds bitcoind
  7. ./src/bitcoind -regtest -> runs Chronik, shows "Got BlockConnected 0f9188f13cb7b2c71f2a335e3a4fc328bf5beb436012afca590b1a11466e2206" and "Added block 0f9188f13cb7b2c71f2a335e3a4fc328bf5beb436012afca590b1a11466e2206 with 1 txs, height 0" in stdout

Diff Detail

Repository
rABC Bitcoin ABC
Branch
master
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 19466
Build 38665: Build Diff
Build 38664: arc lint + arc unit

Event Timeline

tobias_ruck created this revision.Jun 23 2022, 21:47
Herald added a reviewer: Restricted Project. · View Herald TranscriptJun 23 2022, 21:47
tobias_ruck requested review of this revision.Jun 23 2022, 21:47
Harbormaster completed remote builds in B19458: Diff 34112.Jun 23 2022, 21:47
tobias_ruck updated this revision to Diff 34124.Jun 24 2022, 13:13

Install specific Rust version via rustup

Harbormaster completed remote builds in B19466: Diff 34124.Jun 24 2022, 13:13
tobias_ruck edited the test plan for this revision. (Show Details)Jun 24 2022, 13:34
Fabien added inline comments.Jun 24 2022, 13:37
contrib/utils/install-dependencies.sh
147

Nit: Use tar xzf. From testing tar determines the type from the suffix but I couldn't find this behavior in the man pages (it's supposed to be option -a), so better do it explicitely

tobias_ruck updated this revision to Diff 34125.Jun 24 2022, 13:42
tobias_ruck edited the test plan for this revision. (Show Details)

tar xzf (mandatory xkcd https://xkcd.com/1168/)

Harbormaster completed remote builds in B19467: Diff 34125.Jun 24 2022, 13:42
tobias_ruck edited the test plan for this revision. (Show Details)Jun 24 2022, 13:44
Fabien accepted this revision.Jun 24 2022, 13:46
This revision is now accepted and ready to land.Jun 24 2022, 13:46
Closed by commit rABC0a3e7327bde7: Install dependencies for Chronik: Rust and Corrosion (authored by tobias_ruck). · Explain WhyJun 24 2022, 13:50
This revision was automatically updated to reflect the committed changes.
tobias_ruck added a commit: rABC0a3e7327bde7: Install dependencies for Chronik: Rust and Corrosion.
tobias_ruck mentioned this in D11668: [Chronik] Add minimal bitcoinsuite-core Rust crate + CMake to `cargo test` it.Jun 24 2022, 21:00
tobias_ruck mentioned this in rABC5615fb72f897: [Chronik] Add minimal bitcoinsuite-core Rust crate + CMake to `cargo test` it.Jun 27 2022, 12:43

Revision Contents
Changeset List

PathSize
contrib/
utils/
13 lines

Diff 34124

View Options

contrib/utils/install-dependencies.sh

Loading...