On Wed, May 07, 2008 at 08:51:05AM +0200, Adrian von Bidder wrote:
> I added the statoverride things because IIRC I had problems with the dir 
> premissions being reset somehow, but I'm not clear on why/how.
> 
> (You can tell I'm not that an experienced packager, can't you :-)
> 
> So removing the statoverride logic is fine by me, however I'm quite swamped 
> now and would be very happy for a patch (or NMU since this is apparently 
> RC.)

I'm just about to NMU 1.31-2.1 with the following change:


--- postgrey-1.31/debian/postinst
+++ postgrey-1.31/debian/postinst
@@ -14,8 +14,9 @@
         getent group postgrey > /dev/null || \
             addgroup --system postgrey

-        if [ ! -e "$DBDIR" ]; then
-            mkdir -p "$DBDIR"
+        install -d -o postgrey -g postgrey -m 0700 "$DBDIR"
+
+       if ! dpkg-statoverride --list "$DBDIR" > /dev/null 2>&1 ; then
             dpkg-statoverride --update --add \
                 postgrey postgrey 0700 "$DBDIR"
         fi


While I still think that dpkg-statoverride should not be needed, I decided to
keep it as NMU is not a place to deal with such things, especially when
"mysterious permission resets", which I know nothing about, are in question.

-- 
Marcin Owsiany <[EMAIL PROTECTED]>             http://marcin.owsiany.pl/
GnuPG: 1024D/60F41216  FE67 DA2D 0ACA FC5E 3F75  D6F6 3A0D 8AA0 60F4 1216



-- 
To UNSUBSCRIBE, email to [EMAIL PROTECTED]
with a subject of "unsubscribe". Trouble? Contact [EMAIL PROTECTED]

Reply via email to