Re: pkg/48913: "libmilter" and "sendmail" can no longer be built as an unprivileged user

[jnemeth%CornerstoneService.ca@localhost (John Nemeth)]

The following reply was made to PR pkg/48913; it has been noted by GNATS.

From: jnemeth%CornerstoneService.ca@localhost (John Nemeth)
To: gnats-bugs%NetBSD.org@localhost, gnats-admin%NetBSD.org@localhost, 
pkgsrc-bugs%NetBSD.org@localhost,
        tron%zhadum.org.uk@localhost
Cc: 
Subject: Re: pkg/48913: "libmilter" and "sendmail" can no longer be built as an 
unprivileged user
Date: Wed, 18 Jun 2014 22:10:02 -0700

 On Jun 18,  8:30am, Makoto Fujiwara wrote:
 }
 } The following reply was made to PR pkg/48913; it has been noted by GNATS.
 } 
 } From: Makoto Fujiwara <makoto%ki.nu@localhost>
 } Subject: Re: pkg/48913: "libmilter" and "sendmail" can no longer be built as 
an unprivileged user
 } Date: Wed, 18 Jun 2014 17:28:55 +0900
 } 
 }  I think the part of patches/patch-aa is to be reverted.
 
      No.
 
 }  Index: mail/sendmail/Makefile
 }  ===================================================================
 }  RCS file: /cvs/cvsroot/pkgsrc/mail/sendmail/Makefile,v
 }  retrieving revision 1.122
 }  diff -u -r1.122 Makefile
 }  --- mail/sendmail/Makefile  15 Jun 2014 21:04:30 -0000      1.122
 }  +++ mail/sendmail/Makefile  18 Jun 2014 08:24:19 -0000
 }  @@ -27,6 +27,13 @@
 }   MAKE_DIRS_PERMS+=  /${VARBASE}/spool/mqueue ${REAL_ROOT_USER} 
${REAL_ROOT_GROUP} 0700
 }   
 }   SPECIAL_PERMS+=            libexec/sendmail/sendmail ${REAL_ROOT_USER} 
smmsp 2555
 }  +SPECIAL_PERMS+=            share/misc/helpfile     bin bin 0444
 }  +SPECIAL_PERMS+=            sbin/editmap            bin bin 0555
 }  +SPECIAL_PERMS+=            sbin/mailstats          bin bin 0555
 }  +SPECIAL_PERMS+=            sbin/makemap            bin bin 0555
 }  +SPECIAL_PERMS+=            sbin/praliases          bin bin 0555
 }  +SPECIAL_PERMS+=            libexec/sendmail/smrsh  bin bin 0555
 }  +SPECIAL_PERMS+=            bin/vacation            bin bin 0555
 
      There is nothing special about these files, therefore there
 is no reason to give them the SPECIAL_PERMS treatment.
 
 }  Index: mail/sendmail/patches/patch-aa
 }  ===================================================================
 }  RCS file: /cvs/cvsroot/pkgsrc/mail/sendmail/patches/patch-aa,v
 }  retrieving revision 1.21
 }  diff -u -r1.21 patch-aa
 }  --- mail/sendmail/patches/patch-aa  15 Jun 2014 20:48:50 -0000      1.21
 }  +++ mail/sendmail/patches/patch-aa  18 Jun 2014 08:24:19 -0000
 }  @@ -1,8 +1,8 @@
 }  -$NetBSD: patch-aa,v 1.21 2014/06/15 20:48:50 jnemeth Exp $
 }  +$NetBSD: patch-aa,v 1.20 2013/03/23 10:06:12 joerg Exp $
 }   
 }  ---- devtools/OS/NetBSD.orig        2014-03-06 17:31:29.000000000 +0000
 }  +--- devtools/OS/NetBSD.orig        2004-06-16 17:50:00.000000000 +0000
 }   +++ devtools/OS/NetBSD
 }  -@@ -5,15 +5,11 @@ dnl      Place personal settings in devtools/
 }  +@@ -5,15 +5,16 @@ dnl      Place personal settings in devtools/
 }    
 }    define(`confMAPDEF', `-DNEWDB -DNIS -DMAP_REGEX')
 }    define(`confLIBS', `-lutil')
 }  @@ -14,6 +14,11 @@
 }   -define(`confUBINGRP', `wheel')
 }   -define(`confMANOWN', `root')
 }   -define(`confMANGRP', `wheel')
 }  ++define(`confSBINGRP', `${BSD_BINGRP}')
 }  ++define(`confUBINOWN', `${BSD_BINOWN}')
 }  ++define(`confUBINGRP', `${BSD_BINGRP}')
 }  ++define(`confMANOWN', `${BSD_MANOWN}')
 }  ++define(`confMANGRP', `${BSD_MANGRP}')
 
      There is really no point in this, since packages can't be
 setting file ownership at build time.
 
 }-- End of excerpt from Makoto Fujiwara