Skip to content

Add cross-compiling argument to AC_RUN_IFELSE #12966

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

petk
Copy link
Member

@petk petk commented Dec 17, 2023

Autoconf emits a warning if 3rd argument in AC_RUN_IFELSE is empty:

autoconf -Wall
configure.ac:1611: warning: AC_RUN_IFELSE called without default to allow cross compiling
./lib/autoconf/general.m4:2981: AC_RUN_IFELSE is expanded from...
lib/m4sugar/m4sh.m4:699: AS_IF is expanded from...
build/ax_check_compile_flag.m4:39: AX_CHECK_COMPILE_FLAG is expanded from...
configure.ac:1611: the top level

Additionally, call is wrapped in the AC_CACHE_CHECK with php_cv_* cache variable name according to the docs.

Related to #12642

Copy link
Member

@iluuu1994 iluuu1994 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you for fixing this!

@petk
Copy link
Member Author

petk commented Dec 21, 2023

Should we maybe target here PHP-8.1 or PHP-8.2 branch?

@iluuu1994
Copy link
Member

I think backporting this to 8.2 certainly makes sense, since this code is new.

Autotools emits warning if 3rd argument is empty. Call is wrapped in the
AC_CACHE_CHECK with php_cv_* cache variable name according to the docs.
@petk petk force-pushed the patch-autoconf-cross-compile branch from 7c15d2b to bcab663 Compare December 22, 2023 15:13
@petk petk changed the base branch from master to PHP-8.2 December 22, 2023 15:13
@petk petk closed this in de5557b Dec 22, 2023
@petk petk deleted the patch-autoconf-cross-compile branch December 22, 2023 15:37
@petk
Copy link
Member Author

petk commented Dec 22, 2023

Merged to PHP-8.2 to future proof Autoconf generation, just in case.

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