Skip to content

Various fixes and improvements#4

Open
erasmospunk wants to merge 3 commits intocitp:masterfrom
erasmospunk:master
Open

Various fixes and improvements#4
erasmospunk wants to merge 3 commits intocitp:masterfrom
erasmospunk:master

Conversation

@erasmospunk
Copy link

Improvements:

  1. Use a prefix for the temp folders.
  2. When starting the node, wait for the RPC to be ready. Results if faster start of the script.
  3. When terminating the node, wait of the process to finish (kills it after a timeout). Results in faster termination of the script.

Fix crash in latest bitcoind:

When running the latest bitcoin 0.20.0, the script
crashes when "Creating output with 0 BTC" because
the node rejects the "dust" transaction.

Fix travis tests

1) Use a prefix for the temp folders.
2) When starting the node, wait for the RPC to be ready. Results if faster start of the script.
3) When terminating the node, wait of the process to finish (kills it after a timeout). Results in faster termination of the script.
When running the latest bitcoin 0.20.0, the script
crashes when "Creating output with 0 BTC" because
the node rejects the "dust" transaction.

Here we remove the relay fee and fix the issue.
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.

1 participant