On Sat, 16 Feb 2019 06:46:28 -0500 Scott Kitterman <deb...@kitterman.co
m> wrote:
> 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 <debian@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
Hello Scott,
I just tried in a sid chroot with the patch and it seems to me both
things work, so not sure what's going on?
# postconf | grep utf
smtputf8_autodetect_classes = sendmail, verify
smtputf8_enable = ${{$compatibility_level} < {1} ? {no} : {yes}}
strict_smtputf8 = no
# grep icu debian/postfix/DEBIAN/control
Depends: libc6 (>= 2.14), libdb5.3, libicu63 (>= 63.1-1~), libsasl2-2
(>= 2.1.27+dfsg), libssl1.1 (>= 1.1.1), debconf (>= 0.5) | debconf-2.0,
netbase, adduser (>= 3.48), dpkg (>= 1.8.3), lsb-base (>= 3.0-6), ssl-
cert, cpio, e2fsprogs
A build-dependency on pkg-config needs to be added, it's not installed
otherwise when building with pbuilder/sbuild/etc. Could this be the
issue?
--
Kind regards,
Luca Boccassi
signature.asc
Description: This is a digitally signed message part
