Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 110218 - Patch for disabling cyrus-imapd sieve support
Summary: Patch for disabling cyrus-imapd sieve support
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: [OLD] Server (show other bugs)
Hardware: All Linux
: High enhancement (vote)
Assignee: Tobias Scherbaum (RETIRED)
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2005-10-23 05:49 UTC by Arnaud Launay
Modified: 2008-12-29 18:04 UTC (History)
3 users (show)

See Also:
Package list:
Runtime testing required: ---


Attachments
Option to disable sieve support in cyrus (patch-nosieve,2.16 KB, patch)
2005-10-23 05:51 UTC, Arnaud Launay
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
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.