-
Notifications
You must be signed in to change notification settings - Fork 375
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
[0.17] Activate CSV from genesis block for custom chains #505
[0.17] Activate CSV from genesis block for custom chains #505
Conversation
2db3e75
to
fba529d
Compare
fba529d
to
e67541c
Compare
Realized that bitcoin_functional run was actually running both suites, causing it to timeout. fixed in fixup commit. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just some doubt about turning off the tests.
|
||
This soft fork will activate the following BIPS: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is it smart to remove all this instead of disable somehow? I'm not sure, but I think this might make upstream updates harder.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I could remove the test from test_runner, then add a specific check in a custom test?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
That'd work as well.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
rather than disable the test, I just reverted all the changes and had the functional suite default to bitcoin-regtest argument
9c5dbc4
to
3774d46
Compare
3774d46
to
91dd327
Compare
tACK 91dd327 |
91dd327 Disable functional tests for bitcoin_functional build (Gregory Sanders) b8820d1 Activate CSV from genesis block for custom chains (Gregory Sanders) Pull request description: Guts the test for elements mode, can add back later if worked on. Tree-SHA512: a2b7633fa02dae582616ac26d25cd08f895d5940e3775c70cacdd3f678f01a21cbcaf56c6c540b91bd8535bb1aab7bdc59e2a79b5a1124cbd9d3f03f29d0b40e
Does anyone remember the motivation for this? |
Guts the test for elements mode, can add back later if worked on.