Skip to content

Wrap gdImageCreateFrom* checks in AC_CACHE_CHECK #14443

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

Merged
merged 1 commit into from
Jun 2, 2024

Conversation

petk
Copy link
Member

@petk petk commented Jun 2, 2024

This enables cross-compiling edge cases to manually override the checks with the php_cv_lib_gd_gdImageCreateFrom* cache variables.

I'll double recheck this if everything is written correctly... Creating dynamic cache variables names might be a bit error prone. Otherwise, it seems to work ok.

This enables cross-compiling edge cases to manually override the checks
with the php_cv_lib_gd_gdImageCreateFrom* cache variables.
@Girgias Girgias merged commit 6c034ee into php:master Jun 2, 2024
11 checks passed
@petk petk deleted the patch-cache-gd branch June 3, 2024 06:20
@orlitzky
Copy link
Contributor

Targeted for 8.3 maybe? :)

It will probably take another release cycle but my goal is to get cross compilation working without having to autoreconf.

@petk
Copy link
Member Author

petk commented Jul 10, 2024

@orlitzky, sure. I think we can do that, yes. Probably then PHP-8.2 since this change got into PHP-8.2. I'll recheck...

petk added a commit to petk/php-src that referenced this pull request Jul 10, 2024
This enables cross-compiling simplifications when using external GD
library as requested in phpGH-14443 using cache variables:

* php_cv_lib_gd_gdImageCreateFromPng
* php_cv_lib_gd_gdImageCreateFromAvif
* php_cv_lib_gd_gdImageCreateFromWebp
* php_cv_lib_gd_gdImageCreateFromJpeg
* php_cv_lib_gd_gdImageCreateFromXpm
* php_cv_lib_gd_gdImageCreateFromBmp
* php_cv_lib_gd_gdImageCreateFromTga

For example:

    ./configure --host=... --build=... --enable-gd --with-external-gd \
        php_cv_lib_gd_gdImageCreateFromPng=yes
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.

4 participants