Skip to content

Commit 1c21559

Browse files
committed
[skip ci] README: Fix non-existent iconv in macOS instruction
Follows-up 22e444c (GH-18670). There is currently no formula called "iconv". [1][2] ``` $ brew install iconv Warning: No available formula with the name "iconv". Did you mean icon or cconv? ``` There package is called "libiconv". [3] Once installed, `./configure` still fails due to a discovery issue. ``` $ ./configure --enable-debug … checking for libiconv... no configure: error: Please specify the install prefix of iconv with --with-iconv=<DIR> ``` In 2020, as part of the switch from Intel to ARM, Homebrew adopted /opt as the standard location instead of /usr/local. [4][5] Rather than complicating the README with a mandatory `--with-iconv` path (or --without-iconv) for macOS users, improve the discovery to support Homebrew's new location. Place it after /usr/local, so that /usr/local remains the preferred place for system overrides (in case both are present). Place it before /usr, because there may be older system versions of these packages on macOS that would otherwise have precedence, making it impossible to satisfy version requirements. [1]: https://brew.sh/ [2]: https://github.com/Homebrew/homebrew-core/. [3]: https://formulae.brew.sh/formula/libiconv [4]: https://apple.stackexchange.com/a/437622/33762 [5]: https://docs.brew.sh/FAQ#why-is-the-default-installation-prefix-opthomebrew-on-apple-silicon
1 parent 359f442 commit 1c21559

File tree

2 files changed

+2
-2
lines changed

2 files changed

+2
-2
lines changed

README.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -55,7 +55,7 @@ sudo dnf install re2c bison autoconf make libtool ccache libxml2-devel sqlite-de
5555
On MacOS, you can install these using `brew`:
5656

5757
```shell
58-
brew install autoconf bison re2c iconv libxml2 sqlite
58+
brew install autoconf bison re2c libiconv libxml2 sqlite
5959
```
6060

6161
or with `MacPorts`:

build/php.m4

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1796,7 +1796,7 @@ AC_DEFUN([PHP_SETUP_ICONV], [
17961796
17971797
dnl Check external libs for iconv funcs.
17981798
AS_VAR_IF([found_iconv], [no], [
1799-
for i in $PHP_ICONV /usr/local /usr; do
1799+
for i in $PHP_ICONV /usr/local /opt/homebrew/opt/libiconv /usr; do
18001800
if test -r $i/include/gnu-libiconv/iconv.h; then
18011801
ICONV_DIR=$i
18021802
ICONV_INCLUDE_DIR=$i/include/gnu-libiconv

0 commit comments

Comments
 (0)