forked from dashpay/dash
-
Notifications
You must be signed in to change notification settings - Fork 725
[Consensus] Define SPORK_17 #975
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
Merged
furszy
merged 2 commits into
PIVX-Project:master
from
random-zebra:2019_coldStaking_spork
Oct 24, 2019
Merged
[Consensus] Define SPORK_17 #975
furszy
merged 2 commits into
PIVX-Project:master
from
random-zebra:2019_coldStaking_spork
Oct 24, 2019
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
bf1a2ac to
6ea8599
Compare
Author
|
rebased |
6ea8599 to
d668235
Compare
d668235 to
bc4a99f
Compare
Author
|
Rebased. |
bc4a99f to
0ff825d
Compare
bc90cdd to
6240435
Compare
Author
|
Rebased on top of #1000 |
6240435 to
5a6eab5
Compare
Author
|
Rebased. |
4042c88 to
f69e7af
Compare
8e7a032 to
8ec8f13
Compare
41265df to
c5c56f9
Compare
|
rebase needed |
define SPORK_17_COLDSTAKING_ENFORCEMENT. When such spork is not active, transactions with P2CS outputs are rejected
c5c56f9 to
fffc6c5
Compare
Author
|
Rebased |
furszy
approved these changes
Oct 23, 2019
furszy
left a comment
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.
ACK fffc6c5
Fuzzbawls
approved these changes
Oct 23, 2019
Collaborator
Fuzzbawls
left a comment
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.
ACK fffc6c5
furszy
added a commit
that referenced
this pull request
Oct 24, 2019
fffc6c5 [Core] Lower the banscore for P2CS when spork 17 disabled to 10 (random-zebra) 20effe6 [Consensus] Set Cold Staking activation via SPORK (random-zebra) Pull request description: This is based on top of #955 (only the last commit, 5a6eab590505d409481ee3f385d65b09f40bb8fd , is relevant for this PR). It sets Cold Staking activation via SPORK instead of having it height-based. When `SPORK_17_COLDSTAKING_ENFORCEMENT` is **not** active (default), transactions with P2CS outputs are rejected on the network. This way only cold-staking and new delegations are disabled while coin-owners can still spend their coins. ACKs for top commit: furszy: ACK fffc6c5 Fuzzbawls: ACK fffc6c5 Tree-SHA512: bac8e267d0c6f74896cf6915c67de6880d033aaf757f3ac0564bf816bf5fa5abe91b41dc8f90328e8a028026ddd1e6636023c6bcc405041813841ee231e22565
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This is based on top of #955 (only the last commit, 5a6eab590505d409481ee3f385d65b09f40bb8fd , is relevant for this PR).
It sets Cold Staking activation via SPORK instead of having it height-based.
When
SPORK_17_COLDSTAKING_ENFORCEMENTis not active (default), transactions with P2CS outputs are rejected on the network. This way only cold-staking and new delegations are disabled while coin-owners can still spend their coins.