Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

Check Apache HTTP Server command-line utility #14173

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
petk merged 1 commit into php:master from petk:patch-apache2handler-fixes
Jul 8, 2024

Conversation

Copy link
Member

@petk petk commented May 8, 2024

The Apache HTTP server command-line tool (/usr/sbin/apache2) might be part of a separate package, such as apache2-bin or similar. If not installed, the configure script can still find the apxs tool, but previously didn't check for the HTTP server tool separately. Otherwise, configure syntax errors (integer expression expected) are thrown when checking for the Apache version. Version checks have been replaced with Autoconf's AS_VERSION_COMPARE macro.

@@ -58,9 +63,10 @@ if test "$PHP_APXS2" != "no"; then

dnl Test that we're trying to configure with apache 2.x
PHP_AP_EXTRACT_VERSION($APXS_HTTPD)
if test "$APACHE_VERSION" -lt 2000044; then
AS_VERSION_COMPARE([$APACHE_VERSION], [2000044], [
Copy link
Member

@devnexen devnexen May 8, 2024

Choose a reason for hiding this comment

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

off-topic a bit, 2.0.x is really ancient even 2.2.x is.

Copy link
Member Author

Choose a reason for hiding this comment

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

Yes, this should be updated also at some point. Also for Windows. To something like 2.4.

devnexen reacted with thumbs up emoji
Copy link
Member Author

Choose a reason for hiding this comment

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

I'll open a separate discussion for dropping support for EOL Apache branches 2.0 and 2.2. I'm not fully sure yet what all this can impact, but Apache 2.2 is definitely EOL since 2018 and shouldn't be encouraged to use anyway.

Copy link
Member Author

Choose a reason for hiding this comment

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

Apache minimum version bump: #14664
I'm still checking all of this...

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.

MSTM

The Apache HTTP server command-line tool (/usr/sbin/apache2) might be
part of a separate package, such as apache2-bin or similar. If not
installed, the configure script can still find the apxs tool, but
previously didn't check for the HTTP server tool separately. Otherwise,
configure syntax errors (integer expression expected) are thrown when
checking for the Apache version.
@petk petk force-pushed the patch-apache2handler-fixes branch from f571a4a to 08b157e Compare July 8, 2024 15:14
@petk petk merged commit 1f847a0 into php:master Jul 8, 2024
@petk petk deleted the patch-apache2handler-fixes branch July 8, 2024 15:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@devnexen devnexen devnexen approved these changes

Assignees
No one assigned
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

2 participants

AltStyle によって変換されたページ (->オリジナル) /