Bug#639556: smart-notifier: must not restart dbus on install/upgrade

[Osamu Aoki <osamu@debian.org>]

Package: smart-notifier
Version: 0.28-3
Followup-For: Bug #639556

This killed X while loosing all the data on other terminals ...  quite unhappy
...

My first thought was that, if this reload of dbus by smart-notifier
postinst is needed for its functionality and if there is no safer way
with X-server, please do not try to make this active via initscript and
document that this require reboot in README.Debian.

Then I find old discussion on dbus:

  http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=495257

Hmm... restarting dbus seems to be dangerous thing to do.  Then I wonder
what other tools are doing on my system:

| grep 'invoke-rc.d *dbus' *.p*
| avahi-daemon.postinst:      invoke-rc.d dbus force-reload || true
| bluez.postinst:            invoke-rc.d dbus force-reload || true
| dbus.postinst:  invoke-rc.d dbus start || exit $?
| dbus.prerm:     invoke-rc.d dbus stop || exit $?
| hal.postinst:      invoke-rc.d dbus force-reload || true
| hal.postrm:             invoke-rc.d dbus force-reload || true
| network-manager-gnome.postinst:        invoke-rc.d dbus force-reload || true
| network-manager.postinst:            invoke-rc.d dbus force-reload || true
| pommed.postinst:    invoke-rc.d dbus force-reload || true
| pommed.postrm:    invoke-rc.d dbus force-reload || true
| smart-notifier.postinst:                invoke-rc.d dbus restart || true
| system-tools-backends.postrm:        invoke-rc.d dbus force-reload || true


All other similar services except dbus itself uses "force-reload".

Such typical script is:
|     # Ask the bus to reload the config file
|     if [ -x "/etc/init.d/dbus" ]; then
|       invoke-rc.d dbus force-reload || true
|     fi
|   ;;

At least, "invoke-rc.d dbus restart" kills X now.

Let me check if "invoke-rc.d dbus force-reload" is safer.
... (sending this as this before testing to avoid loosing data.)

Osamu

-- System Information:
Debian Release: wheezy/sid
  APT prefers testing
  APT policy: (500, 'testing'), (50, 'unstable')
Architecture: amd64 (x86_64)

Kernel: Linux 3.0.0-2-amd64 (SMP w/2 CPU cores)
Locale: LANG=en_US.utf8, LC_CTYPE=en_US.utf8 (charmap=UTF-8)
Shell: /bin/sh linked to /bin/dash

Versions of packages smart-notifier depends on:
ii  dbus           1.4.16-1      
ii  python         2.7.2-8       
ii  python-dbus    0.84.0-2      
ii  python-glade2  2.24.0-2      
ii  python-gtk2    2.24.0-2      
ii  smartmontools  5.41+svn3365-1

smart-notifier recommends no packages.

smart-notifier suggests no packages.

-- no debconf information