Skip to content
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

Fix type errors in cprofiles #9598

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

jeredfloyd
Copy link

Fix type errors that prevent build on Fedora 40+.

Fluent Bit is licensed under Apache 2.0, by submitting this pull request I understand that this code will be released under the terms of that license.

@arasic
Copy link

arasic commented Nov 18, 2024

Can you inspect futher, after applying your changes, I got this error:

[ 21%] Built target cmetrics-static
[ 22%] Built target ctraces-static
[ 22%] Building C object lib/cprofiles/src/CMakeFiles/cprofiles-static.dir/cprof_profile.c.o
/home/arasic/github/fluent-bit/lib/cprofiles/src/cprof_profile.c: In function ‘cprof_profile_destroy’:
/home/arasic/github/fluent-bit/lib/cprofiles/src/cprof_profile.c:175:9: error: ‘function’ undeclared (first use in this function)
  175 |         function = cfl_list_entry(iterator,
      |         ^~~~~~~~

thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants