Page MenuHomePhabricator

Update doc/release-process.md for ABC
ClosedPublic

Authored by Mengerian on Jul 16 2017, 01:35.

Details

Reviewers
freetrader
deadalnix
Group Reviewers
Restricted Owners Package(Owns No Changed Paths)
Restricted Project
Commits
rSTAGINGa17b22eea0f1: Update doc/release-process.md for ABC
rABCa17b22eea0f1: Update doc/release-process.md for ABC
Summary

This is intended to create a more useful guideline for helping with
Bitcoin ABC releases.

It is fairly minimal for now. The goal is to help provide a checklist
to make releases easier.

This document will need to be expanded and refined over time.

This document was created in conjunction with freetrader

Test Plan

Read the doc as we do a release

Diff Detail

Repository
rABC Bitcoin ABC
Branch
rel-proc
Lint
No Lint Coverage
Unit
No Test Coverage
Build Status
Buildable 521
Build 521: arc lint + arc unit

Event Timeline

Owners added a reviewer: Restricted Owners Package.Jul 16 2017, 01:35
sickpig added inline comments.
doc/release-process.md
20

I think we should keep those.
AssumeValid is a new feature that replace the old checksum thing, but:

  • it does not effect consensus rules (in case of a reorg you would still accept the reorg, but you are going to validate completely all the blocks)
  • IBD perf greatly.
  • it is user configurable

Basically it lowers the barrier to get a new client up and running from scratch by a lot.

deadalnix requested changes to this revision.Jul 17 2017, 14:28
deadalnix added inline comments.
doc/release-process.md
20

+1

This revision now requires changes to proceed.Jul 17 2017, 14:28
Mengerian edited edge metadata.

re-add updating of "defaultAssumeValid" to release checklist

This revision is now accepted and ready to land.Jul 18 2017, 15:41
This revision was automatically updated to reflect the committed changes.
Mengerian marked 2 inline comments as done.