Skip to content

ext/gd/tests/gh17349.phpt: require PNG support#21988

Closed
orlitzky wants to merge 1 commit intophp:PHP-8.3from
orlitzky:gh17349-requires-png
Closed

ext/gd/tests/gh17349.phpt: require PNG support#21988
orlitzky wants to merge 1 commit intophp:PHP-8.3from
orlitzky:gh17349-requires-png

Conversation

@orlitzky
Copy link
Copy Markdown
Contributor

@orlitzky orlitzky commented May 8, 2026

This test loads a PNG image, so if an external libgd is being used, it has to support PNG.

This test loads a PNG image, so if an external libgd is being used, it
has to support PNG.
@devnexen
Copy link
Copy Markdown
Member

devnexen commented May 8, 2026

Thanks, albeit I would say 1 PR is enough to fix gd tests.

@orlitzky
Copy link
Copy Markdown
Contributor Author

orlitzky commented May 8, 2026

Sorry, was in a hurry trying to get the security releases out.

@orlitzky
Copy link
Copy Markdown
Contributor Author

orlitzky commented May 8, 2026

And sorry again in advance for the one needed to handle c1c6520 :)

I'm testing 8.2, 8.3, 8.4, and 8.5 in succession, so not hitting them all at once.

@devnexen
Copy link
Copy Markdown
Member

devnexen commented May 8, 2026

No problem take your time :)

@orlitzky
Copy link
Copy Markdown
Contributor Author

orlitzky commented May 9, 2026

moving to #21987

@orlitzky orlitzky closed this May 9, 2026
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