Bug#425473: system-tools-backends: unguarded use of scripts in preinst
Loïc Minier
lool at dooz.org
Tue May 22 16:41:09 UTC 2007
Hi,
On Tue, May 22, 2007, Niv Sardi wrote:
> > On Tue, May 22, 2007, Sven Arvidsson wrote:
> >> Confirming and merging the two reports about this.
> >> @pkg-gnome, would the attached patch be okay?
> >
> > Looks good; please commit.
>
> No it doesn't it doesn't solve the issue that we should try to see witch
> ubuntu version is incriminated, and I don't personally feel we should
> have ubuntu-bugs patching code in our release, I'm considering top drop
> that check.
There are two or three things here:
- one is that a preinst snippet was added to stop older
system-tools-backends, presumably due to some packaging mistake
before 2.1.0-0ubuntu2
- the other is that this also matches upgrades from non
"/etc/dbus-1/event.d/70system-tools-backends" aware versions of
system-tools-backends, but shouldn't
- the version check was tweaked when applied to Debian, but this was
incorrect -- this changed was reversed too
So while I agree that you could drop the preinst entirely, it would
make merging your package against the Ubuntu one or the contrary
harder.
The proposed fix seems like the correct fix for the bogus preinst that
we currently have; it's bogus because it doesn't handle upgrades from
packages which are lacking /etc/dbus-1/event.d/70system-tools-backends,
such as the 1.4 versions.
Bye,
--
Loïc Minier
More information about the pkg-gnome-maintainers
mailing list