Bug#545512: cyrus-common-2.2: unnecessarily runs cyrus-makedirs on all upgrades

[Sven Mueller]

Ross Boylan schrieb:
> On Mon, 2009-09-07 at 13:13 -0400, Noah Meyerhans wrote:
>> Package: cyrus-common-2.2
>> Version: 2.2.13-10+etch2
>> Severity: normal
>>
>> cyrus-common-2.2.postinst runs cyrus-makedirs on all package updates,
>> even when it's not necessary.  In the case of the recent security update
> This is the same as bug 404446, which was marked as fixed.  I can
> confirm that it is not fixed; I thought I had done something to try to
> keep the old bug open, but it seems not.

Well, it was not exactly fixed when the changelog says it was:
What was done back then is that the old recursive chown/chmod was
replaced by a "find", which already greatly improved performance of
makedirs. Later changes replaced the "find --exec" with "find --print0|
xargs -0" which improved performance even more.

However, I just changed the relevant postinst script so that it only
runs cyrus-makedirs on fresh installs and when the new version is a
different upstream version then the package that is upgraded. I'm not
comfortable with running it more seldom than that.

Regards,
Sven