[Date Prev][Date Next] [Thread Prev][Thread Next] [Date Index] [Thread Index]

Re: Preparation of the next stable Debian GNU/Linux update (I)



I see that the preparations for releasin 3.1r1 are well underway, and I
may be too late for this, but I though I would ask.

I realize (after having it gently pointed out to me in #338004) that
this patch is incomplete, as it missed a stray db_stop.

This one time, at band camp, Stephen Gran said:
> diff -u debian/clamav-daemon.postinst.in.old debian/clamav-daemon.postinst.in
> --- debian/clamav-daemon.postinst.in.old    2005-08-20 18:55:06.000000000 -0400
> +++ debian/clamav-daemon.postinst.in        2005-08-20 18:58:42.000000000 -0400
> @@ -30,10 +30,10 @@
> 
>    UCFVER=`check_ucf`
> 
> -  . /usr/share/debconf/confmodule
>    if [ -n "$2" ]; then
>      if dpkg --compare-versions "$2" lt 0.82-2; then
>        #loading debconf module
> +      . /usr/share/debconf/confmodule
>        db_purge || true
>        db_stop  || true
>      fi
> @@ -60,7 +60,7 @@
>        echo "     delaycompress" >> $DEBROTATEFILE
>        echo "     create 640  $User adm" >> $DEBROTATEFILE
>        echo "     postrotate" >> $DEBROTATEFILE
> -      echo "     kill -HUP \`cat $PidFile\` > /dev/null" >> $DEBROTATEFILE
> +      echo "     [ ! -f $PidFile ] || kill -HUP \`cat $PidFile\` > /dev/null" >> $DEBROTATEFILE
>        echo "     endscript" >> $DEBROTATEFILE
>        echo "     }" >> $DEBROTATEFILE
>        touch "$LogFile"

Would it be possible to add the following:

@@ -78,8 +78,6 @@
     fi
   fi

-  db_stop  || true
-
   ;;
   abort-upgrade|abort-remove|abort-deconfigure)
   ;;

If it is, I will prepare a fixed package as soon as possible.  If not, I
understand.

Thanks,
-- 
 -----------------------------------------------------------------
|   ,''`.                                            Stephen Gran |
|  : :' :                                        sgran@debian.org |
|  `. `'                        Debian user, admin, and developer |
|    `-                                     http://www.debian.org |
 -----------------------------------------------------------------

Attachment: signature.asc
Description: Digital signature


Reply to: