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.

Sign up for GitHub

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

Jump to bottom

headers: cleanup paths for external header inclusion #10054

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

headers: cleanup paths for external header inclusion #10054

wants to merge 3 commits into from
+51 −44

Conversation

edsiper
Copy link
Member

@edsiper edsiper commented Mar 6, 2025

  • move flb_network.h dns context definition inside private context in flb_network.c
  • cmake: c-ares and nghttp2: move headers path inside cmake/headers.cmake

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.

edsiper added 2 commits March 6, 2025 12:13
@edsiper
network: move context declaration to private scope
fcffffa 
Signed-off-by: Eduardo Silva <eduardo@calyptia.com>
@edsiper
cmake: move internal path headers to public headers.cmake file
8339848 
Signed-off-by: Eduardo Silva <eduardo@calyptia.com>
@edsiper edsiper added this to the Fluent Bit v4.0.0 milestone Mar 6, 2025
@edsiper
headers: expose new FLB_PUBLIC macro for shared lib visibility
9ed2889 
Signed-off-by: Eduardo Silva <eduardo@calyptia.com>
@patrick-stephens patrick-stephens added ok-package-test Run PR packaging tests and removed docs-required labels Mar 7, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@fujimotos fujimotos Awaiting requested review from fujimotos fujimotos is a code owner

@niedbalski niedbalski Awaiting requested review from niedbalski niedbalski is a code owner

@patrick-stephens patrick-stephens Awaiting requested review from patrick-stephens patrick-stephens is a code owner

@celalettin1286 celalettin1286 Awaiting requested review from celalettin1286 celalettin1286 is a code owner

@leonardo-albertovich leonardo-albertovich Awaiting requested review from leonardo-albertovich leonardo-albertovich is a code owner

@koleini koleini Awaiting requested review from koleini koleini is a code owner

At least 1 approving review is required to merge this pull request.

Assignees
No one assigned
Labels
ok-package-test Run PR packaging tests
Projects
None yet
Milestone
Fluent Bit v4.0.0
Development

Successfully merging this pull request may close these issues.
2 participants
@edsiper @patrick-stephens