hebasto
commented at 1:49 pm on November 24, 2024:
member
Using helper variables has two issues:
They contaminate the global namespace of the main build script.
They can be used as set(var), effectively exposing a cache variable var, which makes the toolchain file susceptible to the build environment.
The depends/Makefile can generate values with “not-set” semantics as empty strings or strings containing only spaces. For example:https://github.com/bitcoin/bitcoin/blob/2638fdb4f934be96b7c798dbac38ea5ab8a6374a/depends/Makefile#L157
Therefore, MATCHES must be used rather than STREQUAL.
DrahtBot
commented at 1:50 pm on November 24, 2024:
contributor
The following sections might be updated with supplementary metadata relevant to reviewers and maintainers.
DrahtBot removed the label
CI failed
on Mar 18, 2025
DrahtBot added the label
Needs rebase
on May 7, 2025
depends: Avoid using helper variables in toolchain file
Using helper variables has two issues:
1. They contaminate the global namespace of the main build script.
2. They can be used as `set(var)`, effectively exposing a cache variable
`var`, which makes the toolchain file susceptible to the build
environment.
7343a1846c
hebasto force-pushed
on May 7, 2025
hebasto
commented at 4:05 pm on May 7, 2025:
member
Rebased due to a conflict with the merged bitcoin/bitcoin#28710.
DrahtBot removed the label
Needs rebase
on May 7, 2025
fanquake added the label
DrahtBot Guix build requested
on May 8, 2025
fanquake
commented at 9:27 am on May 8, 2025:
member
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: 2025-07-03 00:13 UTC
This site is hosted by @0xB10C More mirrored repositories can be found on mirror.b10c.me