No description provided.
Autoconf improvements, part 2 #3147
pull luke-jr wants to merge 3 commits into bitcoin:master from luke-jr:autoconf_pt2 changing 8 files +122 −91-
luke-jr commented at 10:02 PM on October 24, 2013: member
-
configure: Simplify common AM_CPPFLAGS and AM_LDFLAGS to a Makefile.common 6dcb165dbb
-
configure: Automatically disable bitcoin-qt by default, if dependencies are not met af15eacd5e
-
configure: Add unsupported --with-system-leveldb configure flag fa94a2ee7a
-
BitcoinPullTester commented at 10:31 PM on October 24, 2013: none
Automatic sanity-testing: FAILED BUILD/TEST, see http://jenkins.bluematt.me/pull-tester/fa94a2ee7a3610c285ba57e02938aa05172cf5dd for binaries and test log.
This could happen for one of several reasons:
- It chanages paths in makefile.linux-mingw or otherwise changes build scripts in a way that made them incompatible with the automated testing scripts (please tweak those patches in qa/pull-tester)
- It adds/modifies tests which test network rules (thanks for doing that), which conflicts with a patch applied at test time
- It does not build on either Linux i386 or Win32 (via MinGW cross compile)
- The test suite fails on either Linux i386 or Win32
- The block test-cases failed (lookup the first bNN identifier which failed in https://github.com/TheBlueMatt/test-scripts/blob/master/FullBlockTestGenerator.java)
If you believe this to be in error, please ping BlueMatt on freenode or TheBlueMatt here.
This test script verifies pulls every time they are updated. It, however, dies sometimes and fails to test properly. If you are waiting on a test, please check timestamps to verify that the test.log is moving at http://jenkins.bluematt.me/pull-tester/current/ Contact BlueMatt on freenode if something looks broken.
-
jgarzik commented at 11:14 PM on October 24, 2013: contributor
Looks pretty good, based on a 60-second, untested review ;p
-
laanwj commented at 10:24 AM on October 25, 2013: member
Agree with the first two commits; --with-system-leveldb is a bit controversial, as it may nudge distro packagers the wrong way. There's a good reason that we include our own copy of leveldb.
-
wtogami commented at 8:52 PM on October 25, 2013: contributor
I have to agree that --with-system-leveldb sends the wrong message to distributions.
-
luke-jr commented at 9:23 PM on October 25, 2013: member
Perhaps an AC_WARN would be appropriate.
- luke-jr closed this on Dec 1, 2013
- Bushstar referenced this in commit be127bc2ee on Apr 8, 2020
- DrahtBot locked this on Sep 8, 2021