After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 555831 - be able to mount a volume from recent files
be able to mount a volume from recent files
Status: RESOLVED FIXED
Product: eog
Classification: Core
Component: general
git master
Other Linux
: Normal enhancement
: ---
Assigned To: EOG Maintainers
EOG Maintainers
Depends on:
Blocks:
 
 
Reported: 2008-10-10 18:59 UTC by Felix Riemann
Modified: 2017-06-04 15:32 UTC
See Also:
GNOME target: ---
GNOME version: 2.25/2.26


Attachments
EogJobs: Try to mount volume if not already mounted (3.00 KB, patch)
2017-05-15 09:08 UTC, Ondrej Holy
none Details | Review
EogJobs: Try to mount volume if not already mounted (3.03 KB, patch)
2017-05-23 07:58 UTC, Ondrej Holy
committed Details | Review

Description Felix Riemann 2008-10-10 18:59:11 UTC
When you try to load a file from the recent files  that is on an unmounted volume EOG will fail to load the file. It should try to mount the volume the file is on.

To reproduce:
1. Create a SSH connection in nautilus and open an image over it
2. Close EOG.
3. "Unmount" the SSH connection
4. Open EOG and choose the file from the recent files list.

What happens:
EOG fails saying there could no images be found

What should happen:
EOG should try to re-mount the SSH connection possibly asking for my password.
Comment 1 Ondrej Holy 2017-05-15 09:08:15 UTC
Created attachment 351871 [details] [review]
EogJobs: Try to mount volume if not already mounted

It may happen for various reasons that somebody tries to open a
location, which is not already mounted. Consequently, "No images
found in '[location]'." error is shown to the user currently. Let's
try to mount the volume in such case instead of showing the error.
Comment 2 Ondrej Holy 2017-05-15 09:08:48 UTC
The is also needed due to the proposed changes to the admin backend, Bug 771285.
Comment 3 Felix Riemann 2017-05-21 18:47:03 UTC
Review of attachment 351871 [details] [review]:

Works nice in a quick test with an SMB/CIFS share.

::: src/eog-jobs.c
@@ +702,3 @@
+				       _g_file_mount_enclosing_volume_sync_cb,
+				       data);
+	g_cond_wait (&data->cond, &data->mutex);

Shouldn't this be wrapped in a loop?
Comment 4 Ondrej Holy 2017-05-23 07:58:55 UTC
Created attachment 352389 [details] [review]
EogJobs: Try to mount volume if not already mounted

Thanks for the review! Yes, this should be wrapped in a loop, so I am attaching an updated patch.
Comment 5 Felix Riemann 2017-06-04 15:30:49 UTC
Thanks! :)

commit beb99cfda1bfd9097e7b2ca3ec4d3950cb2b13b8
Author: Ondrej Holy <>
Date:   Mon May 15 10:27:03 2017 +0200

    EogJobs: Try to mount volume if not already mounted
    
    It may happen for various reasons that somebody tries to open a
    location, which is not already mounted. Consequently, "No images
    found in '[location]'." error is shown to the user currently. Let's
    try to mount the volume in such case instead of showing the error.
    
    https://bugzilla.gnome.org/show_bug.cgi?id=555831
---
This problem has been fixed in the unstable development version. The fix will be available in the next major software release. You may need to upgrade your Linux distribution to obtain that newer version.