Bug#684732: unblock: nut/2.6.4-2
Le Thu, 20 Dec 2012 14:33:09 +0100,
Julien Cristau <jcristau@debian.org> a écrit :
> Control: tags -1 moreinfo
>
> On Sun, Dec 2, 2012 at 19:43:44 +0100, Laurent Bigonville wrote:
>
> > [...]
> > > > + # make sure that conffiles are secured and have the correct
> > > > ownerships
> > > > + if [ -d /etc/nut/ ] ; then
> > > > + chown root:nut /etc/nut/
> > > > + fi
> > > > + for file in nut.conf upsmon.conf upssched.conf ; do
> > > > + if [ -f /etc/nut/$file ] ; then
> > > > + chown root:nut /etc/nut/$file
> > > > + chmod 640 /etc/nut/$file
> > > > + fi
> > > > + done
> > > > +
> > >
> > > I still think the chowns/chmods shouldn't be done on upgrades, if
> > > for whatever reason the local admin changed those that's their
> > > choice.
> >
> > Is this really blocking the transition? The version in squeeze is
> > also doing this. We could remove this later I guess.
> >
> Well I'm not comfortable unblocking that code.
>
> > [...]
> > > > - # re process nut.conf MODE so that it can be sourced
> > > > - NUT_MODE=`grep -e '^ *MODE' /etc/nut/nut.conf | tr -d "
> > > > "`
> > > > - sed "s/^ *MODE.*/$NUT_MODE/" /etc/nut/nut.conf
> > > > > /etc/nut/nut.conf.new
> > > > - mv /etc/nut/nut.conf.new /etc/nut/nut.conf
> > > > + if dpkg --compare-versions "$2" le "2.6.4-2~" ; then
> > > > + rm -f /etc/init.d/nut
> > > > + update-rc.d nut remove >/dev/null
> > >
> > > If /etc/init.d/nut was a conffile, I don't think you get to rm -f
> > > it on upgrade, at least if it was modified.
> >
> > IIRC, I didn't use dpkg-maintscript-helper because the file is
> > owned by the nut package in squeeze (which is now a metapackage)
> > and it was not really working as expected. The init file should
> > probably also be removed in the nut-client package to support
> > partial upgrades.
> >
> What's the status here? (You can do thinks correctly without using
> dpkg-maintscript-helper, fwiw.)
I have unfortunately not the time for this ATM, so if somebody else
want to tackle this, I have no objections.
Cheers
Laurent Bigonville
Reply to: