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
  1. luke-jr commented at 10:02 PM on October 24, 2013: member

    No description provided.

  2. configure: Simplify common AM_CPPFLAGS and AM_LDFLAGS to a Makefile.common 6dcb165dbb
  3. configure: Automatically disable bitcoin-qt by default, if dependencies are not met af15eacd5e
  4. configure: Add unsupported --with-system-leveldb configure flag fa94a2ee7a
  5. 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:

    1. 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)
    2. It adds/modifies tests which test network rules (thanks for doing that), which conflicts with a patch applied at test time
    3. It does not build on either Linux i386 or Win32 (via MinGW cross compile)
    4. The test suite fails on either Linux i386 or Win32
    5. 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.

  6. jgarzik commented at 11:14 PM on October 24, 2013: contributor

    Looks pretty good, based on a 60-second, untested review ;p

  7. 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.

  8. 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.

  9. luke-jr commented at 9:23 PM on October 25, 2013: member

    Perhaps an AC_WARN would be appropriate.

  10. laanwj commented at 7:08 AM on October 28, 2013: member

    @luke-jr you know just as well as us that they'll just ignore that

  11. luke-jr closed this on Dec 1, 2013

  12. Bushstar referenced this in commit be127bc2ee on Apr 8, 2020
  13. DrahtBot locked this on Sep 8, 2021

github-metadata-mirror

This is a metadata mirror of the GitHub repository bitcoin/bitcoin. This site is not affiliated with GitHub. Content is generated from a GitHub metadata backup.
generated: 2026-04-13 18:16 UTC

This site is hosted by @0xB10C
More mirrored repositories can be found on mirror.b10c.me