Skip to content

Autotools: Remove dependency related errors #15344

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
Aug 11, 2024

Conversation

petk
Copy link
Member

@petk petk commented Aug 11, 2024

  • ext/dom
  • ext/xsl

These use the PHP_ADD_EXTENSION_DEP macro which throws error when one of the dependencies is disabled or not configured properly.

For example:

./configure --disable-all --enable-dom

or

./configure --disable-all --with-xsl

Will throw default PHP dependency error info, when using PHP_ADD_EXTENSION_DEP.

These errors were once done when PHP_ADD_EXTENSION_DEP macro wasn't yet available.

- ext/dom
- ext/xsl

These use the PHP_ADD_EXTENSION_DEP macro which throws error when one of
the dependencies is disabled or not configured properly.

For example:
    ./configure --disable-all --enable-dom

or

    ./configure --disable-all --with-xsl

Will throw default PHP dependency error info, when using
PHP_ADD_EXTENSION_DEP.

These errors were once done when PHP_ADD_EXTENSION_DEP macro wasn't yet
available.
@petk petk merged commit 687eb91 into php:master Aug 11, 2024
10 checks passed
@petk petk deleted the patch-m4-ext-dependency-errors branch August 11, 2024 15:39
petk added a commit to petk/php-src that referenced this pull request Aug 11, 2024
Follow-up of phpGH-15344 (687eb91)

This removes the customized error messages in PDO extensions when PDO is
not enabled (--disable-all or --disable-pdo) in favor of the default
error done by PHP_ADD_EXTENSION_DEP.
petk added a commit to petk/php-src that referenced this pull request Aug 11, 2024
Follow-up of phpGH-15344 (687eb91)

This removes the customized error messages in PDO extensions when PDO is
not enabled (--disable-all or --disable-pdo) in favor of the default
error done by PHP_ADD_EXTENSION_DEP.
petk added a commit that referenced this pull request Aug 11, 2024
Follow-up of GH-15344 (687eb91)

This removes the customized error messages in PDO extensions when PDO is
not enabled (--disable-all or --disable-pdo) in favor of the default
error done by PHP_ADD_EXTENSION_DEP.
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