[pkgsrc/trunk]: pkgsrc/mail/cucipop * makevpopdb(8) does not need to be setgi...

[jlam <jlam%pkgsrc.org@localhost>]

details:   https://anonhg.NetBSD.org/pkgsrc/rev/319a36a70b76
branches:  trunk
changeset: 530243:319a36a70b76
user:      jlam <jlam%pkgsrc.org@localhost>
date:      Tue Jun 19 21:07:42 2007 +0000

description:
* makevpopdb(8) does not need to be setgid anything.  It is supposed to be
  run as "root".

* Remove unnecessary passing of ROOT_USER and ROOT_GROUP to the
  distribution Makefile.  Instead, we simply use BSD_INSTALL_* macros
  to do the installation and use SPECIAL_PERMS to clean up and special
  permissions after the fact.

* Add a TODO section to the package Makefile that highlights the things
  that need to be done to fix this package.

* Fix MESSAGE to refer to ${PREFIX}, not ${LOCALBASE}, for the location
  of the installed files.

Bump PKGREVISION to 3 due to change in permissions for makevpopdb.

diffstat:

 mail/cucipop/MESSAGE          |   6 +++---
 mail/cucipop/Makefile         |  25 ++++++++++++++++++-------
 mail/cucipop/distinfo         |   4 ++--
 mail/cucipop/patches/patch-ad |  21 ++++++++++++++-------
 4 files changed, 37 insertions(+), 19 deletions(-)

diffs (127 lines):

diff -r 3b40d0fe2106 -r 319a36a70b76 mail/cucipop/MESSAGE
--- a/mail/cucipop/MESSAGE      Tue Jun 19 21:06:05 2007 +0000
+++ b/mail/cucipop/MESSAGE      Tue Jun 19 21:07:42 2007 +0000
@@ -1,15 +1,15 @@
 ===========================================================================
-$NetBSD: MESSAGE,v 1.3 2002/09/29 15:16:22 grant Exp $
+$NetBSD: MESSAGE,v 1.4 2007/06/19 21:07:42 jlam Exp $
 
 Please note that this package includes shareware software.
 
 In order to enable the POP service, you should add the following
 line to your /etc/inetd.conf:
 
-  pop3 stream tcp nowait root ${LOCALBASE}/sbin/cucipop cucipop -Y
+  pop3 stream tcp nowait root ${PREFIX}/sbin/cucipop cucipop -Y
 
 And for IPv6:
 
-  pop3 stream tcp6 nowait root ${LOCALBASE}/sbin/cucipop cucipop -Y
+  pop3 stream tcp6 nowait root ${PREFIX}/sbin/cucipop cucipop -Y
 
 ===========================================================================
diff -r 3b40d0fe2106 -r 319a36a70b76 mail/cucipop/Makefile
--- a/mail/cucipop/Makefile     Tue Jun 19 21:06:05 2007 +0000
+++ b/mail/cucipop/Makefile     Tue Jun 19 21:07:42 2007 +0000
@@ -1,8 +1,8 @@
-# $NetBSD: Makefile,v 1.28 2007/06/15 18:15:17 jlam Exp $
+# $NetBSD: Makefile,v 1.29 2007/06/19 21:07:42 jlam Exp $
 #
 
 DISTNAME=      cucipop-1.31
-PKGREVISION=   2
+PKGREVISION=   3
 CATEGORIES=    mail
 MASTER_SITES=  ftp://ftp.informatik.rwth-aachen.de/pub/packages/cucipop/
 
@@ -13,14 +13,25 @@
 MAINTAINER=    pkgsrc-users%NetBSD.org@localhost
 COMMENT=       The Cubic Circle POP3 mail server
 
-
 LICENSE=       cucipop-license
 
-MAKE_ENV+=     ROOT_USER=${ROOT_USER:Q}
-MAKE_ENV+=     ROOT_GROUP=${ROOT_GROUP:Q}
+REQD_DIRS+=    ${PREFIX}/share/cucipop/bulletins
+
+# cucipop(8) needs to be able to create dotlocks in the mail spool
+# directory.  The appropriate permissions should be set on sbin/cucipop
+# so that it can do this.
+#
+SPECIAL_PERMS+=        ${PREFIX}/sbin/cucipop ${ROOT_USER} ${ROOT_GROUP} 2555
 
