depends: use a patch instead of sed in libxcb #25738

pull fanquake wants to merge 1 commits into bitcoin:master from fanquake:remove_sed_libxcb changing 2 files +14 −1

fanquake commented at 1:42 PM on July 29, 2022: member

To remove the unneeded pthread-stubs requirement.

Should almost be enough to close #16838.

seds dead (mostly). The usage left in qt.mk are for substituting runtime values.

libxcb: use a patch instead of sed

To remove the unneeded pthread-stubs requirements.

8f1ff487b3

fanquake added the label Build system on Jul 29, 2022
hebasto commented at 1:45 PM on July 29, 2022: member

Concept ACK.
hebasto approved
hebasto commented at 4:40 PM on July 29, 2022: member

ACK 8f1ff487b3df6f3e4cb53f88045a32ea4fbc0839.

Using #21995, I've verified that package archive hashes are identical for builds with and without this PR. Also verified that the patch is being applied cleanly.
fanquake merged this on Jul 30, 2022
fanquake closed this on Jul 30, 2022
fanquake deleted the branch on Jul 30, 2022
hebasto commented at 10:09 AM on July 30, 2022: member

Using #21995, I've verified that package archive hashes are identical for builds with and without this PR. @fanquake Mind looking into #21995?
sidhujag referenced this in commit 1cdef88224 on Aug 1, 2022
bitcoin locked this on Jul 30, 2023

Contributors

Labels