Re: mail/postfix: insufficient permissions for var/spool/postfix/...

[Frédéric Fauberteau <triaxx%NetBSD.org@localhost>]

Le 2020-01-26 22:25, Matthias Ferdinand a écrit :
> Hi,
>
> if I remove the var/spool/postfix/ subdirs from PLIST and leave only
>     @pkgdir var/spool/postfix
> postfix starts fine after package install. So I propose the attached
> patch to the PLIST file.
>
> Regards
> Matthias

Hi Matthias,

Thanks for providing a patch to fix this issue!
The only problem I see is that manually edition of PLIST could break the 
'print-PLIST' command. I suggest to complete your patch by removing the 
subdirectories of /var/spool/postfix directly from DESTDIR.

Index: Makefile
===================================================================
RCS file: /cvsroot/pkgsrc/mail/postfix/Makefile,v
retrieving revision 1.315
diff -u -r1.315 Makefile
--- Makefile    18 Jan 2020 21:49:53 -0000      1.315
+++ Makefile    28 Jan 2020 08:09:06 -0000
@@ -76,5 +76,8 @@
                ${DESTDIR}${PREFIX}/sbin/qshape
        ${INSTALL_MAN} ${WRKSRC}/man/man1/qshape.1 \
                ${DESTDIR}${PREFIX}/${PKGMANDIR}/man1
+.for dir in ${POSTFIX_QUEUE_SUBDIR}
+       ${RMDIR} ${DESTDIR}${POSTFIX_QUEUE_DIR}/${dir}
+.endfor

 .include "../../mk/bsd.pkg.mk"
Index: Makefile.common
===================================================================
RCS file: /cvsroot/pkgsrc/mail/postfix/Makefile.common,v
retrieving revision 1.22
diff -u -r1.22 Makefile.common
--- Makefile.common     9 Dec 2019 08:45:14 -0000       1.22
+++ Makefile.common     28 Jan 2020 08:09:07 -0000
@@ -30,6 +30,8 @@
 #
 POSTFIX_DATA_DIR?=     ${VARBASE}/db/postfix
 POSTFIX_QUEUE_DIR?=    ${VARBASE}/spool/postfix
+POSTFIX_QUEUE_SUBDIR=  active bounce corrupt defer deferred flush hold 
\
+                       incoming maildrop pid private public saved trace
 BUILD_DEFS+=           VARBASE POSTFIX_DATA_DIR POSTFIX_QUEUE_DIR

 # CCARGS is a list of options to pass to the preprocessor/compiler.
Index: PLIST
===================================================================
RCS file: /cvsroot/pkgsrc/mail/postfix/PLIST,v
retrieving revision 1.31
diff -u -r1.31 PLIST
--- PLIST       17 Jul 2019 13:33:00 -0000      1.31
+++ PLIST       28 Jan 2020 08:09:07 -0000
@@ -203,20 +203,7 @@
 ${PLIST.csasl}share/examples/postfix/smtpd.conf
 share/examples/postfix/transport
 share/examples/postfix/virtual
-@pkgdir var/spool/postfix/trace
-@pkgdir var/spool/postfix/saved
-@pkgdir var/spool/postfix/public
-@pkgdir var/spool/postfix/private
-@pkgdir var/spool/postfix/pid
-@pkgdir var/spool/postfix/maildrop
-@pkgdir var/spool/postfix/incoming
-@pkgdir var/spool/postfix/hold
-@pkgdir var/spool/postfix/flush
-@pkgdir var/spool/postfix/deferred
-@pkgdir var/spool/postfix/defer
-@pkgdir var/spool/postfix/corrupt
-@pkgdir var/spool/postfix/bounce
-@pkgdir var/spool/postfix/active
+@pkgdir var/spool/postfix
 @pkgdir var/db/postfix
 @pkgdir libexec/postfix/postfix-files.d
 @pkgdir libexec/postfix/dynamicmaps.cf.d

Best regards,
Fred