Skip to content

Conversation

petk
Copy link
Member

@petk petk commented Mar 18, 2024

This introduces a new --disable-configure-command option to optionally suppress the display of the configure command in the phpinfo() output:

./configure --disable-configure-command

When building PHP, many packages have patches to hide the PHP build configure command and its options in the phpinfo() output:

Configure Command: './configure' '--enable-foo' 'CFLAGS=...' '...'

This simplifies the PHP build process for package maintainers who prefer more concise phpinfo() output without detailed build configuration.

The CONFIGURE_COMMAND symbol remains defined in the configuration headers (main/build-defs.h for *nix, main/config.w32.h for Windows) regardless of this option.

By default the configure command and options are displayed as before.

This introduces a new --disable-configure-command option to optionally
suppress the display of the configure command in the phpinfo() output:

    ./configure --disable-configure-command

When building PHP, many packages have patches to hide the PHP build
configure command and its options in the phpinfo() output:

    Configure Command: './configure' '--enable-foo' 'CFLAGS=...' '...'

This simplifies the PHP build process for package maintainers who prefer
more concise phpinfo() output without detailed build configuration.

The CONFIGURE_COMMAND symbol remains defined in the configuration
headers (main/build-defs.h for *nix, main/config.w32.h for Windows)
regardless of this option.

By default the configure command and options are displayed as before.
@devnexen
Copy link
Member

devnexen commented Mar 19, 2024

Makes sense, I might be more in favor of --without-configure-command but that s just personal taste :)

@petk
Copy link
Member Author

petk commented Mar 19, 2024

Makes sense, I might be more in favor of --without-configure-command but that s just personal taste :)

I know. --hide-configure-command :) Issue is that --with/--without is meant for things that require additional system dependency (package). I'll check how this could be made even simpler...

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