Bill Allombert wrote:
> Then it is a bug in debhelper. It should still call update-menus in the
> postinst, (or triggers it using dpkg-trigger).
> 
> The problem is that menu only display menu entries for configured
> packages, because unconfigured or half-configured packages generally
> do not work. Without an explicit trigger in the postinst, dpkg run
> the trigger before the package is configured and then update-menus
> ignores it. This is related to bug #473461 which cause update-menus 
> to be run twice in some case which is a regression from Etch.

It'd be a real shame if it turns out we cannot use triggers to eliminate
all the update-menus calls and eventually get rid of the ugly
backgrounding code in update-menus.

I wonder how useful it is for menu to test for
unconfigured/half-configured packages at all.

* That test does not cover unconfigured/half-configured
  dependencies, which could also potentially break a menu item.
* If a package is temporarily de-configured in the middle of an apt run
  and then re-configured, update-menus will not be run in time to
  notice this happened, and the menu item will remain available
  throughout.
* There's a window in package removal where all the files are gone,
  but the postrm has not run, when the menu items are certianly broken
  but still vidible. This is surely a more common case than any of
  the others, and yet update-menus doesn't handle it. (Triggers may
  have narrowed this window, I haven't checked.)
* No such checks are done for desktop files, which are of course, where
  most naive users will really get their menus from..

So I guess it's up to you if you want to leave this test in. If you do,
I can either revert the debhelper change or add code to manually trigger
menu in the postinst. I'm not sure if explicitly triggering menu will
result in update-menus running less often or not, so am inclined not to
bother trying to do that.

> Fixing bug 473461 might alleviate this problem when using apt.

Only partially at best. (Since there are other package managers than
apt.)

-- 
see shy jo

Attachment: signature.asc
Description: Digital signature

Reply via email to