New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
guix: document when certain patches can be dropped #27668
Conversation
The following sections might be updated with supplementary metadata relevant to reviewers and maintainers. ReviewsSee the guideline for information on the review process.
If your review is incorrectly listed, please react with 👎 to this comment and the bot will ignore it on the next update. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Concept ACK.
contrib/guix/patches/glibc-2.27-riscv64-Use-__has_include-to-include-asm-syscalls.h.patch
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ACK a09269a, I have reviewed the changes and they look OK.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ACK a09269a
Can confirm the note on the long jump patch ;)
a09269a guix: document when certain guix patches can be dropped (fanquake) Pull request description: Additional notes for when patches can be dropped. ACKs for top commit: hebasto: ACK a09269a, I have reviewed the changes and they look OK. jarolrod: ACK a09269a Tree-SHA512: c1876b9a4e3cf73645d25c9077cef19a9b6b7fe2eda5dc9d82fd3ca3f9105453406c1b197e6635035b6ce19c9f255c070bebed5563f68913033d04627202155a
…dant glibc patches Summary: > guix: document when certain guix patches can be dropped Note that we don't support powerpc, and the glibc-2.28-fcommon patch mentions that it specifically fixes an issue for this platform, but the patch is applied by Core to all builds, so it is best to keep it. > guix: remove redundant glibc patches > > These should only be relevant for a glibc that is built as part of a > Guix system, and should not be required for a glibc that is just being > built to compile our binaries against. A x86_64 linux bitcoind produced > with Guix using master vs this change has no difference. i.e: This is a backport of [[bitcoin/bitcoin#27668 | core#27668]] and [[bitcoin/bitcoin#27670 | core#27670]] Depends on D15330 Test Plan: `contrib/guix/guix-build` Reviewers: #bitcoin_abc, Fabien Reviewed By: #bitcoin_abc, Fabien Differential Revision: https://reviews.bitcoinabc.org/D15331
Additional notes for when patches can be dropped.