New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[qa] wallet: Wait for reindex to catch up #7757

Merged
merged 1 commit into from Mar 29, 2016

Conversation

Projects
None yet
4 participants
@MarcoFalke
Member

MarcoFalke commented Mar 28, 2016

Should fix #7745

@MarcoFalke

This comment has been minimized.

Show comment
Hide comment
@MarcoFalke

MarcoFalke Mar 28, 2016

Member

Travis fail seems unrelated: #7470 (comment)

Member

MarcoFalke commented Mar 28, 2016

Travis fail seems unrelated: #7470 (comment)

@laanwj

This comment has been minimized.

Show comment
Hide comment
@laanwj

laanwj Mar 28, 2016

Member

utACK, thanks

Member

laanwj commented Mar 28, 2016

utACK, thanks

@jonasschnelli jonasschnelli added the Tests label Mar 29, 2016

@jonasschnelli

This comment has been minimized.

Show comment
Hide comment
@jonasschnelli
Member

jonasschnelli commented Mar 29, 2016

utACK fa3fafc

@laanwj laanwj merged commit fa3fafc into bitcoin:master Mar 29, 2016

1 check passed

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

laanwj added a commit that referenced this pull request Mar 29, 2016

Merge #7757: [qa] wallet: Wait for reindex to catch up
fa3fafc [qa] wallet: Wait for reindex to catch up (MarcoFalke)

@MarcoFalke MarcoFalke deleted the MarcoFalke:Mf1603-qaWalletReindex branch Mar 29, 2016

MarcoFalke added a commit to MarcoFalke/bitcoin that referenced this pull request Apr 5, 2016

@NicolasDorier

This comment has been minimized.

Show comment
Hide comment
@NicolasDorier

NicolasDorier Apr 12, 2016

Member

@MarcoFalke it does not seem to have fixed it I got the problem when compiling NicolasDorier@596e1f3

image

Member

NicolasDorier commented on fa3fafc Apr 12, 2016

@MarcoFalke it does not seem to have fixed it I got the problem when compiling NicolasDorier@596e1f3

image

This comment has been minimized.

Show comment
Hide comment
@MarcoFalke

MarcoFalke Apr 12, 2016

Member

I can't see what the line above Assertion failed says, but I guess it is the known "salvagewallet problem": #7379 (comment)?

Member

MarcoFalke replied Apr 12, 2016

I can't see what the line above Assertion failed says, but I guess it is the known "salvagewallet problem": #7379 (comment)?

This comment has been minimized.

Show comment
Hide comment
@NicolasDorier

NicolasDorier Apr 12, 2016

Member

@MarcoFalke I don't know if it is related, no trace of salvage wallet, but this is indeed the same line.

http://pastebin.com/vjLZ5Ktr

Member

NicolasDorier replied Apr 12, 2016

@MarcoFalke I don't know if it is related, no trace of salvage wallet, but this is indeed the same line.

http://pastebin.com/vjLZ5Ktr

This comment has been minimized.

Show comment
Hide comment
@MarcoFalke

MarcoFalke Apr 12, 2016

Member

@NicolasDorier Oh, I forgot that 6aae129 is not yet backported/merged into 0.12, so it is not printing anything right now. But I am almost sure that this is unrelated to reindexing.

Member

MarcoFalke replied Apr 12, 2016

@NicolasDorier Oh, I forgot that 6aae129 is not yet backported/merged into 0.12, so it is not printing anything right now. But I am almost sure that this is unrelated to reindexing.

zander added a commit to zander/bitcoinclassic that referenced this pull request Jun 16, 2016

nomnombtc added a commit to nomnombtc/bitcoin that referenced this pull request Nov 11, 2016

nomnombtc added a commit to nomnombtc/bitcoin that referenced this pull request Nov 12, 2016

nomnombtc added a commit to nomnombtc/bitcoin that referenced this pull request Nov 13, 2016

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment