This was added by me a while back, with the intention of expanding what this did. That hasn't happened, and this hasn't gained much use. There's also been some discussion of some configure option fatigue, so just remove it for now. Note that -Wl,--no-insert-timestamp is also already used in the Guix build.
build: remove --enable-determinism configure option #22406
pull fanquake wants to merge 1 commits into bitcoin:master from fanquake:remove_enable_determinism changing 1 files +0 −13-
fanquake commented at 3:59 PM on July 5, 2021: member
-
build: remove --enable-determinism configure option e46287853f
- fanquake added the label Build system on Jul 5, 2021
-
MarcoFalke commented at 4:03 PM on July 5, 2021: member
review ACK e46287853f3a41c3f0772d3448d8df4ea01a156f
-
jarolrod commented at 4:22 PM on July 5, 2021: member
Code Review ACK e46287853f3a41c3f0772d3448d8df4ea01a156f
This reverts https://github.com/bitcoin/bitcoin/pull/18135
- fanquake merged this on Jul 6, 2021
- fanquake closed this on Jul 6, 2021
- fanquake deleted the branch on Jul 6, 2021
-
laanwj commented at 9:19 AM on July 6, 2021: member
Posthumous ACK, thanks for helping prune down the number of configure options.
- sidhujag referenced this in commit c77f59c1ba on Jul 6, 2021
- gwillen referenced this in commit f4b20c9b0d on Jun 1, 2022
- DrahtBot locked this on Aug 16, 2022
Contributors
Labels