-REQD_DIRS+=    ${PREFIX}/share/cucipop/bulletins
-SPECIAL_PERMS+=        ${PREFIX}/sbin/cucipop ${ROOT_USER} ${ROOT_GROUP} 2555
+# XXX TODO:
+# XXX
+# XXX (1) sbin/cucipop doesn't need to be setid anything if the mail spool
+# XXX     is world-writable.
+# XXX (2) Needs to honor ${PKG_SYSCONFDIR} for location of vpop.db.
+# XXX     It currently looks in SENDMAILLIB == /etc/mail.
+# XXX (3) It needs to use _PATH_MAILDIR from <paths.h> if available
+# XXX     instead of hardcoding /var/mail.
+#
 
 PKG_OPTIONS_VAR=       PKG_OPTIONS.cucipop
 PKG_SUPPORTED_OPTIONS= db inet6
diff -r 3b40d0fe2106 -r 319a36a70b76 mail/cucipop/distinfo
--- a/mail/cucipop/distinfo     Tue Jun 19 21:06:05 2007 +0000
+++ b/mail/cucipop/distinfo     Tue Jun 19 21:07:42 2007 +0000
@@ -1,4 +1,4 @@
-$NetBSD: distinfo,v 1.8 2006/01/08 15:12:11 joerg Exp $
+$NetBSD: distinfo,v 1.9 2007/06/19 21:07:42 jlam Exp $
 
 SHA1 (cucipop-1.31.tar.gz) = d9b5dc2960e96ad1ee63f0a6f8879165f4477434
 RMD160 (cucipop-1.31.tar.gz) = dbca1b313a755544578f89586c200a0227858f92
@@ -9,7 +9,7 @@
 SHA1 (patch-aa) = baf7a7a262d8d50d404bcc1eed9fe8da72a7b927
 SHA1 (patch-ab) = 3dcbfb7525bd59a1a5ede8cabe286f1bc3704cca
 SHA1 (patch-ac) = 5d768b6b0badf8357bdb6668efb144bc14f98318
-SHA1 (patch-ad) = 2acb395d031e5919cacbcf178d58fe8ec4a75e0b
+SHA1 (patch-ad) = 3755ac6cfd60c192eff2fe406c4ff8e9daba43f9
 SHA1 (patch-ae) = 30715d0e00ed4862601ea0cb69fdb797a9eb234a
 SHA1 (patch-af) = 3521a623741f7dda8d8c837e9dd46e070b1719a4
 SHA1 (patch-ag) = 9c6bb228b5a00ef241cd3edd57a7d523f5474f42
diff -r 3b40d0fe2106 -r 319a36a70b76 mail/cucipop/patches/patch-ad
--- a/mail/cucipop/patches/patch-ad     Tue Jun 19 21:06:05 2007 +0000
+++ b/mail/cucipop/patches/patch-ad     Tue Jun 19 21:07:42 2007 +0000
@@ -1,13 +1,13 @@
-$NetBSD: patch-ad,v 1.5 2003/07/15 17:32:33 jmmv Exp $
+$NetBSD: patch-ad,v 1.6 2007/06/19 21:07:42 jlam Exp $
 
---- Makefile.orig      Tue May 12 17:09:14 1998
+--- Makefile.orig      1998-05-12 17:09:14.000000000 -0400
 +++ Makefile
 @@ -2,3 +2,3 @@
  
 -BASENAME= /usr
 +BASENAME= ${PREFIX}
  
-@@ -12,7 +12,7 @@
+@@ -12,7 +12,7 @@ GCC_WARNINGS = -O2 -pedantic -Wreturn-ty
  
 -CFLAGS        = -O -DUSE_DB #$(GCC_WARNINGS)
 -LDFLAGS = -lcrypt -ldb2
@@ -18,10 +18,17 @@
 -CUCIPOPLIB=/var/lib/cucipop
 +CUCIPOPLIB=${PREFIX}/share/cucipop
  CUCIPOPBULLETINS=$(CUCIPOPLIB)/bulletins
-@@ -24,4 +24,4 @@
+@@ -24,6 +24,2 @@ MANDIR=$(BASENAME)/man/man8
  
 -INSTALL=install -o root -m
 -BINPERM=02755 -s -g mail
-+INSTALL=install -c -o ${ROOT_USER} -m
-+BINPERM=02755 -s -g ${ROOT_GROUP}
- REGPERM=0644
+-REGPERM=0644
+-
+ #
+@@ -70,4 +66,4 @@ $(MD5_OBJ): md5/md5c.c md5/md5.h
+ install: $(BINS) $(MANS)
+-      $(INSTALL) $(BINPERM) $(BINS) $(BINDIR)
+-      $(INSTALL) $(REGPERM) $(MANS) $(MANDIR)
++      $(BSD_INSTALL_PROGRAM) $(BINS) $(BINDIR)
++      $(BSD_INSTALL_DATA) $(MANS) $(MANDIR)
+       mkdir $(CUCIPOPLIB) 2>/dev/null; exit 0