-
Notifications
You must be signed in to change notification settings - Fork 38.7k
Description
@sdaftuar Noticed trouble with his builds, and it looks like its due to an unmaintained file in bitcoinj (which is used in pull-testing)
For accurate build results, I think this should be either updated (the tool*) or deprecated until it is maintained. @theuni do you have any preference ? I ask since it seems you had your hands in the travis yml last, I can look at the bitcoinj side to see what it would take, seems it might just need an update to handle 0.10?
failing build: https://travis-ci.org/bitcoin/bitcoin/jobs/61800593
passing build (throws exceptions all over - Ctrl-F "Exception" ):https://travis-ci.org/bitcoin/bitcoin/jobs/61800595
*tool hasn't seen a commit since december, and likely needs 0.10 updates: https://github.com/bitcoinj/bitcoinj/blob/master/core/src/test/java/org/bitcoinj/core/BitcoindComparisonTool.java
edit: link to erroneous build command https://github.com/bitcoin/bitcoin/blob/master/Makefile.am#L195-L197