[pkg-fetchmail-maint] Bug in postinst

[Loïc Minier]

On Tue, Nov 15, 2005, Héctor García wrote:
> >  1/ the permission always overriden in postinst on this dir, but /var is
> >  not /etc, and I find it acceptable; I would call it safer
> I don't like the permision been always overriden. If the dir existed and
> the admin changed something, we should not change it back.

 I don't support the admin changing /var, sorry.  Nor do I support him
 changing the +x flag on /usr/bin/fetchmail.  That will be overriden
 between package upgrades, *exactly* as it would have been would this
 directory be shipped in the package files.

> At the moment, the only strange thing I see on svn (taken apart my
> dislikes, which are only personal opinions not tecnical ones), are this
> lines
> if [ -d /var/run/fetchmail ]; then
> It should be at least
> if ! [ -d /var/run/fetchmail ]; then

 That's incorrect, the SVN has the later since yesterday:
 http://svn.debian.org/wsvn/pkg-fetchmail/trunk/debian/fetchmail.postinst?op=file&rev=0&sc=0

-- 
Loïc Minier <lool at dooz.org>
"What do we want? BRAINS!    When do we want it? BRAINS!"