Re: [bug-mailutils] Status of 3.0

[Jordi Mallach]

Hey!

On Sun, Oct 16, 2011 at 01:18:19PM +0300, Sergey Poznyakoff wrote:
> >  - lber_ldadd.diff: needs to be considered. Fixes builds with
> >    binutils-gold.
> 
> ... and breaks everything else:^)
> /bin/ld: cannot find -lber collect2: ld returned 1 exit status

Well, actually what was needed was -llber, not -lber. :)

> >  - ssl_key_permission_check.diff: needs to be discussed, see comments in
> >    the patch.
> Yes, that's important, but the proposed solution is pretty Debian-specific.
> I'll better see how to make permission checks configurable at run-time.

Yep, that's the point. The patch isn't usable in the current form, as
ideally I'd like to have a configurable check that allows you to configure
which group should be able to read the certificates, ie ssl-certs in
Debian. The patch I'm using is lax but makes things a bit easier. The
correct fix would be to check if the group is the one intended. Thanks for
considering this!

> >  - fix_FTBFS_with_format-security.diff: needs to be fixed, but needs
> >    adapting in the new codebase.
> 
> As far as I can see, that has been fixed already.

Woudn't occurrences like libproto/nntp/nntp_sendline.c (mu_nntp_sendline)
need a similar fix?

mu_nntp_sendline (mu_nntp_t nntp, const char *line)
{
  if (line)
    {
      int status = mu_nntp_writeline (nntp, line);
      if (status)
        return status;
    }
}

> >  - sieve.scm_help_output.diff: _please_ apply this trivial one.
> Applied.

\o/

> >  - mailutils.scm_variables.diff: we discussed that mailutils.scm had some
> >    problems with its path autogeneration. Apparently it hasn't changed in
> >    git. Any chance we can get this addressed with a real fix?
> I have removed mailutils.scm from EXTRA_DIST, this should fix it.

Heh, that surely fixes it. :)

> >  - disable_am_gnu_radius.diff: regarding this one, I proposed addign
> >    radius.m4 to gnulib; was there any progress with that?
> It has already been fixed: bootstrap now downloads the latest version of
> this file directly from the Radius repo (see 608036ea).

Ok, I haven't checked how a recent tarball looks like (where are they?),
but it looks good indeed. Thanks again!

> >  - remove_MAXHOSTNAMELEN.diff: ignore; backport of your Hurd fixes.
> This too has already been fixed.  There's only one instance of that
> constant left, it is in libmailutils/base/hostname.c, where its use is
> intended.

> Let me know if I have missed something.

It looks like that was all. The only outstanding issue would be making the
certificate check a bit flexible.

Thanks Sergey!

Jordi
-- 
Jordi Mallach Pérez  --  Debian developer     http://www.debian.org/
address@hidden     address@hidden     http://www.sindominio.net/
GnuPG public key information available at http://oskuro.net/