Skip to content

[TEST] [Travis] Remove hostname workaround #8253

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.

Sign up for GitHub

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

Jump to bottom
Closed
fanquake wants to merge 1 commit into from
Closed

[TEST] [Travis] Remove hostname workaround #8253

fanquake wants to merge 1 commit into from

Conversation

@fanquake
Copy link
Member

@fanquake fanquake commented Jun 24, 2016

According to comments here the upstream fix for the hostname bug has been deployed, so we should be able to remove the workaround from travis.yml. Test PR to see build results.

@fanquake
Copy link
Member Author

fanquake commented Jun 24, 2016

Looks like the changes might not have been made upstream..

make[2]: Entering directory `/home/travis/build/bitcoin/bitcoin/build'
/bin/mkdir -p qa/tmp
USAGE: bitcoinjBlockStoreLocation runLargeReorgs(1/0) [port=18444]
07:33:01 1 H2FullPrunedBlockStore.<init>: org.h2.Driver loaded. 
*** buffer overflow detected ***: /usr/bin/java terminated
======= Backtrace: =========
/lib/x86_64-linux-gnu/libc.so.6(+0x7329f)[0x2b1be62db29f]
/lib/x86_64-linux-gnu/libc.so.6(__fortify_fail+0x5c)[0x2b1be6372bbc]
/lib/x86_64-linux-gnu/libc.so.6(+0x109a90)[0x2b1be6371a90]
/usr/lib/jvm/java-7-openjdk-amd64/jre/lib/amd64/libnet.so(Java_java_net_Inet4AddressImpl_getLocalHostName+0x1a5)[0x2b1bf3219665]
[0x2b1bf0014e98]

@maflcko maflcko added the Tests label Jun 24, 2016
@maflcko
Copy link
Member

maflcko commented Jul 5, 2016

Concept ACK

@fanquake fanquake force-pushed the remove-travis-workaround branch from ca931d3 to 211b4fd Compare July 21, 2016 08:22
@maflcko
Copy link
Member

maflcko commented Aug 6, 2016

Travis fails. I think this is not important enough to keep open. Can be reopened later...

@maflcko maflcko closed this Aug 6, 2016
@maflcko maflcko mentioned this pull request Aug 17, 2016
Merged
@fanquake fanquake deleted the remove-travis-workaround branch January 22, 2020 11:29
@bitcoin bitcoin locked as resolved and limited conversation to collaborators Feb 15, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

No reviews

Assignees

No one assigned

Labels

Tests

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@fanquake @maflcko