Skip to content

Commit 44ed17c

Browse files
authored
Replace configure option --with-openssl-dir (#14028)
This is a leftover from the refactoring of the --with-openssl-dir option that once accepted the path to OpenSSL but wasn't renamed back then. Instead of --with-openssl-dir, SSL support in ext/ftp and ext/mysqlnd can be enabled implicitly when building with ext/openssl enabled (--with-openssl) or explicitly by using new separate configure options --with-ftp-ssl and --with-mysqlnd-ssl.
1 parent f207074 commit 44ed17c

File tree

3 files changed

+22
-7
lines changed

3 files changed

+22
-7
lines changed

UPGRADING.INTERNALS

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -102,6 +102,10 @@ PHP 8.4 INTERNALS UPGRADE NOTES
102102
- The configure option --with-oci8 has been removed.
103103
- The configure option --with-zlib-dir has been removed.
104104
- The configure option --with-kerberos has been removed.
105+
- The configure option --with-openssl-dir has been removed. SSL support in
106+
ext/ftp and ext/mysqlnd is enabled implicitly, when building with
107+
ext/openssl (--with-openssl), or explicitly by using new configure options
108+
--with-ftp-ssl and --with-mysqlnd-ssl.
105109
- COOKIE_IO_FUNCTIONS_T symbol has been removed (use cookie_io_functions_t).
106110
- HAVE_SOCKADDR_UN_SUN_LEN symbol renamed to HAVE_STRUCT_SOCKADDR_UN_SUN_LEN.
107111
- HAVE_UTSNAME_DOMAINNAME symbol renamed to HAVE_STRUCT_UTSNAME_DOMAINNAME.

ext/ftp/config.m4

Lines changed: 7 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -3,22 +3,23 @@ PHP_ARG_ENABLE([ftp],
33
[AS_HELP_STRING([--enable-ftp],
44
[Enable FTP support])])
55

6-
dnl TODO: Rename this option for master.
7-
PHP_ARG_WITH([openssl-dir],
6+
PHP_ARG_WITH([ftp-ssl],
87
[whether to explicitly enable FTP SSL support],
9-
[AS_HELP_STRING([[--with-openssl-dir]],
10-
[FTP: Whether to enable FTP SSL support without ext/openssl])],
8+
[AS_HELP_STRING([--with-ftp-ssl],
9+
[Explicitly enable SSL support in ext/ftp when not building with
10+
ext/openssl. If ext/openssl is enabled at the configure step, SSL is enabled
11+
implicitly.])],
1112
[no],
1213
[no])
1314

1415
if test "$PHP_FTP" = "yes"; then
1516
AC_DEFINE(HAVE_FTP,1,[Whether you want FTP support])
1617
PHP_NEW_EXTENSION(ftp, php_ftp.c ftp.c, $ext_shared)
1718

18-
dnl Empty variable means 'no'
19+
dnl Empty variable means 'no' (for phpize builds).
1920
test -z "$PHP_OPENSSL" && PHP_OPENSSL=no
2021

21-
if test "$PHP_OPENSSL" != "no" || test "$PHP_OPENSSL_DIR" != "no"; then
22+
if test "$PHP_OPENSSL" != "no" || test "$PHP_FTP_SSL" != "no"; then
2223
PHP_SETUP_OPENSSL(FTP_SHARED_LIBADD)
2324
PHP_SUBST(FTP_SHARED_LIBADD)
2425
AC_DEFINE(HAVE_FTP_SSL,1,[Whether FTP over SSL is supported])

ext/mysqlnd/config9.m4

Lines changed: 11 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -6,6 +6,15 @@ PHP_ARG_ENABLE([mysqlnd],
66
[no],
77
[yes])
88

9+
PHP_ARG_WITH([mysqlnd-ssl],
10+
[whether to explicitly enable SSL support in mysqlnd],
11+
[AS_HELP_STRING([--with-mysqlnd-ssl],
12+
[Explicitly enable SSL support in ext/mysqlnd when not building with
13+
ext/openssl. If ext/openssl is enabled at the configure step, SSL is enabled
14+
implicitly.])],
15+
[no],
16+
[no])
17+
918
PHP_ARG_ENABLE([mysqlnd-compression-support],
1019
[whether to enable compressed protocol support in mysqlnd],
1120
[AS_HELP_STRING([--disable-mysqlnd-compression-support],
@@ -32,9 +41,10 @@ if test "$PHP_MYSQLND" != "no" || test "$PHP_MYSQLND_ENABLED" = "yes"; then
3241

3342
AC_DEFINE([MYSQLND_SSL_SUPPORTED], 1, [Enable core mysqlnd SSL code])
3443

44+
dnl Empty variable means 'no' (for phpize builds).
3545
test -z "$PHP_OPENSSL" && PHP_OPENSSL=no
3646

37-
if test "$PHP_OPENSSL" != "no" || test "$PHP_OPENSSL_DIR" != "no"; then
47+
if test "$PHP_OPENSSL" != "no" || test "$PHP_MYSQLND_SSL" != "no"; then
3848
PHP_SETUP_OPENSSL(MYSQLND_SHARED_LIBADD, [AC_DEFINE(MYSQLND_HAVE_SSL,1,[Enable mysqlnd code that uses OpenSSL directly])])
3949
fi
4050

0 commit comments

Comments
 (0)