diff options
author | fanquake <fanquake@gmail.com> | 2023-10-12 11:59:47 +0200 |
---|---|---|
committer | fanquake <fanquake@gmail.com> | 2023-10-12 12:00:12 +0200 |
commit | 06d469c26b556962e0e8e0f141a576b6aea95564 (patch) | |
tree | 08a23dde9618241d8ab3981dc7256f0e5a948cfa /contrib | |
parent | a927d5cabefbb3d2c60d69d6314cb0b768431dac (diff) | |
parent | 4077e43bf62e5afe90d204b9ede9290ef54dee0f (diff) |
Merge bitcoin/bitcoin#28629: test: fix usdt undeclared function errors on mantis
4077e43bf62e5afe90d204b9ede9290ef54dee0f test: fix usdt undeclared function errors on mantis (willcl-ark)
Pull request description:
This is one way to fix #28600
Recently usage of undeclared functions became an error rather than a warning, in C2x. https://reviews.llvm.org/D122983?id=420290
This change has migrated into the build tools of Ubuntu 23.10 which now causes the USDT tests to fail to compile, see
https://github.com/bitcoin/bitcoin/issues/28600
I think there are various potential fixes:
1. Manually declare the functions we use
2. Fix imports so that manual declarations aren't needed
3. Revert the new C2X behaviour and don't error on implicit function declarations
I would have preferred solution 2, but I believe this will require changes to the upstream bcc package. Having played with the imports I can get things working in a standalone C program, using system headers, but when building the program from a python context as we do in the test it uses its own headers (bundled with the python lib) rather than the system ones, and manually importing (some) system headers results in definition mismatches. I also investigated explicitly importing required headers from the package, which use paths like `#import </virtual/bcc/bcc_helpers.h>`, but this seems more obtuse and brittle than simply ignoring the warning.
Therefore I think that until the upstream python pacakge fixes their declarations, we should fix this by setting `-Wno-error=implicit-function-declaration` for the tracing programs.
cc maflcko 0xB10C
ACKs for top commit:
maflcko:
lgtm ACK 4077e43bf62e5afe90d204b9ede9290ef54dee0f
Tree-SHA512: 8368bb1155e920a95db128dc893267f8dab64f1ae53f6d63c6d9294e2e4e92bef8515e3697e9113228bedc51c0afdbc5bbcf558c119bf0eb3293dc2ced86b435
Diffstat (limited to 'contrib')
0 files changed, 0 insertions, 0 deletions