Skip to content

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

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

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

Conversation

@orlitzky

@orlitzky orlitzky commented May 8, 2026

Copy link
Copy Markdown
Contributor

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

devnexen commented May 8, 2026

Copy link
Copy Markdown
Member

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

@orlitzky

orlitzky commented May 8, 2026

Copy link
Copy Markdown
Contributor Author

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

@orlitzky

orlitzky commented May 8, 2026

Copy link
Copy Markdown
Contributor Author

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

devnexen commented May 8, 2026

Copy link
Copy Markdown
Member

No problem take your time :)

@orlitzky

orlitzky commented May 9, 2026

Copy link
Copy Markdown
Contributor Author

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