Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Use dynamic linking for RocksDB when building electrs #2786

Draft
wants to merge 1 commit into
base: dev
Choose a base branch
from

Conversation

romanz
Copy link
Contributor

@romanz romanz commented Dec 10, 2021

@romanz
Copy link
Contributor Author

romanz commented Dec 10, 2021

CC: @Kixunil @openoms

@@ -286,7 +286,7 @@ if [ "$1" = "1" ] || [ "$1" = "on" ]; then
sudo -u electrs git reset --hard $ELECTRSVERSION
sudo -u electrs /home/admin/config.scripts/blitz.git-verify.sh \
"${PGPsigner}" "${PGPpubkeyLink}" "${PGPpubkeyFingerprint}" || exit 1
sudo -u electrs /home/electrs/.cargo/bin/cargo build --locked --release || exit 1
sudo -u electrs ROCKSDB_INCLUDE_DIR=/usr/include ROCKSDB_LIB_DIR=/usr/lib /home/electrs/.cargo/bin/cargo build --locked --release || exit 1
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Haven't tested it on an actual RPi devce, but it should (hopefully) work 🤞

Copy link

@Kixunil Kixunil left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code looks OK

@openoms
Copy link
Collaborator

openoms commented Dec 10, 2021

It is building ok without errors and faster due to having rocksdb already:

$ time config.scripts/bonus.electrs.sh on
real    25m46.872s                                  
user    50m13.638s                                  
sys     1m43.499s  

but when started running into this:
romanz/electrs#616

admin@raspberrypi:~ $ sudo -u electrs /home/electrs/electrs/target/release/electrs
Starting electrs 0.9.3 on aarch64 linux with Config { network: Bitcoin, db_path: "/mnt/hdd/app-storage/electrs/db/bitcoin", daemon_dir: "/home/electrs/.bitcoin", daemon_auth: UserPass("raspibolt", "<sensitive>"), daemon_rpc_addr: 127.0.0.1:8332, daemon_p2p_addr: 127.0.0.1:8333, electrum_rpc_addr: 127.0.0.1:50001, monitoring_addr: 127.0.0.1:4224, wait_duration: 10s, jsonrpc_timeout: 15s, index_batch_size: 10, index_lookup_limit: Some(200), reindex_last_blocks: 0, auto_reindex: true, ignore_mempool: false, sync_once: false, disable_electrum_rpc: false, server_banner: "Welcome to electrs v0.9.3 - the Electrum Rust Server on your RaspiBlitz", args: [] }
Illegal instruction

Same when installing Rust with:
sudo apt install cargo -y

@romanz romanz marked this pull request as draft December 11, 2021 07:58
@romanz
Copy link
Contributor Author

romanz commented Dec 11, 2021

Marking as "draft" until romanz/electrs#616 is resolved :(

@rootzoll rootzoll added this to the 1.8 Release milestone Dec 14, 2021
@rootzoll rootzoll modified the milestones: 1.8.0 Release, 1.8.3 Release May 9, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants