Skip to content

Conversation

@jtimon
Copy link
Contributor

@jtimon jtimon commented Feb 17, 2018

Any class extending CChainParams directly should be fine.
Any class extending CRegTestParams should be fine, just extend CCustomParams instead (or just use the custom params if that's enough for your chain).

Dependencies:

- [ ] Chainparams: Every chain should have a name #307

@jtimon jtimon force-pushed the e14-unify-regtest-custom branch from 4747444 to 361faad Compare February 17, 2018 04:25
@jtimon jtimon mentioned this pull request Feb 17, 2018
3 tasks
@jtimon jtimon force-pushed the e14-unify-regtest-custom branch from 361faad to 0053760 Compare February 17, 2018 09:01
@jtimon jtimon changed the title HF: Chainparams: Unify CRegTestParams and CCustomParams Chainparams: Unify CRegTestParams and CCustomParams Feb 17, 2018
@jtimon jtimon force-pushed the e14-unify-regtest-custom branch 6 times, most recently from a659102 to 6dca473 Compare February 19, 2018 08:13
bitcoin-backport-12128: One CBaseChainParams should be enough
@jtimon jtimon force-pushed the e14-unify-regtest-custom branch from 72705e4 to b553501 Compare February 19, 2018 08:39
@jtimon
Copy link
Contributor Author

jtimon commented Feb 19, 2018

replace-by-fee.py was failing because fRequireStandard was true by default instead of false, updated

@instagibbs instagibbs merged commit b553501 into ElementsProject:elements-0.14.1 Feb 20, 2018
instagibbs added a commit that referenced this pull request Feb 20, 2018
b553501 Chainparams: Unify CRegTestParams and CCustomParams (Jorge Timón)
@instagibbs
Copy link
Contributor

merged

@jtimon jtimon deleted the e14-unify-regtest-custom branch February 20, 2018 23:09
@jtimon jtimon mentioned this pull request Feb 21, 2018
instagibbs added a commit that referenced this pull request Feb 21, 2018
6be1e70 f'Chainparams: Unify CRegTestParams and CCustomParams' (Jorge Timón)
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.

2 participants