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
perl plugin: fix Clang compilation by ignoring a warning #4309
Conversation
Changed title to better describe what is being fixed, and added the required (Fix) label & "ChangeLog" entry. |
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.
If the warned macro issue cannot be fixed (I know very little of Perl, so I have no idea how hard that would be), at least the #if
guard should be fixed to check for correct compiler version.
@mrunge, do you have Linux setup for checking whether Perl plugin has same issue with current Linux Clang / or any other comments on this? (I don't have any Perl dev stuff on my setup, and I'd rather keep it out of it, as I'm rather a Python user. :-)) |
Commit title should be also be fixed, to indicate that it's the perl plugin that is being fixed (not clang one). |
A new warning type introduced since clang 12 causes a build failure. Ignore it to fix. https://reviews.llvm.org/D86751 Fixes: collectd#4308
@eero-t please re-review |
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.
Thanks, this looks fine now! => Triggered checks & approved.
I am not using the perl plugin, and I'm also not building collectd on a regular basis anymore, so can't add test cases or feedback to this. |
@jvsg thank you for your PR, it looks good to me. |
ChangeLog: perl plugin: fix Clang compilation by ignoring a warning
Fixes: #4308