Viktor Dukhovni:
> Change the above to:
> 
> # There's no need to mention "-I/usr/local/include" multiple times.
> #
> set -- '-DUSE_TLS -I/usr/local/opt/openssl/include'
> set -- "$@" '-I/usr/local/opt/icu4c/include'

On 19 Nov 2018, at 19:33, Wietse Venema <wie...@porcupine.org> wrote:
> Please don't hard-code ICU pathnames. EAI support is
> enabled by default when the "icu-config" command is found.
> 
> Postfix uses 'icu-config --cppflags' to locate header files, and
> 'icu-config --ldflags' to locate libraries.
> 
> Now, if Apple decided not to install "icu-config", then they are
> not helping developers.

Robert Chalmers:
> sorry Wietse, you?ve lost me there.
> 
> What does your statement mean in relation to the commands of Viktor
> that now build it?

It says please don't hard-code ICU pathnames.

        Wietse

Reply via email to