search

cyrusimap / cassandane

Other
6 stars 11 forks source link

Required /usr/cyrus better be <something different> #10

Closed brong closed 12 years ago

brong commented 12 years ago

From: Jeroen van Meeuwen (Kolab Systems) Bugzilla-Id: 3552 Version: unspecified Owner: Greg Banks

brong commented 12 years ago

From: Jeroen van Meeuwen (Kolab Systems)

Not knowing whether it is at all possible, the doc/setting_up.txt lists a required /usr/cyrus directory or symbolic link to where a user can install.

Perhaps it is worth considering requiring ~/.cassandane/cyrus-imapd/ instead, or if ~/. doesn't at all translate, maybe /usr/local/cyrus-imapd/?

I would append the '-imapd' in order to allow Cyrus SASL to also be tested through cassandane at some point in the future.

Thanks, in advance, for your thoughts / considerations.

brong commented 12 years ago

From: Greg Banks

Ok, but ~/.cassandane/ is a bit of a problem given that you need to be sudo'd to root or user "cyrus" to run Cassandane. It might be better to add one of a commandline option, a symlink in the Cassandane directory, or a config file in the Cassandane directory. I'd lean toward the config file for two reasons:

a) at some point I want to extend Cassandane to test replication and murder between multiple released versions, which implies having instances with different cyrus base directories, which means some kind of mapping syntax.

b) I want to be able to turn off certain groups of tests if the target Cyrus install is missing certain features, e.g. conversations or annotations, which implies even more state.

brong commented 12 years ago

From: Greg Banks

The following changes since commit fb7ca2c25da0399b53ae771fbfeaf8f55bda89b8:

Bug #3551: correct url for cyrus getuid patch (2011-09-08 10:46:32 +1000)

are available in the git repository at: ssh://git.cyrusimap.org/git/cassandane master

Greg Banks (1): Bug #3552 allow overriding /usr/cyrus in an .ini

Cassandane/Cassini.pm | 86 ++++++++++++++++++++++++++++++++++++++++++++++++ Cassandane/Instance.pm | 14 +++++++- cassandane.ini.example | 54 ++++++++++++++++++++++++++++++ doc/README.deps | 5 +++ doc/setting_up.txt | 14 ++++++++ 5 files changed, 171 insertions(+), 2 deletions(-) create mode 100644 Cassandane/Cassini.pm create mode 100644 cassandane.ini.example