Re: [patch] time/anacron spooldir location

[Andreas Kusalananda Kähäri <andreas.kahari%bils.se@localhost>]

On Thu, May 07, 2015 at 05:56:13PM +0200, Joerg Sonnenberger wrote:
> On Thu, May 07, 2015 at 12:40:04PM +0200, Andreas Kusalananda Kähäri wrote:
> > I don't know if there's some policy against using ${PREFIX}/var/spool in
> > pkgsrc, but here's a simple patch for anacron that makes it work for me
> > (see attachment).
> 
> Yes, it should be VARBASE to be an actual improvement.
> 
> Joerg
> 

Thanks. Fixed. Tested. Attached.

-- 
:: Andreas Kusalananda Kähäri, Bioinformatics Developer, BILS,
:: Uppsala University, Sweden
::----------------------------------------------------------------------

Index: Makefile
===================================================================
RCS file: /Volumes/NetBSD-pkgsrc/pkgsrc-cvs/pkgsrc/time/anacron/Makefile,v
retrieving revision 1.14
diff -u -r1.14 Makefile
--- Makefile	21 Nov 2013 23:50:05 -0000	1.14
+++ Makefile	7 May 2015 17:23:52 -0000
@@ -2,7 +2,7 @@
 #
 
 DISTNAME=	anacron-2.3
-PKGREVISION=	2
+PKGREVISION=	3
 CATEGORIES=	time
 MASTER_SITES=	${MASTER_SITE_SOURCEFORGE:=anacron/}
 
@@ -29,7 +29,8 @@
 SUBST_MESSAGE.man=	Setting up manpages.
 SUBST_STAGE.man=	pre-configure
 SUBST_FILES.man=	anacrontab.5 anacron.8
-SUBST_SED.man=		-e 's|/etc/anacrontab|${PKG_SYSCONFDIR}/anacrontab|g'
+SUBST_SED.man=		-e 's|/etc/anacrontab|${PKG_SYSCONFDIR}/anacrontab|g' \
+			-e 's|/var/spool/anacron|${VARBASE}/spool/anacron|g'
 
 MAKE_FLAGS+=		ANACRONTAB='${PKG_SYSCONFDIR}/anacrontab'
 MAKE_FLAGS+=		BINDIR='${PREFIX}/sbin'
@@ -40,7 +41,7 @@
 MAKE_FLAGS+=		MANDIR='${PREFIX}/man'
 MAKE_FLAGS+=		PREFIX=${PREFIX:Q}
 MAKE_FLAGS+=		SHELL=${SH:Q}
-MAKE_FLAGS+=		SPOOLDIR='/var/spool/anacron'
+MAKE_FLAGS+=		SPOOLDIR='${VARBASE}/spool/anacron'
 
 EGDIR=			${PREFIX}/share/examples/anacron
 .if exists(${FILESDIR}/anacrontab.${OPSYS})
@@ -49,7 +50,7 @@
 CONF_FILES_MODE=	0600
 .endif
 
-OWN_DIRS=		/var/spool/anacron
+OWN_DIRS=		${VARBASE}/spool/anacron
 
 RCD_SCRIPTS=		anacron