Page MenuHomePhabricator
Differential D316

Update /doc/bips.md from Core to ABC
ClosedPublic
Actions

Authored by Mengerian on Jul 10 2017, 21:29.

Details

Reviewers
deadalnix
freetrader
Group Reviewers
Restricted Owners Package(Owns No Changed Paths)
Restricted Project
Commits
rSTAGINGcd58893b70a2: Update /doc/bips.md from Core to ABC
rABCcd58893b70a2: Update /doc/bips.md from Core to ABC
Summary

Remove BIP 125 (RBF)
Remove BIP 144, 144, 145 (Segwit)
Leave BIP 143 in place, used in Bitcoin ABC replay protected sighash

Test Plan

None

Diff Detail

Repository
rABC Bitcoin ABC
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

Mengerian created this revision.Jul 10 2017, 21:29
Owners added a reviewer: Restricted Owners Package.Jul 10 2017, 21:29
Herald added a reviewer: Restricted Project. · View Herald TranscriptJul 10 2017, 21:29
Harbormaster completed remote builds in B487: Diff 784.Jul 10 2017, 21:29
Mengerian updated this revision to Diff 785.Jul 10 2017, 21:39

fix BIP numbers

Harbormaster completed remote builds in B488: Diff 785.Jul 10 2017, 21:39
deadalnix requested changes to this revision.Jul 10 2017, 22:09
deadalnix added inline comments.
doc/bips.md
27 ↗(On Diff #785)

I think we should keep these around. It can be edited to mention these have been removed in Bitcoin ABC.

This revision now requires changes to proceed.Jul 10 2017, 22:09
Mengerian updated this revision to Diff 788.Jul 10 2017, 22:24
Mengerian edited edge metadata.

Address deadalnix comment to leave BIPs 125, 144, 145, 147 in file, but note that they are removed in Bitcoin ABC

Mengerian updated this revision to Diff 789.Jul 10 2017, 22:27
Mengerian marked an inline comment as done.

Update git commit message

deadalnix accepted this revision.Jul 11 2017, 09:26
This revision is now accepted and ready to land.Jul 11 2017, 09:26
Closed by commit rABCcd58893b70a2: Update /doc/bips.md from Core to ABC (authored by Mengerian, committed by Mengerian). · Explain WhyJul 11 2017, 15:10
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathSize
doc/
12 lines

Diff 791

View Options

doc/bips.md

Loading...