On Saturday, February 16, 2019 12:00:51 PM László Böszörményi wrote:
> Control: tags -1 +patch
>
> On Sat, Feb 16, 2019 at 10:21 AM Scott Kitterman <deb...@kitterman.com>
wrote:
> > It would have been nice if you had investigated the impact of dropping
> > icu-
> > config. The postfix build system uses the presence of icu-config to
> > determine if libicu is present (and thus SMTPUTF8 support should be built
> > in). See #921075.
>
> I did build testing and postfix compiled successfully without any
> warning that an important library for its working might be missing.
> I don't have the expertise to fire up test mail servers to investigate
> its functionality in every aspect.
>
> > In my opinion, postfix is not releasable in its current state. Please put
> > icu-config back or provide a tested alternative to enable postfix to build
> > correctly.
>
> No offense at all, just a note that the fix might take shorter time
> than writing your email. It is attached and you need to add pkg-config
> to your build dependencies.
> Postfix is build tested very closely. Now it builds _without_ the
> NO_EAI flag and links to ICU correctly. Still can't run it (I don't
> have a server nor VM), but if you may give me a test case showing it
> might still not work then I will do everything to fix it.
It doesn't work.
The easiest way to tell is if you build postfix and it detected libicu
correctly, the binary will depend on libicu. From stable:
Depends: libc6 (>= 2.14), libdb5.3, libicu57 (>= 57.1-1~), ...
This can be verified by installing the package (even in a chroot where it
won't run) and checking the defaults:
# postconf | grep utf
smtputf8_autodetect_classes = sendmail, verify
smtputf8_enable = ${{$compatibility_level} < {1} ? {no} : {yes}}
strict_smtputf8 = no
If smtputf8 support is not compiled in, it will be:
# postconf | grep utf
smtputf8_autodetect_classes = sendmail, verify
smtputf8_enable = no
strict_smtputf8 = no
When I built with your patch, I got both indications of no smtputf8 support.
Scott K
