Moovida

[hal] No DeviceRemoved message is sent on the bus when ejecting a DVD

Bug #252524 reported by Guillaume Emont on 2008-07-28

Affects		Status	Importance	Assigned to	Milestone
	Moovida	Fix Released	High	Guillaume Emont	Moovida 0.5.6

Bug Description

Steps to reproduce:
1. with PYTHON_PATH correctly set, run python elisa-plugins/elisa/plugins/hal/interactive_hal.py
2. press enter
3. insert a DVD
4. wait a few seconds
5. eject the DVD
6. wait a few seconds
7. type common.application.bus.messages in the ipython shell, and it will output its contents

Expected behaviour: common.application.bus.messages contains a NewDeviceDetected object AND a DeviceRemoved object.
Current behaviour: common.application.bus.messages only contains a NewDeviceDetected object and no DeviceRemoved object.

Related branches

lp:~guijemont/moovida/hal_insanity_fix (Merged)

lp:~moovida-developers/moovida/0.5

Guillaume Emont (guijemont) on 2008-07-28

Changed in elisa:
importance:	Undecided → High
milestone:	none → 0.5.3

Revision history for this message

Benjamin Kampmann (lightyear) wrote on 2008-07-28:

fix_missing_remove_message_for_none_volume_devices.patch Edit (758 bytes, text/plain)

this patch should fix the issue

Florian Boucault (fboucault) on 2008-07-28

Changed in elisa:
assignee:	nobody → guijemont
milestone:	0.5.3 → 0.5.4

Revision history for this message

Guillaume Emont (guijemont) wrote on 2008-07-29:

I've vaguely started a minimal cleanup in lp:~guijemont/elisa/hal_insanity_fix.
I think the real problem is that we shouldn't send a udi or try to (very hackily) send a mount_point in the DeviceRemoved, but rather put a DeviceModel (or something similar) in the DeviceRemoved, so that it has a chance to work fine for any kind of device.

Florian Boucault (fboucault) on 2008-07-30

Changed in elisa:
status:	New → In Progress

Florian Boucault (fboucault) on 2008-08-05

Changed in elisa:
milestone:	0.5.4 → 0.5.5

yyoyo (lionel.martin) on 2008-08-11

Changed in elisa:
milestone:	0.5.5 → 0.5.6

Olivier Tilloy (osomon) on 2008-08-13

Changed in elisa:
status:	In Progress → Fix Committed

Olivier Tilloy (osomon) on 2008-08-19

Changed in elisa:
status:	Fix Committed → Fix Released

Report a bug

This report contains Public information

Everyone can see this information.

You are

Subscribing...

Edit bug mail

Other bug subscribers

Patches

fix_missing_remove_message_for_none_volume_devices.patch Edit

Add patch

Remote bug watches

Bug watches keep track of this bug in other bug trackers.