MIME-Version: 1.0
In-Reply-To: <CALqxMTFkgGx0FxMiZ77inOZSs_+TQ88Wpj-q-c12COkO9tP4gQ@mail.gmail.com>
References: <CALqxMTFkgGx0FxMiZ77inOZSs_+TQ88Wpj-q-c12COkO9tP4gQ@mail.gmail.com>
Date: Thu, 20 Aug 2015 11:00:14 +0200
Message-ID: <CA+w+GKTXAvq76sX4ttiYHhRcGp1MOynGAovYs8Dq1A1Q=9K4KQ@mail.gmail.com>
From: Mike Hearn <hearn@vinumeris.com>
To: Adam Back <adam@cypherspace.org>
Content-Type: multipart/alternative; boundary=089e0111d0c82a6d8b051dba61fa
Cc: Bitcoin Dev <bitcoin-dev@lists.linuxfoundation.org>
Subject: Re: [bitcoin-dev] Bitcoin XT Fork
Precedence: list

--089e0111d0c82a6d8b051dba61fa
Content-Type: text/plain; charset=UTF-8

>
> It is just that no one else is reckless enough to bypass the review process


I keep seeing this notion crop up.

I want to kill this idea right now:

   - There were months of public discussion leading to up the authoring of
   BIP 101, both on this mailing list and elsewhere.

   - BIP 101 was submitted for review via the normal process. Jeff Garzik
   specifically called Gavin out on Twitter and thanked him for following the
   process:

   https://twitter.com/jgarzik/status/614412097359708160

   https://github.com/bitcoin/bips/pull/163

   As you can see, other than a few minor typo fixes and a comment by sipa,
   there was no other review offered.

   - The implementation for BIP 101 was submitted to Bitcoin Core as a pull
   request, to invoke the code review process:

   https://github.com/bitcoin/bitcoin/pull/6341

   Some minor code layout suggestions were made by Cory and incorporated.
   Peter popped up to say there was no chance it'd ever be accepted ..... and
   no further review was done.

So the entire Bitcoin Core BIP process was followed to the letter. The net
result was this. There were, in fact, bugs in the implementation of BIP
101. They were found when Gavin submitted the code to the XT community
review process, which resulted in *actual* peer review. Additionally, there
was much discussion of technical details on the XT mailing list that
Bitcoin Core entirely ignored.

--089e0111d0c82a6d8b051dba61fa
Content-Type: text/html; charset=UTF-8
Content-Transfer-Encoding: quoted-printable

<div dir=3D"ltr"><div class=3D"gmail_extra"><div class=3D"gmail_quote"><blo=
ckquote class=3D"gmail_quote" style=3D"margin:0 0 0 .8ex;border-left:1px #c=
cc solid;padding-left:1ex">It is just that no one else is reckless enough t=
o bypass the review=C2=A0process</blockquote><div><br></div><div>I keep see=
ing this notion crop up.</div><div><br></div><div>I want to kill this idea =
right now:</div><div><ul><li>There were months of public discussion leading=
 to up the authoring of BIP 101, both on this mailing list and elsewhere.<b=
r><br></li><li>BIP 101 was submitted for review via the normal process. Jef=
f Garzik specifically called Gavin out on Twitter and thanked him for follo=
wing the process:<br><br><a href=3D"https://twitter.com/jgarzik/status/6144=
12097359708160">https://twitter.com/jgarzik/status/614412097359708160</a><b=
r><br><a href=3D"https://github.com/bitcoin/bips/pull/163">https://github.c=
om/bitcoin/bips/pull/163</a><br><br>As you can see, other than a few minor =
typo fixes and a comment by sipa, there was no other review offered.<br><br=
></li><li>The implementation for BIP 101 was submitted to Bitcoin Core as a=
 pull request, to invoke the code review process:<br><br><a href=3D"https:/=
/github.com/bitcoin/bitcoin/pull/6341">https://github.com/bitcoin/bitcoin/p=
ull/6341</a><br><br>Some minor code layout suggestions were made by Cory an=
d incorporated. Peter popped up to say there was no chance it&#39;d ever be=
 accepted ..... and no further review was done.</li></ul><div>So the entire=
 Bitcoin Core BIP process was followed to the letter. The net result was th=
is. There were, in fact, bugs in the implementation of BIP 101. They were f=
ound when Gavin submitted the code to the XT community review process, whic=
h resulted in <i>actual</i>=C2=A0peer review. Additionally, there was much =
discussion of technical details on the XT mailing list that Bitcoin Core en=
tirely ignored.</div></div><div><br></div><div><br></div></div></div></div>

--089e0111d0c82a6d8b051dba61fa--