Skip to content

lib: miniz: only require a C compiler - #10315

Merged
edsiper merged 1 commit into
fluent:masterfrom
ThomasDevoogdt:bugfix/lib-miniz-only-require-a-c-compiler
May 29, 2025
Merged

lib: miniz: only require a C compiler#10315
edsiper merged 1 commit into
fluent:masterfrom
ThomasDevoogdt:bugfix/lib-miniz-only-require-a-c-compiler

Conversation

@ThomasDevoogdt

Copy link
Copy Markdown
Contributor

Commit richgel999/miniz@05ab4dc "Add some catch2 tests" dropped that C flag, but should instead only have added a simple enable_language call for the tests only.

Upstream: richgel999/miniz@383e551

Commit richgel999/miniz@05ab4dc
"Add some catch2 tests" dropped that C flag, but should instead only have added a simple
enable_language call for the tests only.

Upstream: richgel999/miniz@383e551
Signed-off-by: Thomas Devoogdt <thomas@devoogdt.com>
@ThomasDevoogdt
ThomasDevoogdt force-pushed the bugfix/lib-miniz-only-require-a-c-compiler branch from 3eeea12 to 945ce04 Compare May 8, 2025 20:40
@cosmo0920 cosmo0920 added this to the Fluent Bit v4.0.2 milestone May 9, 2025
@edsiper
edsiper merged commit 9db9746 into fluent:master May 29, 2025
@ThomasDevoogdt
ThomasDevoogdt deleted the bugfix/lib-miniz-only-require-a-c-compiler branch May 29, 2025 17:50
arnout pushed a commit to buildroot/buildroot that referenced this pull request Jun 6, 2025
News:
- https://fluentbit.io/announcements/v4.0.3/

Other remarks:
- Dropped 0003-lib-miniz-only-require-a-C-compiler.patch
  as it is upstream. [1]

- Added msgpack-c as a system dependency since this has been
  added to buildroot.

- Added sqlite as system dependency as it can't really be
  disabled otherwise due to an upstream compile bug. [2]

- Sorted the config entries while at it.

[1] fluent/fluent-bit#10315
[2] fluent/fluent-bit#9757

Signed-off-by: Thomas Devoogdt <thomas@devoogdt.com>
Signed-off-by: Julien Olivain <ju.o@free.fr>
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.

3 participants