build: comparison tool NPE fix #6318

Merged
merged 1 commit into from Jun 22, 2015

Conversation

Projects
None yet
4 participants
@theuni
Member

theuni commented Jun 21, 2015

This should fix the spurious comparison tool failures.
See discussion here: #6305

The race fix was cherry-picked on top of the version we're currently using, so it should be functionally identical otherwise.

build: comparison tool NPE fix
This should fix the spurious comparison tool failures.
See discussion here: #6305

The race fix was cherry-picked on top of the version we're currently using, so
it should be functionally identical otherwise.
@laanwj

This comment has been minimized.

Show comment
Hide comment
@laanwj

laanwj Jun 21, 2015

Member

ACK (actual code diff can be found here: bitcoinj/bitcoinj@ce50e0b )

Member

laanwj commented Jun 21, 2015

ACK (actual code diff can be found here: bitcoinj/bitcoinj@ce50e0b )

@laanwj laanwj added the Tests label Jun 21, 2015

@dexX7

This comment has been minimized.

Show comment
Hide comment
@dexX7

dexX7 Jun 21, 2015

Contributor

Confirmed, this one works.

Contributor

dexX7 commented Jun 21, 2015

Confirmed, this one works.

@fanquake

This comment has been minimized.

Show comment
Hide comment
@fanquake

fanquake Jun 22, 2015

Member

ACK

Member

fanquake commented Jun 22, 2015

ACK

@laanwj laanwj merged commit 7c2c270 into bitcoin:master Jun 22, 2015

1 check passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details

laanwj added a commit that referenced this pull request Jun 22, 2015

Merge pull request #6318
7c2c270 build: comparison tool NPE fix (Cory Fields)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment