add IUSE (bug #47358); don't rdepend on sed; error check sed; tidy

2 files changed, 28 insertions, 19 deletions

diff --git a/net-misc/rsync/ChangeLog b/net-misc/rsync/ChangeLog
index a3f1daffefef..75c274f44915 100644
--- a/net-misc/rsync/ChangeLog
+++ b/net-misc/rsync/ChangeLog
@@ -1,6 +1,9 @@
 # ChangeLog for net-misc/rsync
-# Copyright 2002-2003 Gentoo Technologies, Inc.; Distributed under the GPL v2
-# $Header: /var/cvsroot/gentoo-x86/net-misc/rsync/ChangeLog,v 1.36 2004/01/03 18:00:58 vapier Exp $
+# Copyright 2002-2004 Gentoo Technologies, Inc.; Distributed under the GPL v2
+# $Header: /var/cvsroot/gentoo-x86/net-misc/rsync/ChangeLog,v 1.37 2004/04/12 00:07:05 mr_bones_ Exp $
+
+  11 Apr 2004; Michael Sterrett <mr_bones_@gentoo.org> rsync-2.6.0.ebuild:
+  add IUSE (bug #47358); don't rdepend on sed; error check sed; tidy
 
 *rsync-2.6.0 (03 Jan 2004)
 
diff --git a/net-misc/rsync/rsync-2.6.0.ebuild b/net-misc/rsync/rsync-2.6.0.ebuild
index 6d7fdb58146e..ffca598d8631 100644
--- a/net-misc/rsync/rsync-2.6.0.ebuild
+++ b/net-misc/rsync/rsync-2.6.0.ebuild
@@ -1,6 +1,6 @@
-# Copyright 1999-2003 Gentoo Technologies, Inc.
+# Copyright 1999-2004 Gentoo Technologies, Inc.
 # Distributed under the terms of the GNU General Public License v2
-# $Header: /var/cvsroot/gentoo-x86/net-misc/rsync/rsync-2.6.0.ebuild,v 1.3 2004/03/02 16:44:52 iggy Exp $
+# $Header: /var/cvsroot/gentoo-x86/net-misc/rsync/rsync-2.6.0.ebuild,v 1.4 2004/04/12 00:07:05 mr_bones_ Exp $
 
 inherit eutils flag-o-matic gcc
 
@@ -12,43 +12,49 @@ SRC_URI="http://rsync.samba.org/ftp/rsync/${P}.tar.gz
 LICENSE="GPL-2"
 SLOT="0"
 KEYWORDS="x86 ppc sparc mips alpha hppa amd64 ia64 ppc64 s390"
+IUSE="build static"
 
-DEPEND="virtual/glibc
-	>=sys-apps/sed-4
+RDEPEND="virtual/glibc
 	!build? ( >=dev-libs/popt-1.5 )"
+DEPEND="${RDEPEND}
+	>=sys-apps/sed-4"
 
 src_unpack() {
-	unpack ${P}.tar.gz
+	unpack "${P}.tar.gz"
 	cd ${S}
-	epatch ${DISTDIR}/${PN}-2.5.6-proxy-auth-1.patch
+	epatch "${DISTDIR}/${PN}-2.5.6-proxy-auth-1.patch"
 
 	# change confdir to /etc/rsync rather than just /etc (the --sysconfdir
 	# configure option doesn't work
-	sed	-i 's|/etc/rsyncd.conf|/etc/rsync/rsyncd.conf|g' rsync.h
+	sed	-i \
+		-e 's|/etc/rsyncd.conf|/etc/rsync/rsyncd.conf|g' rsync.h \
+			|| die "sed rsync.h failed"
 	# yes, updating the man page is very important.
-	sed -i 's|/etc/rsyncd|/etc/rsync/rsyncd|g' rsyncd.conf.5
+	sed -i \
+		-e 's|/etc/rsyncd|/etc/rsync/rsyncd|g' rsyncd.conf.5 \
+			|| die "sed rsyncd.conf.5 failed"
 }
 
 src_compile() {
 	[ "`gcc-version`" == "2.95" ] && append-ldflags -lpthread
 	use static && append-ldflags -static
-	econf `use_with build included-popt` || die
-	emake || die
+	econf $(use_with build included-popt) || die
+	emake || die "emake failed"
 }
 
 src_install() {
-	make install DESTDIR=${D} || die
-	insinto /etc/conf.d && newins ${FILESDIR}/rsyncd.conf.d rsyncd
-	exeinto /etc/init.d && newexe ${FILESDIR}/rsyncd.init.d rsyncd
-	if [ ! `use build` ] ; then
+	make DESTDIR="${D}" install || die "make install failed"
+	insinto /etc/conf.d && newins "${FILESDIR}/rsyncd.conf.d" rsyncd
+	exeinto /etc/init.d && newexe "${FILESDIR}/rsyncd.init.d" rsyncd
+	if ! use build ; then
 		dodir /etc/rsync
-		dodoc COPYING NEWS OLDNEWS README TODO tech_report.tex
+		dodoc NEWS OLDNEWS README TODO tech_report.tex
 		if [ ! -e /etc/rsync/rsyncd.conf ] ; then
 			insinto /etc/rsync
-			doins ${FILESDIR}/rsyncd.conf
+			doins "${FILESDIR}/rsyncd.conf"
 		fi
 	else
-		rm -rf ${D}/usr/share
+		rm -rf "${D}/usr/share"
 	fi
 }