build: remove unneeded configure $*val setting #24671

pull fanquake wants to merge 29 commits into bitcoin:master from fanquake:remove_uneeded_configure_defaults changing 2 files +230 −226
  1. fanquake commented at 11:42 AM on March 25, 2022: member

    Setting these values has been redundant since autoconf ~2.60, and we require 2.69. Should not change behaviour. Includes minor formatting improvements.

    secp256k1 also recently made the same change: https://github.com/bitcoin-core/secp256k1/pull/1079.

  2. fanquake added the label Build system on Mar 25, 2022
  3. fanquake added the label DrahtBot Guix build requested on Mar 25, 2022
  4. fanquake commented at 11:42 AM on March 25, 2022: member

    cc @real-or-random

  5. real-or-random commented at 12:05 PM on March 25, 2022: contributor

    The automatic setting works only for variables enable_foo ( --enable-foo) or variables with_foo (--with-foo). But some of the removed assignments assign to use_foo or build_bitcoin_foo variables.

  6. fanquake removed the label DrahtBot Guix build requested on Mar 25, 2022
  7. fanquake commented at 12:16 PM on March 25, 2022: member

    The automatic setting works only for variables enable_foo ( --enable-foo) or variables with_foo (--with-foo).

    😅 Right. I guess we try should clean these up as well; given we currently use a mix of everything throughout configure.. Will update the changes.

  8. fanquake force-pushed on Mar 30, 2022
  9. fanquake commented at 8:29 PM on March 30, 2022: member

    Right. I guess we try should clean these up as well; given we currently use a mix of everything throughout configure.. Will update the changes.

    Have done this now. The changes are too granular as-is, and can be split-out / squashed down, but this should better reflect what we want to accomplish.

  10. jb55 commented at 9:08 PM on March 30, 2022: contributor

    ACK, tested that {enable,disable}-usdt still works

  11. DrahtBot cross-referenced this on Mar 31, 2022 from issue build: Bump libevent minimum version up to 2.1.8 by hebasto
  12. DrahtBot commented at 4:49 AM on March 31, 2022: contributor

    <!--e57a25ab6845829454e8d69fc972939a-->

    The following sections might be updated with supplementary metadata relevant to reviewers and maintainers.

    <!--174a7506f384e20aa4161008e828411d-->

    Conflicts

    Reviewers, this pull request conflicts with the following ones:

    • #24958 (build: Fix macOS Apple M1 build with miniupnpc and libnatpmp. Again :) by hebasto)
    • #24798 ([POC] build: Hello Qt 6 by hebasto)
    • #24291 (build: Remove negated --enable-fuzz checks from build system by MarcoFalke)
    • #24051 (Bugfix: configure: bitcoin-{cli,tx,util} don't need UPnP, NAT-PMP, or ZMQ by luke-jr)
    • #22644 (Deprecate UPnP support, require 2.1 or later by fanquake)

    If you consider this pull request important, please also help to review the conflicting pull requests. Ideally, start with the one that should be merged first.

  13. bitcoin deleted a comment on Mar 31, 2022
  14. DrahtBot cross-referenced this on Mar 31, 2022 from issue build: Fix ccache behavior when cross-compiling for darwin hosts by hebasto
  15. DrahtBot cross-referenced this on Mar 31, 2022 from issue build: stop overriding user autoconf flags by fanquake
  16. DrahtBot cross-referenced this on Mar 31, 2022 from issue [kernel 1/n] Introduce initial `libbitcoinkernel` by dongcarl
  17. DrahtBot cross-referenced this on Mar 31, 2022 from issue build: Remove negated --enable-fuzz checks from build system by MarcoFalke
  18. DrahtBot cross-referenced this on Mar 31, 2022 from issue Bugfix: configure: bitcoin-{cli,tx,util} don't need UPnP, NAT-PMP, or ZMQ by luke-jr
  19. DrahtBot cross-referenced this on Mar 31, 2022 from issue build: deprecate UPnP support by fanquake
  20. DrahtBot added the label Needs rebase on Apr 5, 2022
  21. build: autoconfize --with-seccomp bb640da744
  22. build: autoconfize --enable-external-signer 45f7ad05ac
  23. build: autoconfize --disable-zmq dc1f954df3
  24. build: autoconfize --with-sanitizers 4f9004bd3a
  25. build: autoconfize --enable_usdt e92c6f667a
  26. build: autotoolize --enable-suppress-external-warnings e9b1f840f9
  27. build: autotoolize --enable-asm b03d3bf6ff
  28. build: autotoolize --enable-lto feb3f6e8ec
  29. build: autotoolize --enable-werror ebcb7ef813
  30. build: autotoolize --enable-gprof 95208ed27c
  31. build: autotoolize --enable-debug e593a63e1f
  32. build: autotoolize --disable-man 6ffec0f1d3
  33. build: autotoolize --multiprocess related flags 6dd2418756
  34. build: autotoolize --enable-threadlocal 832c0df56b
  35. build: autotoolize lcov flags 4e8f39a66c
  36. build: autotoolize --enable-ccache 2a085bd44c
  37. build: autotoolize --enable-reduce-exports 3e34bbb4c2
  38. build: autotoolize --enable-hardening ddcea824a8
  39. build: autotoolize --with-qrencode fa6328b332
  40. build: autotoolize --enable-fuzz* 8a13e95c18
  41. build: autotoolize --enable-extended-functional-tests a5513eb2f7
  42. build: autotoolize --enable-bench 4dea2cdaf7
  43. build: autotoolize --enable-gui-tests ecffefc6b5
  44. build: autotoolize --enable-tests 7294cb65b2
  45. build: autotoolize --with-natpmp* 184fb629a7
  46. build: autotoolize --enable-wallet 8d5933ac47
  47. build: autotoolize upnp 891deb9faf
  48. build: autotoolize --with-bdb ae6e709e26
  49. build: autotoolize --with-sqlite 92c9d75b4f
  50. fanquake force-pushed on Apr 20, 2022
  51. fanquake commented at 12:49 PM on April 20, 2022: member

    Rebased past #24391 and #24681. I'll start splitting a few changes off from here so we can move things forward.

  52. DrahtBot removed the label Needs rebase on Apr 20, 2022
  53. DrahtBot cross-referenced this on Apr 20, 2022 from issue [POC] build: Hello Qt 6 by hebasto
  54. real-or-random commented at 11:25 AM on April 23, 2022: contributor

    I guess it's clear from the context but:

    Concept ACK

  55. DrahtBot cross-referenced this on Apr 25, 2022 from issue build: Fix macOS Apple M1 build with miniupnpc and libnatpmp. Again :) by hebasto
  56. DrahtBot added the label Needs rebase on Apr 28, 2022
  57. DrahtBot commented at 7:21 PM on April 28, 2022: contributor

    <!--cf906140f33d8803c4a75a2196329ecb-->

    🐙 This pull request conflicts with the target branch and needs rebase.

    <sub>Want to unsubscribe from rebase notifications on this pull request? Just convert this pull request to a "draft".</sub>

  58. laanwj commented at 6:16 AM on June 2, 2022: member

    Concept and light code review ACK 92c9d75b4f55e853e8148ede2a3334af1bd3e204

  59. fanquake commented at 10:36 AM on August 31, 2022: member

    May follow up with some related changes later on.

  60. fanquake closed this on Aug 31, 2022
  61. bitcoin locked this on Aug 31, 2023
  62. fanquake deleted the branch on Sep 14, 2023
Contributors
fanquakereal-or-randomjb55DrahtBotlaanwj
Labels
Build systemNeeds rebase
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-05-20 06:53 UTC