@laanwj here are some edits to the release notes I didn't add anything about abandontransaction (maybe it should go in the detailed changes section though) or add clarification in the conflicted section about how this changes whether prevouts are respendable or not. I think it's fine to leave as is, but just pointing out in case you though otherwise.
release note fixups #7336
pull morcos wants to merge 1 commits into bitcoin:0.12 from morcos:rnfixes changing 1 files +22 −19-
morcos commented at 8:24 PM on January 13, 2016: member
-
release note fixups 82667afd78
- laanwj added this to the milestone 0.12.0 on Jan 13, 2016
-
laanwj commented at 9:14 PM on January 13, 2016: member
Thanks, ACK
-
MarcoFalke commented at 9:27 PM on January 13, 2016: member
Great, thanks for taking a look!
ACK 82667af
-
MarcoFalke commented at 9:28 PM on January 13, 2016: member
maybe it should go in the detailed changes section though
Right. Could you add another commit to do this?
-
laanwj commented at 9:38 PM on January 13, 2016: member
Re: abandontransaction it is already mentioned in the detailed changes:
- #7312
fd4bd50Add RPC call abandontransaction
As I understood it's meant as a mostly hidden feature, to be used if there is another mempool flood / malleation attack, so I think that is mention enough.
- #7312
-
morcos commented at 9:43 PM on January 13, 2016: member
Oh sorry, I was looking for it in wallet No I agree, I think its ok not to draw explicit attention to it
-
MarcoFalke commented at 10:31 AM on January 14, 2016: member
Missed it as well. 82667af looks ready to merge.
-
jonasschnelli commented at 7:27 PM on January 14, 2016: contributor
ACK
- laanwj merged this on Jan 16, 2016
- laanwj closed this on Jan 16, 2016
- laanwj referenced this in commit d7c54c5a9d on Jan 16, 2016
- MarcoFalke locked this on Sep 8, 2021
Contributors
Milestone
0.12.0