namecoin / electrum-nmc

Namecoin port of Electrum Bitcoin client.
https://www.namecoin.org/
MIT License
29 stars 24 forks source link

Extra patches for Electrum-NMC stable branches #164

Open JeremyRand opened 5 years ago

JeremyRand commented 5 years ago

Some concerns have been raised by one of our funding sources about the slow progress getting patches merged to upstream Electrum. Due to this, Electrum-NMC's stable branches will begin carrying extra patches that have been submitted upstream but not yet merged upstream. The unstable bitcoin/auxpow/master branches will not carry those patches, in order to minimize merge conflicts when the patches eventually get merged upstream. This issue will track which patches are being carried in our stable branches, and link to the corresponding upstream PR's.

JeremyRand commented 5 years ago

Status on relevant patches:

JeremyRand commented 5 years ago

Updated status on relevant patches:

JeremyRand commented 5 years ago

@domob1812 Will adding these patches to Electrum-NMC stable branches cause any disruption to Xaya?

domob1812 commented 5 years ago

@JeremyRand: I've not looked at them, but in general you should not be concerned about that. Electrum-NMC should do what is best for Namecoin, and I will worry about how that relates to Xaya.

domob1812 commented 5 years ago

FYI, I've now looked at the patches in a bit more detail. The only one that is relevant for Xaya is spesmilo#5147. Xaya has continuous difficulty retargeting, and because of that needs a lot more information in checkpoints in order to be able to compute the following difficulty. But since your patch essentially simplifies checkpoint handling, it will probably also simplify Xaya's changes. Hence it will probably require some work on my side after merging, but will then probably make life easier in the long term.

JeremyRand commented 4 years ago

Updated status on relevant patches:

JeremyRand commented 4 years ago

Updated status on relevant patches:

JeremyRand commented 4 years ago

https://github.com/spesmilo/electrum/pull/5537 is merged upstream, so is being removed from this list.

Updated status on relevant patches:

JeremyRand commented 4 years ago

Updated status on relevant patches:

JeremyRand commented 4 years ago

https://github.com/spesmilo/electrum/pull/5432 is merged upstream, so is being removed from this list.

JeremyRand commented 4 years ago

Updated status on relevant patches:

JeremyRand commented 4 years ago

Updated status on relevant patches:

JeremyRand commented 4 years ago

Updated status on relevant patches:

JeremyRand commented 4 years ago

Updated status on relevant patches:

JeremyRand commented 4 years ago

Updated status on relevant patches:

JeremyRand commented 4 years ago

Updated status on relevant patches:

JeremyRand commented 4 years ago

Updated status on relevant patches:

JeremyRand commented 4 years ago

Updated status on relevant patches: