Skip to content

Add Swoole extension to container #40

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

Merged
merged 2 commits into from
May 26, 2021

Conversation

weierophinney
Copy link
Member

This patch compiles and installs the Swoole extension (at version 4.6.7) for PHP versions 7.3, 7.4, and 8.0. It is NOT enabled by default; enablement is done selectively, as it is done with sqlsrv (combination of a mods-available INI file, and conditional logic when enabling extensions in the entrypoint). I have built the container locally, and it builds correctly; more importantly, I then ran it against the mezzio-swoole library to ensure I could run unit tests with it, and tests passed.

At some point, we will likely want to:

  • Add a script per extension we want to install manually
  • Add a script for each of the various functions/conditionals in the entrypoint to make it more modular

For now, however, this works, and will allow us to move forward with adding GHA to mezzio-swoole.

This patch compiles and installs the Swoole extension (at version 4.6.7) for PHP versions 7.3, 7.4, and 8.0.
It is NOT enabled by default; enablement is done selectively, as it is done with sqlsrv (combination of a mods-available INI file, and conditional logic when enabling extensions in the entrypoint).
I have built the container locally, and it builds correctly; more importantly, I then ran it against the mezzio-swoole library to ensure I could run unit tests with it, and tests passed.

At some point, we will likely want to:

- Add a script per extension we want to install manually
- Add a script for each of the various functions/conditionals in the entrypoint to make it more modular

For now, however, this works, and will allow us to move forward with adding GHA to mezzio-swoole.

Signed-off-by: Matthew Weier O'Phinney <[email protected]>
@weierophinney weierophinney added this to the 1.9.0 milestone May 26, 2021
Per shellcheck

Signed-off-by: Matthew Weier O'Phinney <[email protected]>
@weierophinney weierophinney merged commit 37e8bb0 into laminas:1.9.x May 26, 2021
@weierophinney weierophinney deleted the feature/swoole-extension branch May 26, 2021 17:20
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.

1 participant