If a [cyrus murder], [cyrus replica], or [cyrus backup] installation is configured, but its configured prefix doesn't currently exist, the correct behaviour is to use the prefix configured for the [cyrus default] installation. Only fall back to "/usr/cyrus" if nothing is configured!
Fixes a bug where MurderIMAP.xfer_mailbox_altns_unixhs would be unable to figure out the Cyrus version being run as backend2 if cassandane.ini had a prefix configured for [cyrus murder] but there was nothing installed there.
In this case, the rest of Cassandane falls back to using the [cyrus default] installation for murder-specific instances (and only using "/usr/cyrus" if that was not configured), but get_version() didn't know that and was defaulting directly to "/usr/cyrus" instead.
elliefm
commented
2 years ago
If a
[cyrus murder],[cyrus replica], or[cyrus backup]installation is configured, but its configured prefix doesn't currently exist, the correct behaviour is to use the prefix configured for the[cyrus default]installation. Only fall back to "/usr/cyrus" if nothing is configured!Fixes a bug where MurderIMAP.xfer_mailbox_altns_unixhs would be unable to figure out the Cyrus version being run as backend2 if cassandane.ini had a prefix configured for
[cyrus murder]but there was nothing installed there.In this case, the rest of Cassandane falls back to using the
[cyrus default]installation for murder-specific instances (and only using "/usr/cyrus" if that was not configured), butget_version()didn't know that and was defaulting directly to "/usr/cyrus" instead.