jockey should use the new Lucid restart notification system

Bug #570215 reported by Johan on 2010-04-26
10
This bug affects 2 people
Affects Status Importance Assigned to Milestone
jockey (Ubuntu)
Low
Martin Pitt
Declined for Lucid by Sebastien Bacher
Maverick
Low
Martin Pitt

Bug Description

When you choose to install a restricted display driver, for example, by activating it in the jockey-gtk interface in Lucid, after the installation, you can close the window, and nothing reminds you to restart your computer.

Expected behavior : The new "power" button on the upper right corner of the screen should turn red, and the "restart" entry of that menu should turn to "restart required", like when you update your system by synaptic, and installed a new kernel for example. There should be also a notification in the new notification area.

ProblemType: Bug
DistroRelease: Ubuntu 10.04
Package: jockey-common 0.5.8-0ubuntu8
ProcVersionSignature: Ubuntu 2.6.32-21.32-generic 2.6.32.11+drm33.2
Uname: Linux 2.6.32-21-generic x86_64
NonfreeKernelModules: nvidia
Architecture: amd64
Date: Mon Apr 26 16:15:56 2010
InstallationMedia: Ubuntu 10.04 "Lucid Lynx" - Release Candidate amd64 (20100419.1)
MachineType: System manufacturer System Product Name
PackageArchitecture: all
ProcCmdLine: BOOT_IMAGE=/boot/vmlinuz-2.6.32-21-generic root=UUID=51193219-9eba-402c-b875-abaf169ea6af ro quiet splash
ProcEnviron:
 LANG=en_US.utf8
 SHELL=/bin/bash
SourcePackage: jockey
dmi.bios.date: 09/06/2006
dmi.bios.vendor: American Megatrends Inc.
dmi.bios.version: 0509
dmi.board.asset.tag: To Be Filled By O.E.M.
dmi.board.name: P5B
dmi.board.vendor: ASUSTeK Computer INC.
dmi.board.version: Rev 1.xx
dmi.chassis.asset.tag: Asset-1234567890
dmi.chassis.type: 3
dmi.chassis.vendor: Chassis Manufacture
dmi.chassis.version: Chassis Version
dmi.modalias: dmi:bvnAmericanMegatrendsInc.:bvr0509:bd09/06/2006:svnSystemmanufacturer:pnSystemProductName:pvrSystemVersion:rvnASUSTeKComputerINC.:rnP5B:rvrRev1.xx:cvnChassisManufacture:ct3:cvrChassisVersion:
dmi.product.name: System Product Name
dmi.product.version: System Version
dmi.sys.vendor: System manufacturer

Johan (deberghes-johan) wrote :
Changed in jockey (Ubuntu):
importance: Undecided → Low
Sebastien Bacher (seb128) wrote :

Not sure if that's still the case in the current version but it would be nice to fix if that's correct

summary: - [Lucid] jockey should use the new Lucid restart notification system
+ jockey should use the new Lucid restart notification system
Changed in jockey (Ubuntu Maverick):
assignee: nobody → Martin Pitt (pitti)
Sebastien Bacher (seb128) wrote :

seems jockey should write /var/run/reboot-required

Michael Vogt (mvo) wrote :

The backend just needs to call
/usr/share/update-notifier/notify-reboot-required
to trigger the menu.

Martin Pitt (pitti) on 2010-09-02
Changed in jockey (Ubuntu Maverick):
status: New → In Progress
Martin Pitt (pitti) on 2010-09-15
Changed in jockey (Ubuntu Maverick):
status: In Progress → Fix Committed
Martin Pitt (pitti) wrote :

jockey (0.5.10-0ubuntu3) maverick; urgency=low

  * Add support for bypassing local detection for search_driver(). (Merged
    from trunk). Thanks to Till Kamppeter for the initial patches!
  * jockey/ui.py: Ubuntu does not currently support local printer driver
    handlers, so speed up the lookup of remote ones by disabling local
    detection of hardware/handlers in search_driver() if we are looking for a
    printer driver. (Note that this is a hack, but the speedup is significant)
    (LP: #574396)
  * Add OSLib.notify_reboot_required() and call it in set_handler_enabled().
    Merged from trunk, and add Ubuntu specific implementation using
    /usr/share/update-notifier/notify-reboot-required in OSLib. (LP: #570215)
  * data/handlers/b43.py: Install firmware-b43-installer package instead of
    b43-fwcutter. (LP: #595344)

 -- Martin Pitt <email address hidden> Wed, 15 Sep 2010 13:35:48 +0200

Changed in jockey (Ubuntu Maverick):
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Other bug subscribers