8000 [RFC] Update test environment and work around segfault with Xdebug 3.5.0-dev by clue · Pull Request #276 · clue/framework-x · GitHub
[go: up one dir, main page]

Skip to content

[RFC] Update test environment and work around segfault with Xdebug 3.5.0-dev #276

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

Closed
wants to merge 1 commit into from

Conversation

clue
Copy link
Owner
@clue clue commented Mar 5, 2025

This changeset updates the test environment to work around a segfault with Xdebug 3.5.0-dev. Right now, this causes an unrelated build error for all builds on Windows with PHP 8.2+ (see also clue/reactphp-redis#173).

I wonder how widespread this problem would be exactly, as I don't see anything special that this project would execute (except perhaps for Fibers as of #117?). In either case, we should probably report this upstream to Xdebug and/or shivammathur/setup-php.

Builds on top of #267, #221 and clue/reactphp-redis#173

@clue clue added this to the v0.18.0 milestone Mar 5, 2025
@clue clue requested a review from PaulRotmann March 5, 2025 14:31
@PaulRotmann
Copy link
Collaborator

I agree on reporting this issue. Will wait with merging this PR til we get back an answer.

@clue
Copy link
Owner Author
clue commented Mar 6, 2025

I wonder how widespread this problem would be exactly, as I don't see anything special that this project would execute (except perhaps for Fibers as of #117?). In either case, we should probably report this upstream to Xdebug and/or shivammathur/setup-php.

Reported upstream and confirmed fixed already shivammathur/setup-php#923 (comment) :shipit:

I'll close this one and apply the minimal changeset to apply the CI/CD update in follow-up PR #277 👍

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.

2 participants
0