[gentoo-commits] gentoo-x86 commit in sys-process/fcron: metadata.xml ChangeLog fcron-3.0.6-r3.ebuild

[Justin Lecher (jlec)]

jlec        12/08/30 06:42:46

  Modified:             metadata.xml ChangeLog fcron-3.0.6-r3.ebuild
  Log:
  sys-process/fcron: User src_prepare instead of src_unpack; indent input redirections; don't use ewarn for blank lines; use defualt src_ompile(); fix potential buffer overflow
  
  (Portage version: 2.2.0_alpha123/cvs/Linux x86_64)

Revision  Changes    Path
1.8                  sys-process/fcron/metadata.xml

file : http://sources.gentoo.org/viewvc.cgi/gentoo-x86/sys-process/fcron/metadata.xml?rev=1.8&view=markup
plain: http://sources.gentoo.org/viewvc.cgi/gentoo-x86/sys-process/fcron/metadata.xml?rev=1.8&content-type=text/plain
diff : http://sources.gentoo.org/viewvc.cgi/gentoo-x86/sys-process/fcron/metadata.xml?r1=1.7&r2=1.8

Index: metadata.xml
===================================================================
RCS file: /var/cvsroot/gentoo-x86/sys-process/fcron/metadata.xml,v
retrieving revision 1.7
retrieving revision 1.8
diff -u -r1.7 -r1.8
--- metadata.xml	29 Aug 2012 17:38:40 -0000	1.7
+++ metadata.xml	30 Aug 2012 06:42:46 -0000	1.8
@@ -1,34 +1,32 @@
 <?xml version="1.0" encoding="UTF-8"?>
 <!DOCTYPE pkgmetadata SYSTEM "http://www.gentoo.org/dtd/metadata.dtd">
 <pkgmetadata>
-  <herd>cron</herd>
-  <maintainer>
-    <email>wschlich@g.o</email>
-    <name>Wolfram Schlich</name>
-    <description>Primary maintainer</description>
-  </maintainer>
-  <maintainer>
-    <email>flameeyes@g.o</email>
-  </maintainer>
-  <use>
-    <flag name="debug">
+	<herd>cron</herd>
+	<maintainer>
+		<email>wschlich@g.o</email>
+		<name>Wolfram Schlich</name>
+		<description>Primary maintainer</description>
+	</maintainer>
+	<maintainer>
+		<email>flameeyes@g.o</email>
+	</maintainer>
+	<use>
+		<flag name="debug">
       Enable debug code and output. Since version 3.0.5 this will no
       longer force foreground execution, and fcron will be able to run
       as a service properly.
     </flag>
-
-    <flag name="pam">
+		<flag name="pam">
       Enable PAM support for fcron. This means that fcron will pass
       through the "fcron" stack before executing the jobs, and
       fcrontab will use the "fcrontab" stack to authenticate the user
       before editing its crontab file.
     </flag>
-
-    <flag name="system-crontab">
+		<flag name="system-crontab">
       Set up fcron to respect /etc/crontab and /etc/cron.d. If this
       flag is disabled, /etc/cron.d and /etc/crontab will be ignored,
       but /etc/cron.{hourly,daily,weekly,monthly} will still be
       respected.
     </flag>
-  </use>
+	</use>
 </pkgmetadata>



1.95                 sys-process/fcron/ChangeLog

file : http://sources.gentoo.org/viewvc.cgi/gentoo-x86/sys-process/fcron/ChangeLog?rev=1.95&view=markup
plain: http://sources.gentoo.org/viewvc.cgi/gentoo-x86/sys-process/fcron/ChangeLog?rev=1.95&content-type=text/plain
diff : http://sources.gentoo.org/viewvc.cgi/gentoo-x86/sys-process/fcron/ChangeLog?r1=1.94&r2=1.95

Index: ChangeLog
===================================================================
RCS file: /var/cvsroot/gentoo-x86/sys-process/fcron/ChangeLog,v
retrieving revision 1.94
retrieving revision 1.95
diff -u -r1.94 -r1.95
--- ChangeLog	29 Aug 2012 17:48:36 -0000	1.94
+++ ChangeLog	30 Aug 2012 06:42:46 -0000	1.95
@@ -1,6 +1,12 @@
 # ChangeLog for sys-process/fcron
 # Copyright 1999-2012 Gentoo Foundation; Distributed under the GPL v2
-# $Header: /var/cvsroot/gentoo-x86/sys-process/fcron/ChangeLog,v 1.94 2012/08/29 17:48:36 flameeyes Exp $
+# $Header: /var/cvsroot/gentoo-x86/sys-process/fcron/ChangeLog,v 1.95 2012/08/30 06:42:46 jlec Exp $
+
+  30 Aug 2012; Justin Lecher <jlec@g.o> fcron-3.0.6-r3.ebuild,
+  +files/fcron-3.0.6-buffer-overflow.patch, metadata.xml:
+  User src_prepare instead of src_unpack; indent input redirections; don't use
+  ewarn for blank lines; use defualt src_ompile(); fix potential buffer
+  overflow
 
   29 Aug 2012; Diego E. Pettenò <flameeyes@g.o> fcron-3.0.6-r3.ebuild:
   Fix comment in pam.d file.



