Bug#887201: cyrus-common should depend on e2fsprogs explicitly

[Andreas Henriksson]

On Sun, Jan 14, 2018 at 08:03:29PM +0100, Helmut Grohne wrote:
> Package: cyrus-common
[...]
> /usr/lib/cyrus/bin/makedirs contains chattr. According to file it is a POSIX shell script, ASCII text executable
[...]

This file is found at debian/cyrus-makedirs in the source and indeed
uses the chattr command.

The file is renamed/installed via debian/rules as:
        install debian/cyrus-makedirs $(TMPPKG)/usr/lib/cyrus/bin/makedirs

(And then installed in cyrus-common package via debian/cyrus-common.install)

The debian/README.Debian refers to cyrus-makedirs in multiple places
and among others says:
 o You can use /usr/sbin/cyrus-makedirs to generate the needed directories
 [...]
.. but there's so such file shipped. It seems like the cyrus-makedirs
might have been available under it's original name earlier and only
later got renamed/moved into a location where I assume it's executed
via a wrapper command (similar to how 'git foo' works).

Assuming the wrapper command is 'cyrus' (which seems to align well with
a cursory glance at ./debian/cyrus ) the makedirs script might be
invoked  from the debian/cyrus-common.postinst :
        if [ -z "$2" ]; then
            echo -n "cyrus-common: Creating cyrus-imapd directories..."
            cyrus makedirs --cleansquat
            echo "done."
        fi

There are also several other places which references cyrus-makedirs
and even a cyrus-makedirs.8 manpage.....

My conlusion is thus that a dependency seems warranted (along with a
good cleanup), but given that the package is already RC buggy and has
been removed from testing a long time ago and that noone seems to be
interested it might simply be better to ask for the package to be
removed (from unstable).

Would be great to hear from maintainers (or anyone else interested!) about
this....

Regards,
Andreas Henriksson