Bug 110218

Summary:	Patch for disabling cyrus-imapd sieve support
Product:	Gentoo Linux	Reporter:	Arnaud Launay <asl>
Component:	[OLD] Server	Assignee:	Tobias Scherbaum (RETIRED) <dertobi123>
Status:	RESOLVED FIXED
Severity:	enhancement	CC:	jakub, me, net-mail+disabled
Priority:	High
Version:	unspecified
Hardware:	All
OS:	Linux
Whiteboard:
Package list:		Runtime testing required:	---
Attachments:	Option to disable sieve support in cyrus

Description Arnaud Launay 2005-10-23 05:49:04 UTC

Cyrus may be configured to not support sieve, which lightens the server.

Reproducible: Always
Steps to Reproduce:
1. emerge cyrus-imapd
2.
3.
Actual Results:  
Cyrus gets compiled and installed with sieve support

Expected Results:  
Propose an option to disable sieve support

The patch adds a local flags called "nosieve".

Comment 1 Arnaud Launay 2005-10-23 05:51:55 UTC

Created attachment 71277 [details, diff]
Option to disable sieve support  in cyrus

Patch against cyrus-imapd-2.2.12-r3.ebuild; adds a "nosieve" local use flag to
disable sieve support, installs pam file only if use pam enabled.

Comment 2 Jakub Moc (RETIRED) gentoo-dev

2005-10-23 07:00:40 UTC

Oh no, not yet another evil no* flag. :/

Comment 3 Arnaud Launay 2005-10-23 08:18:46 UTC

I thought about that, but as sieve is historically enabled, it seemed the
appropriate way... If a flag isn't specified, is it enabled or disabled ? Then I
couuld change nosieve to "sieve", and only "-sieve" would drop support...

Hmm, tests seem to show flag's disabled by default...

Comment 4 Tuan Van (RETIRED) gentoo-dev

2006-02-27 10:21:14 UTC

I am going to bump cyrus to 2.3.2 and thinking about adding this to it.
Is there a reason you remove notifyd from SUBDIRS ? NOTIFYD(8) doesn't say anything about it has any relation with sieve.

@@ -226,7 +230,14 @@
 
 src_install() {
 	dodir /usr/bin /usr/lib
-	for subdir in master imap imtest timsieved notifyd sieve; do
+
+	if use nosieve; then
+		SUBDIRS="master imap imtest"
+	else
+		SUBDIRS="master imap imtest timsieved notifyd sieve"
+	fi
+
+	for subdir in ${SUBDIRS}; do
 		make -C "${subdir}" DESTDIR="${D}" install || die "make install failed"
 	done

Comment 5 Arnaud Launay 2006-02-27 10:24:32 UTC

AFAIR, in Cyrus 2.2, notifyd wouldn't be built when sieve was disabled. Haven't tried again with 2.3 anyway.

Comment 6 Benny Pedersen 2007-09-24 11:15:17 UTC

dovecot using sieve use flag, have this in mind :)

Comment 7 Tobias Scherbaum (RETIRED) gentoo-dev

2008-12-29 18:04:23 UTC

Added as USE="sieve" in 2.3.13-r1.