Skip to content

Comments

Autotools: Add libtool --silent option to phpize#15703

Closed
petk wants to merge 2 commits intophp:masterfrom
petk:patch-phpize-libtool-silent
Closed

Autotools: Add libtool --silent option to phpize#15703
petk wants to merge 2 commits intophp:masterfrom
petk:patch-phpize-libtool-silent

Conversation

@petk
Copy link
Member

@petk petk commented Sep 1, 2024

When building PHP extensions with phpize there is a lot of additional log output done by libtool in the make step that cannot be turned off. This syncs the php-src build and phpize by enabling the Libtool --silent option.

This can wait for PHP after 8.4 version, otherwise the default output by make is almost always sufficient. The php-src's configure.ac already has this option set but phpize didn't have it yet.

When building PHP extensions with phpize there is a lot of additional
log output done by libtool in the make step that cannot be turned off.
This syncs the php-src build and phpize by enabling the Libtool --silent
option.
Copy link
Member

@devnexen devnexen left a comment

Choose a reason for hiding this comment

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

lgtm, php 9 milestone perhaps tough ?

@petk petk added this to the PHP 9 milestone Sep 2, 2024
@petk
Copy link
Member Author

petk commented Sep 2, 2024

lgtm, php 9 milestone perhaps tough ?

Yes, agreed. Added.

@petk
Copy link
Member Author

petk commented Feb 23, 2026

In the meantime I've discovered that some packagers have even patches to enable the additional libtool output for php-src. Probably it isn't that problematic to have some additional info when building with phpize. And probably it could be enabled in the configure.ac instead. I'm simply closing this one for now and considering it as a "feature" for phpize.

@petk petk closed this Feb 23, 2026
@petk petk deleted the patch-phpize-libtool-silent branch February 23, 2026 22:13
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