Page MenuHomePhabricator
Differential D335

Update doc/release-process.md for ABC
ClosedPublic
Actions

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

Mengerian created this revision.Jul 16 2017, 01:35
Owners added a reviewer: Restricted Owners Package.Jul 16 2017, 01:35
Herald added a reviewer: Restricted Project. · View Herald TranscriptJul 16 2017, 01:35
Harbormaster completed remote builds in B521: Diff 836.Jul 16 2017, 01:35
sickpig added a subscriber: sickpig.Jul 17 2017, 13:52
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 updated this revision to Diff 848.Jul 17 2017, 20:10
Mengerian edited edge metadata.

re-add updating of "defaultAssumeValid" to release checklist

freetrader accepted this revision as: freetrader.Jul 17 2017, 20:40

Fine with me.

deadalnix accepted this revision.Jul 18 2017, 15:41
This revision is now accepted and ready to land.Jul 18 2017, 15:41
Closed by commit rABCa17b22eea0f1: Update doc/release-process.md for ABC (authored by Mengerian, committed by Mengerian). · Explain WhyJul 18 2017, 15:44
This revision was automatically updated to reflect the committed changes.
Mengerian marked 2 inline comments as done.

Revision Contents
Changeset List

PathSize
doc/
316 lines

Diff 836

View Options

doc/release-process.md

Loading...