Ubuntu
unity package

Opening a volume that is not mounted fails

Bug #676514 reported by Olivier Tilloy on 2010-11-17

This bug affects 1 person

	Status	Importance	Assigned to	Milestone
The Leprechaun project	Fix Released	Medium	Olivier Tilloy	The Leprechaun project m1
The Upicek project	Won't Fix	Undecided	Unassigned
Unity	Invalid	Undecided	Unassigned
unity (Ubuntu)	Invalid	Undecided	Unassigned

Bug Description

Binary package hint: unity

If nautilus is not launched when the session is started, removable devices won’t auto-mount.
Plugging in a removable device will append an icon in the launcher, but clicking this icon will fail to open the file manager to browse this device.

The reason is that the code for open_volume() in places-volume-child-controller.vala calls the asynchronous method to mount the volume but doesn’t wait for it to complete before trying to open up the URI for the volume.

To reproduce, one should make sure that nautilus is not launched when the session is started by removing 'filemanager' from the GConf key /desktop/gnome/session/required_components_list.

ProblemType: Bug
DistroRelease: Ubuntu 10.10
Package: unity 0.2.46-0ubuntu5
ProcVersionSignature: Ubuntu 2.6.35-22.35-generic 2.6.35.4
Uname: Linux 2.6.35-22-generic x86_64
NonfreeKernelModules: nvidia wl
Architecture: amd64
Date: Wed Nov 17 13:37:00 2010
EcryptfsInUse: Yes
InstallationMedia: Ubuntu 10.10 "Maverick Meerkat" - Alpha amd64 (20100926)
ProcEnviron:
LANG=fr_FR.utf8
SHELL=/bin/bash
SourcePackage: unity

Tags:

Revision history for this message

Olivier Tilloy (osomon) wrote on 2010-11-17:

Dependencies.txt Edit (6.1 KiB, text/plain; charset="utf-8")

Revision history for this message

Olivier Tilloy (osomon) wrote on 2010-11-17:

monitor_mount.patch Edit (1.5 KiB, text/plain)

The attached patch solves the problem by monitoring the mounts added and continuing the open operation once the volume is mounted.

Note that I’m not fluent in Vala and as you can see I couldn’t find a way to connect a callback to the volume.mount.begin(…) method, so I had to work around it by monitoring all mount added signals and matching the volume against the volume being mounted. Suggestions on how to achieve this in a more elegant manner are welcome.

Olivier Tilloy (osomon) on 2010-11-17

Changed in leprechaun:
assignee:	nobody → Olivier Tilloy (osomon)
importance:	Undecided → Medium
milestone:	none → m1
status:	New → Confirmed

Olivier Tilloy (osomon) on 2010-11-17

Changed in leprechaun:
status:	Confirmed → In Progress

Brian Murray (brian-murray) on 2010-11-17

tags:

added: patch

Olivier Tilloy (osomon) on 2010-11-18

Changed in leprechaun:
status:	In Progress → Fix Committed

Revision history for this message

Bill Filler (bfiller) wrote on 2010-11-18:

lets make sure this makes it into upicek as well as we won't be autotarting nautilus. for some reason I couldn't "also affects" upicek.

Alex Launi (alexlauni) on 2010-11-18

Changed in unity:
status:	New → Confirmed
Changed in unity (Ubuntu):
status:	New → Confirmed

Revision history for this message

Neil J. Patel (njpatel) wrote on 2010-12-01:

We count on nautilus being launched at startup, either with the "run without windows" mode or when it's drawing the desktop.

Changed in unity:
status:	Confirmed → Invalid

Olivier Tilloy (osomon) on 2010-12-03

Changed in leprechaun:
status:	Fix Committed → Fix Released

Revision history for this message

Florian Boucault (fboucault) wrote on 2011-01-17:

It will be fixed in Natty as nautilus will be started automatically. I think Maverick users of unity-2d ppa can live with that bug too until we make that version use nautilus automatically too.