[patch] a patch to the previous ebuild

openwebmail-2.51.ebuild.patch (text/plain), 2.32 KB, created by Tomoyuki Sakurai (RETIRED) on 2005-09-08 07:37:18 UTC

Index: openwebmail-2.51.ebuild
===================================================================
--- openwebmail-2.51.ebuild	(revision 53)
+++ openwebmail-2.51.ebuild	(working copy)
@@ -13,11 +13,12 @@
 KEYWORDS="~x86"
 IUSE="spell zlib ssl pam apache2"
 
+DEPEND=""
 # BUG:
-# installation proccess requires no dependency
-# rather, these packages are runtime-dependency
-# should i use RDEPEND, instead of DEPEND?
-DEPEND="dev-lang/perl
+# no httpd other than apache2 on my dev box
+# other httpd, such as lighttpd should work, too
+RDEPEND="${DEPEND}
+	dev-lang/perl
 	perl-core/CGI
 	perl-core/MIME-Base64
 	dev-perl/libnet
@@ -30,12 +31,7 @@
 	ssl? ( >=dev-libs/openssl-0.9.7d
 		>=dev-perl/Net-SSLeay-1.25
 		>=dev-perl/IO-Socket-SSL-0.96 )
-	pam? ( dev-perl/Authen-PAM )"
-
-# BUG:
-# no httpd other than apache2 on my dev box
-# other httpd, such as lighttpd should work, too
-RDEPEND="${DEPEND}
+	pam? ( dev-perl/Authen-PAM )
 	virtual/mta
 	apache2? ( >=net-www/apache-2 )
 	!apache2? ( =net-www/apache-1* )
@@ -59,6 +55,13 @@
 	# BUG:
 	# the webapp has vhost capability built-in and uses suidperl,
 	# it's a bad idea to install it with webapp.eclass
+	# AFAIK (no, i haven't tested this throughly, i'm a happy squirrelmail
+	# user :), this webapp has capabilities to add, remove users(thus, requires
+	# suidperl). multiple installation is something i want to avoid. exactly for
+	# that reason, the upstream implemented vhost capability in it, I beleive.
+	# besides, vhost configuration, for example Postfix, varies depending on
+	# your/sites requirement.
+	# correct me if i'm wrong.
 	# consider to die when vhost USE flag is enabled
 	if ! built_with_use dev-lang/perl perlsuid ;then
 		ewarn "dev-lang/perl needs to be compiled with perlsuid USE flag enabled"
@@ -94,6 +97,13 @@
 	webapp_configfile ${MY_CGIBINDIR}/openwebmail/etc/openwebmail.conf
 	local MY_FILES
 	local MY_FILE
+	# BUG:
+	# cannot think of better idea, suggestion?
+	# VAR=`command_that_will_fail || die` is what i need?
+	# 'find' might fail on Gentoo/FreeBSD, native (BSD version) find doesn't
+	# support '-printf' option, GNU-find does.
+	find cgi-bin/openwebmail/etc -type f -printf '%h/%f\n' | \
+		sed -e 's#^cgi-bin/##' || die "sed failed"
 	MY_FILES=`find cgi-bin/openwebmail/etc -type f -printf '%h/%f\n' | \
 		sed -e 's#^cgi-bin/##'`
 	for MY_FILE in $MY_FILES ; do