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

build: Install OpenSSL with chocolatey only for x64 #7719

Merged
merged 2 commits into from
Jul 19, 2023

Conversation

cosmo0920
Copy link
Contributor

@cosmo0920 cosmo0920 commented Jul 19, 2023

This is because Chocolatey only supports x64 installation after OpenSSL 3.1.1 package.

ref: https://community.chocolatey.org/packages/openssl/3.1.1#files

The CI result is:
https://github.com/fluent/fluent-bit/actions/runs/5596570106/jobs/10233770791


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.

  • Run local packaging test showing all targets (including any new ones) build.
  • Set ok-package-test label to test for all targets (requires maintainer to do).

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.

This is because Chocolatey only supports x64 installation after OpenSSL
3.1.1 package.

ref: https://community.chocolatey.org/packages/openssl/3.1.1#files

Signed-off-by: Hiroshi Hatake <[email protected]>
@cosmo0920 cosmo0920 temporarily deployed to pr July 19, 2023 07:39 — with GitHub Actions Inactive
@cosmo0920 cosmo0920 temporarily deployed to pr July 19, 2023 07:39 — with GitHub Actions Inactive
@cosmo0920 cosmo0920 temporarily deployed to pr July 19, 2023 08:03 — with GitHub Actions Inactive
@cosmo0920 cosmo0920 temporarily deployed to pr July 19, 2023 08:03 — with GitHub Actions Inactive
@cosmo0920 cosmo0920 temporarily deployed to pr July 19, 2023 08:03 — with GitHub Actions Inactive
@cosmo0920 cosmo0920 temporarily deployed to pr July 19, 2023 08:28 — with GitHub Actions Inactive
@cosmo0920 cosmo0920 marked this pull request as ready for review July 19, 2023 09:17
@patrick-stephens patrick-stephens added ci ok-package-test Run PR packaging tests and removed docs-required labels Jul 19, 2023
@patrick-stephens
Copy link
Contributor

Confirming a full build works, once it does we can merge.

@patrick-stephens patrick-stephens merged commit 103196d into master Jul 19, 2023
@patrick-stephens patrick-stephens deleted the cosmo0920-build-openssl-x86-and-arm64 branch July 19, 2023 09:55
Wiston999 pushed a commit to Wiston999/fluent-bit that referenced this pull request Aug 7, 2023
* build: Install OpenSSL with chocolatey only for x64

This is because Chocolatey only supports x64 installation after OpenSSL
3.1.1 package.

ref: https://community.chocolatey.org/packages/openssl/3.1.1#files

Signed-off-by: Hiroshi Hatake <[email protected]>

* build: Use the correct directory to refer the installed place of OpenSSL

Signed-off-by: Hiroshi Hatake <[email protected]>

---------

Signed-off-by: Hiroshi Hatake <[email protected]>
leonardo-albertovich pushed a commit that referenced this pull request Oct 5, 2023
* build: Install OpenSSL with chocolatey only for x64

This is because Chocolatey only supports x64 installation after OpenSSL
3.1.1 package.

ref: https://community.chocolatey.org/packages/openssl/3.1.1#files

Signed-off-by: Hiroshi Hatake <[email protected]>

* build: Use the correct directory to refer the installed place of OpenSSL

Signed-off-by: Hiroshi Hatake <[email protected]>

---------

Signed-off-by: Hiroshi Hatake <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ci ok-package-test Run PR packaging tests
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants