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

workflows: macos: fix libyaml package name #5606

Merged
merged 1 commit into from
Jun 22, 2022
Merged

Conversation

Dentrax
Copy link
Contributor

@Dentrax Dentrax commented Jun 21, 2022

Signed-off-by: Furkan furkan.turkal@trendyol.com

The correct equivalent of libyaml-dev in Homebrew is just libyaml.

Also ignore .vscode dir


Enter [N/A] in the box, if an item is not applicable to your change.

Testing
Before we can approve your change; please submit the following in a comment:

  • Example configuration file for the change
  • Debug log output from testing the change
  • Attached Valgrind output that shows no leaks or memory corruption was found

If this is a change to packaging of containers or native binaries then please confirm it works for all targets.

Documentation

  • Documentation required for this feature

Backporting

  • Backport to latest stable release.

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.

Signed-off-by: Furkan <furkan.turkal@trendyol.com>
@niedbalski niedbalski merged commit a6e3a2a into fluent:master Jun 22, 2022
@patrick-stephens
Copy link
Contributor

It looks like this does not resolve the issue: https://github.com/fluent/fluent-bit/runs/7017803108?check_suite_focus=true

In fact, libyaml is already on the runners. I'm guessing some kind of issue with finding the relevant libraries from cmake maybe?

@Dentrax
Copy link
Contributor Author

Dentrax commented Jun 23, 2022

Actually I'm getting same error on my local so couldn't find a way out except passing -DFLB_CONFIG_YAML=Off to cmake.

Also cross ref: #5131

pwhelan pushed a commit that referenced this pull request Jun 28, 2022
Signed-off-by: Furkan <furkan.turkal@trendyol.com>
pwhelan pushed a commit that referenced this pull request Jun 28, 2022
Signed-off-by: Furkan <furkan.turkal@trendyol.com>
Signed-off-by: Phillip Whelan <phil@calyptia.com>
mgeriesa pushed a commit to mgeriesa/fluent-bit that referenced this pull request Oct 25, 2022
Signed-off-by: Furkan <furkan.turkal@trendyol.com>
Signed-off-by: Manal Geries <mgeriesa@gmail.com>
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