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

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

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