gthumb is called with --import-photos on any directory

Bug #318694 reported by Francesco Potortì
4
Affects Status Importance Assigned to Milestone
gthumb (Ubuntu)
Fix Released
Low
Unassigned

Bug Description

Binary package hint: nautilus

I am not sure who is the culprit here, I wrote nautilus as a guess.

I have a directory full of images on my desktop. I right click on it, because I use to open it with Gthumb. However, after upgrading to Ubuntu 8.10, Nautilus does not seem to remember this. So I choose "Open with other application" and select Gthumb. Unfortunately, this does not work as expected, because for some reason Gthumb is called with the --import-photos option, so instead of opening the directory it tries to import photos from a camera.

Related branches

Revision history for this message
Chris Coulson (chrisccoulson) wrote :

Re-assigning to gthumb, as the way it is launched is controlled by gthumb's desktop files

Revision history for this message
Francesco Potortì (pot) wrote :

Could you please suggest a workaround?
I do not know where gthumb's desktop files are.

Revision history for this message
Chris Coulson (chrisccoulson) wrote :

What is happening is gthumb ships 2 desktop files. One of them is to launch gthumb normally, and the other one is to launch gthumb with "--import". They both specify different mimetypes though, so you should be presented with the correct gthumb launcher when you right-click on an image. The desktop file that launches it with "--import" specifies x-content mimetypes and is used to launch gthumb when you insert removable media containing images.

What is happening though is when you right-click on a folder, Nautilus presents both gthumb launchers in the list of applications to choose from (so gthumb will appear twice). A workaround is to select the other gthumb launcher from the list.

I don't think this is a gthumb bug. I shall re-assign back to nautilus for now.

Changed in gthumb:
importance: Undecided → Low
Revision history for this message
A. Walton (awalton) wrote :

I'm not entirely sure it's a Nautilus bug either; it doesn't really make sense not to show any of the entries in the application chooser (i.e the app chooser really shouldn't parse "NoDisplay=" since you are assigning files to content types and not displaying it in a menu). Except maybe those with the "Hidden=" key set we shouldn't show...

What makes more sense is for the "Name=" or "Comment=" field to contain the explanation of what the desktop entry file actually does (e.g. gthumb-import.desktop should have something like "GThumb Directory Importer" as it's "Name=" key, or "Import Directory into GThumb" in its "Comment=" field, or both).

Revision history for this message
Chris Coulson (chrisccoulson) wrote :

Changing the name of the launcher would be acceptable I think

Changed in gthumb:
status: New → Triaged
Revision history for this message
mjc (mjc-avtechpulse) wrote :
Revision history for this message
mjc (mjc-avtechpulse) wrote :

Oops, forgot to update the comment field:

http://svn.gnome.org/viewvc/gthumb?rev=2495&view=rev

Revision history for this message
Chris Coulson (chrisccoulson) wrote :

Yeah, that would do it. I didn't think we used the upstream desktop files for this though. Let me check

Revision history for this message
Chris Coulson (chrisccoulson) wrote :

Right, I get it now. We ship our own desktop file because the version of gthumb currently in Ubuntu doesn't have it's own (bug 287689). Upstream have added an import desktop file now (http://bugzilla.gnome.org/show_bug.cgi?id=560352) with the less confusing names. This will land in Jaunty next time ghtumb is updated.

Changed in gthumb:
status: Triaged → Fix Committed
Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package gthumb - 3:2.10.11-1ubuntu1

---------------
gthumb (3:2.10.11-1ubuntu1) karmic; urgency=low

  * Merge with Debian unstable; remaining Ubuntu changes:
    - ubuntu-lpi.patch, debian/control: Launchpad Integration, add
      liblaunchpad-integration-dev build dependency.
    - debian/rules: Build PO template on package build.
    - debian/rules: Add gettext domain to *.desktop and *.server files.
    - 04-fix-gvfs-umount.patch: Fix up gphoto-import.desktop to work with
      current gvfs. (LP #351122)
    - debian/control: Recommend gvfs-bin, so that the unmounting of the
      libgphoto gvfs can actually work.
  * debian/control: Drop libopenrawgnome-dev build dependency. It's in
    universe, and we shouldn't spread more than one raw processing library in
    main.
  * debian/control: Fix "automake1.9" build dependency to be "automake", since
    the upstream scripts expect 1.10.
  * debian/rules: Call autoreconf before build, since the upstream Makefiles
    call libtool/autoconf in an insufficient way, leading to an FTBFS.
  * 10-fix_POTFILES.skip.patch: Add quilt's .pc/ cruft from our patches to the
    ignore list, to fix FTBFS.

gthumb (3:2.10.11-1) unstable; urgency=low

  * New Upstream Version:
    - fix crasher bug in exif tag reading, caused by malformed tiff headers.
      (LP: #316017)
    - give the photo-import desktop file different name parameters, so that
      users can distinguish it from the normal desktop file. (LP: #318694)
    - fixes GtkSpinButton deprecated non-zero page size adjustment setting,
      patch from Ubuntu merged upstream (LP: #319349)
  * Forgot bugnumber to close in 3:2.10.10-1, added (mostly for
    changelog reasons, closing it manually)
  * debian/control:
    - scrollkeeper moved from gthumb's Recommends to gthumb-data's
      Depends (Closes: #508709)
    - yelp Recommends moved from gthumb to gthumb-data
    - Standards-Version bumped to 3.8.1 (no changes needed)
  * debian/patches/:
    - 05-fix_locales.patch, 06-add_keybinding_DoNotSave.patch.
      10-fix_POTFILES.skip.patch refreshed
    - 12-remove_Save_button_in_crop_dialog.patch added (Closes: #495444)

gthumb (3:2.10.10-2) unstable; urgency=low

  * Brown-paper-bag bug upload
  * debian/patches/:
    - 01-PIC.patch forwarded upstream, annotated.
    - 06-add_keybinding_DoNotSave.patch forwarded upstream, annotated.
    - 07-fix_zooms.patch, 08-fix_preferences.patch forwarded upstream,
      annotated.
    - 09-fix_infinite_loop_dragging_parent_dir.patch forwarded upstream,
      annotated.
    - 10-fix_POTFILES.skip.patch updated
    - 11-fix_Film_on_import.patch added, thanks to Matijs van Zuijlen
      (Closes: #510143)
    - series updated: disable 03-libgthumb_moduledir.patch (i.e. let's
      fix this bug before someone else notices :) )

 -- Martin Pitt <email address hidden> Tue, 05 May 2009 10:32:50 +0200

Changed in gthumb (Ubuntu):
status: Fix Committed → Fix Released
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.