Page MenuHomePhabricator

[chronik] Bump the RocksDb crate version to 0.21.0
ClosedPublic

Authored by Fabien on Jul 31 2023, 12:22.

Details

Summary

This fixes the build with recent Clang versions that used to fail for a missing stdint header inclusion.

Test Plan
mkdir buildChronik && cd buildChronik
cmake -GNinja .. -DBUILD_BITCOIN_CHRONIK=ON
ninja all check-crates
./test/functional/test_runner.py chronik_*

Diff Detail

Repository
rABC Bitcoin ABC
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

Fabien requested review of this revision.Jul 31 2023, 12:22

for test plans involving ninja ...

Is it implied that I should also be building everything in a build folder, then running from there?

If I run ninja check-crates from build/ , I get ninja: error: unknown target 'check-crates'

for test plans involving ninja ...

Is it implied that I should also be building everything in a build folder, then running from there?

If I run ninja check-crates from build/ , I get ninja: error: unknown target 'check-crates'

Yes. I can update the test plan with full instructions for this time

This revision is now accepted and ready to land.Aug 1 2023, 20:46