Skip to content

Ensure MacOS support by tests #539

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

Draft
wants to merge 8 commits into
base: php8
Choose a base branch
from

Conversation

JoshuaBehrens
Copy link
Contributor

These are either necessary steps or helpful changes to let this run on MacOS. I have further changes to do so this is just a draft

@JoshuaBehrens
Copy link
Contributor Author

This error is weird

v8js/v8js/v8js_object_export.cc:678:9: error: ‘php_strtolower’ was not declared in this scope; did you mean ‘php_strtok_r’?

This method exists for over 25 years and the file is included :D
https://github.com/php/php-src/blame/2a794e8205363e0ef5aea8800c8f04b2b381a03a/ext/standard/php_string.h

@JoshuaBehrens JoshuaBehrens force-pushed the feature/build-on-macos-brew branch 7 times, most recently from 28b917c to 3fe1f39 Compare October 26, 2024 23:52
@JoshuaBehrens JoshuaBehrens mentioned this pull request Jan 4, 2025
@JoshuaBehrens JoshuaBehrens force-pushed the feature/build-on-macos-brew branch 11 times, most recently from d2b18ca to 658bd2b Compare May 10, 2025 18:22
Change GitHub actions to include more and more detailed tests

Rename other workflow targets to better explain the differences between the runs
Add more php versions to default build test to ensure complete range is tested
Add macos to self-built tests
Make artifact names unique per step
Extract v8 cache building to run less often to reduce computation costs
Ensure to use right architecture on built-target
Speed up v8 fetch
@JoshuaBehrens JoshuaBehrens force-pushed the feature/build-on-macos-brew branch 2 times, most recently from 77a3680 to af78606 Compare May 10, 2025 22:43
@JoshuaBehrens JoshuaBehrens force-pushed the feature/build-on-macos-brew branch from af78606 to c42f095 Compare May 11, 2025 00:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant