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

Fix fileinfo support #1179

Merged
merged 9 commits into from
Apr 3, 2024
Merged

Fix fileinfo support #1179

merged 9 commits into from
Apr 3, 2024

Conversation

bgrgicak
Copy link
Collaborator

@bgrgicak bgrgicak commented Apr 2, 2024

Fixes #1127

What is this PR doing?

It enables fileinfo support in kitchen-sink and Node and adds tests for it.

What problem is it solving?

Allows users to use fileinfo.

How is the problem addressed?

By fixing a bug in the build script that prevented fileinfo from being enabled.

Testing Instructions

  • ensure tests pass

@bgrgicak bgrgicak self-assigned this Apr 2, 2024
@bgrgicak bgrgicak added [Type] Bug An existing feature does not function as intended [Feature] PHP.wasm labels Apr 2, 2024
@bgrgicak bgrgicak marked this pull request as ready for review April 2, 2024 08:26
@bgrgicak bgrgicak requested a review from a team April 2, 2024 08:27
@mho22
Copy link
Contributor

mho22 commented Apr 2, 2024

@bgrgicak Thank you, I forgot a lot of things in my previous Pull Request.

Copy link
Collaborator

@adamziel adamziel left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Approving, but merging will require rebuilding WASM since I merged your other WASM PR. Thank you!

@bgrgicak bgrgicak force-pushed the fix/kitchen-sink-fileinfo-support branch from 13cdd83 to 30b4302 Compare April 3, 2024 05:35
@bgrgicak bgrgicak merged commit 2109ce6 into trunk Apr 3, 2024
5 checks passed
@bgrgicak bgrgicak deleted the fix/kitchen-sink-fileinfo-support branch April 3, 2024 05:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[Feature] PHP.wasm [Type] Bug An existing feature does not function as intended
Projects
None yet
Development

Successfully merging this pull request may close these issues.

PHP Class finfo not found
3 participants