Skip to content

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
@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