fanquake
commented at 2:54 pm on October 29, 2022:
member
Otherwise our config.guess and config.sub will be copied over. This problem has been masked by the fact that modern systems ship with versions that recognise all the triplets we use (namely arm64-apple-darwin). However building on ubuntu 20.04 surfaces the issue.
build: copy config.{guess,sub} post autogen in zmq package
Otherwise our config.guess and config.sub will be copied over. This
problem has been masked by the fact that modern systems ship with
versions that recognise all the triplets we use (namely
arm64-apple-darwin). However building on ubuntu 20.04 surfaces the
issue.
Fixes #26420.
1914e470e3
fanquake added the label
Build system
on Oct 29, 2022
maflcko added the label
DrahtBot Guix build requested
on Oct 29, 2022
hebasto approved
hebasto
commented at 11:36 am on October 30, 2022:
member
ACK1914e470e327091c625f627b4447beb336f7fe5a, tested on Ubuntu 18.04.
Updating of config.{guess,sub} files was introduced in 0d1f38c45ff40f17b42074e3b58211e794a19edb. And it worked well until got broken in 6897c4bdf51a4aa74320ebfffa9467db14670109, where the added ./autogen.sh invocation overwrites them back.
Does it worth moving such copying commands from $(package)_preprocess_cmds into $(package)_config_cmds for all packages, for the sake of consistency?
DrahtBot
commented at 6:39 am on October 31, 2022:
contributor
If your review is incorrectly listed, please react with 👎 to this comment and the bot will ignore it on the next update.
fanquake
commented at 4:42 pm on February 6, 2023:
member
Hmm, won’t this break native builds that require a newer config.* than we ship?
Possibly, however I don’t think we’ve ever run into an instance of that happening. More often than not, the issue is the opposite, and the system config.guess/sub are not new enough. Hence why we use this copying-over method in multiple other depends packages; it’s been a necessity for RISC-V and arm64-darwin.
fanquake merged this
on Feb 7, 2023
fanquake closed this
on Feb 7, 2023
sidhujag referenced this in commit
7ea4a1a9dd
on Feb 7, 2023
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: 2024-11-21 12:12 UTC
This site is hosted by @0xB10C More mirrored repositories can be found on mirror.b10c.me