1.3                  sys-process/fcron/fcron-3.0.6-r3.ebuild

file : http://sources.gentoo.org/viewvc.cgi/gentoo-x86/sys-process/fcron/fcron-3.0.6-r3.ebuild?rev=1.3&view=markup
plain: http://sources.gentoo.org/viewvc.cgi/gentoo-x86/sys-process/fcron/fcron-3.0.6-r3.ebuild?rev=1.3&content-type=text/plain
diff : http://sources.gentoo.org/viewvc.cgi/gentoo-x86/sys-process/fcron/fcron-3.0.6-r3.ebuild?r1=1.2&r2=1.3

Index: fcron-3.0.6-r3.ebuild
===================================================================
RCS file: /var/cvsroot/gentoo-x86/sys-process/fcron/fcron-3.0.6-r3.ebuild,v
retrieving revision 1.2
retrieving revision 1.3
diff -u -r1.2 -r1.3
--- fcron-3.0.6-r3.ebuild	29 Aug 2012 17:48:36 -0000	1.2
+++ fcron-3.0.6-r3.ebuild	30 Aug 2012 06:42:46 -0000	1.3
@@ -1,6 +1,6 @@
 # Copyright 1999-2012 Gentoo Foundation
 # Distributed under the terms of the GNU General Public License v2
-# $Header: /var/cvsroot/gentoo-x86/sys-process/fcron/fcron-3.0.6-r3.ebuild,v 1.2 2012/08/29 17:48:36 flameeyes Exp $
+# $Header: /var/cvsroot/gentoo-x86/sys-process/fcron/fcron-3.0.6-r3.ebuild,v 1.3 2012/08/30 06:42:46 jlec Exp $
 
 EAPI=4
 
@@ -35,9 +35,8 @@
 	[[ ${rootgroup} ]] || rootgroup=root
 }
 
-src_unpack() {
-	unpack ${A}
-	cd "${S}"
+src_prepare() {
+	epatch "${FILESDIR}"/${P}-buffer-overflow.patch
 
 	# respect LDFLAGS
 	sed -i "s:\(@LIBS@\):\$(LDFLAGS) \1:" Makefile.in || die "sed failed"
@@ -76,7 +75,7 @@
 }
 
 src_compile() {
-	emake || die "make failed"
+	default
 
 	# bug #216460
 	sed -i \
@@ -135,17 +134,17 @@
 	fperms 0640 /etc/fcron/fcron.{allow,deny,conf}
 
 	pamd_mimic system-services fcron auth account session
-	cat - > "${T}"/fcrontab.pam <<EOF
-# Don't ask for the user's password; fcrontab will only allow to
-# change user if running as root.
-auth		sufficient		pam_permit.so
-
-# Still use the system-auth stack for account and session as the
-# sysadmin might have set up stuff properly, and also avoids
-# sidestepping limits (since fcrontab will run \$EDITOR).
-account		include			system-auth
-session		include			system-auth
-EOF
+	cat > "${T}"/fcrontab.pam <<- EOF
+	# Don't ask for the user's password; fcrontab will only allow to
+	# change user if running as root.
+	auth		sufficient		pam_permit.so
+
+	# Still use the system-auth stack for account and session as the
+	# sysadmin might have set up stuff properly, and also avoids
+	# sidestepping limits (since fcrontab will run \$EDITOR).
+	account		include			system-auth
+	session		include			system-auth
+	EOF
 	newpamd "${T}"/fcrontab.pam fcrontab
 
 	newinitd "${FILESDIR}"/fcron.init.3 fcron
@@ -170,9 +169,9 @@
 	elog "  emerge --config ${CATEGORY}/${PN}"
 	elog "to configure the proper settings."
 	if ! use system-crontab; then
-		ewarn ""
+		echo ""
 		ewarn "Remember that fcron will *not* use /etc/cron.d in this configuration"
-		ewarn ""
+		echo ""
 	fi
 }
 
@@ -192,11 +191,11 @@
 	else
 		elog "This is going to set up fcron to set up a default systab that"
 		elog "executes /etc/cron.{hourly,daily,weekly,monthly}."
-		fcrontab -u systab - <<EOF
-0  *  * * *      rm -f /var/spool/cron/lastrun/cron.hourly
-1  3  * * *      rm -f /var/spool/cron/lastrun/cron.daily
-15 4  * * 6      rm -f /var/spool/cron/lastrun/cron.weekly
-30 5  1 * *      rm -f /var/spool/cron/lastrun/cron.monthly
-EOF
+		fcrontab -u systab - <<- EOF
+		0  *  * * *      rm -f /var/spool/cron/lastrun/cron.hourly
+		1  3  * * *      rm -f /var/spool/cron/lastrun/cron.daily
+		15 4  * * 6      rm -f /var/spool/cron/lastrun/cron.weekly
+		30 5  1 * *      rm -f /var/spool/cron/lastrun/cron.monthly
+		EOF
 	fi
 }