Opening a volume that is not mounted fails

Bug #676514 reported by Olivier Tilloy
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
The Leprechaun project
Fix Released
Medium
Olivier Tilloy
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

Revision history for this message
Olivier Tilloy (osomon) wrote :
Revision history for this message
Olivier Tilloy (osomon) wrote :

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)
Changed in leprechaun:
assignee: nobody → Olivier Tilloy (osomon)
importance: Undecided → Medium
milestone: none → m1
status: New → Confirmed
Olivier Tilloy (osomon)
Changed in leprechaun:
status: Confirmed → In Progress
tags: added: patch
Olivier Tilloy (osomon)
Changed in leprechaun:
status: In Progress → Fix Committed
Revision history for this message
Bill Filler (bfiller) wrote :

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)
Changed in unity:
status: New → Confirmed
Changed in unity (Ubuntu):
status: New → Confirmed
Revision history for this message
Neil J. Patel (njpatel) wrote :

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)
Changed in leprechaun:
status: Fix Committed → Fix Released
Revision history for this message
Florian Boucault (fboucault) wrote :

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.

Changed in upicek:
status: New → Won't Fix
Changed in unity:
status: Invalid → Confirmed
Omer Akram (om26er)
Changed in unity:
status: Confirmed → Invalid
Changed in unity (Ubuntu):
status: Confirmed → Invalid
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

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