Can't open multiple files from File Browser in F-Spot Photo Viewer

Bug #202717 reported by Edward
8
Affects Status Importance Assigned to Milestone
f-spot (Ubuntu)
Fix Released
Low
Ubuntu Desktop Bugs
Hardy
Fix Released
Low
Unassigned

Bug Description

Binary package hint: f-spot

In Hardy, if you select multiple files, right click them and select 'Open with "F-Spot Photo Viewer"' the viewer silently fails to load, whereas it used to load all the selected files. It still works if you select only one file.

F-Spot 0.4.2-0ubuntu2 appears to require a '-view' argument for every file viewed - i.e. this will not work:

# f-spot -view file-a.cr2 file-b.cr2

but this will:

# f-spot -view file-a.cr2 -view file-b.cr2

Related branches

Revision history for this message
Martin Schwenke (martin-meltin) wrote :

3 things to add:

* In Gutsy, it was possible to select multiple files in Nautilus and open them
  in F-Spot. Therefore, the behaviour we're seeing in Hardy is a regression.

* If this isn't going to be considered an F-Spot bug, then it is a bug in Nautilus
  since it allows you to select multiple files and attempt to view them using
  F-Spot. :-)

* It is possible to work around this bug by right-clicking on a folder in Nautilus,
  selecting "Open with Other Application" and then selecting "F-Spot Photo
  Viewer". One reason for doing this is that, when viewing multiple CR2
  (Canon Raw) images, switching between images is an order of magnitude
  faster in F-Spot that it is in GThumb.

Revision history for this message
Martin Schwenke (martin-meltin) wrote :

However, opening a whole folder doesn't maintain the sort order. The files are displayed in directory order (same as "ls -U") rather than sorted alphabetically.

Therefore, this isn't as useful as the old behaviour where multiple files could be selected and they would appear in F-Spot in the order they appeared in the Nautilus listing that they were selected from.

Revision history for this message
Martin Schwenke (martin-meltin) wrote :

This is fixed upstream:

  http://svn.gnome.org/viewvc/f-spot/trunk/src/main.cs?r1=3883&r2=3882&pathrev=3883

That patch applies with a bit of fuzz. I've attached a rediffed version that should apply with no fuzz.

Revision history for this message
Pedro Villavicencio (pedro) wrote :

to be looked by a desktop-dev thanks for the patch.

Changed in f-spot:
assignee: nobody → desktop-bugs
importance: Undecided → Low
status: New → Confirmed
Revision history for this message
Sebastien Bacher (seb128) wrote :

the bug has been fixed upstream now

Changed in f-spot:
status: Confirmed → Fix Committed
Revision history for this message
Martin Schwenke (martin-meltin) wrote :

To ensure that the patch fixes the bug and so that I (and others) have a working version I've uploaded a patched version to my PPA at:

  https://launchpad.net/~martin-meltin/+archive

You'll want the hardy version (my gutsy package for f-spot fixes a bug in the gutsy version).

I've tested the i386 version and the bug is fixed.

Changed in f-spot:
importance: Undecided → Low
status: New → Confirmed
Revision history for this message
Martin Schwenke (martin-meltin) wrote :

I'm a surprised to see this tagged with "Low" importance. Given that F-spot gets its own page in the list of Hardy features http://www.ubuntu.com/products/whatisubuntu/804features/photos/ I would have thought the prominence of the package, and the availability of a fix, would make the importance higher? :-)

Revision history for this message
Martin Pitt (pitti) wrote :

Accepted into -proposed, please test and give feedback here

Changed in f-spot:
status: Confirmed → Fix Committed
Revision history for this message
Martin Pitt (pitti) wrote :

Copied this hardy-proposed version to intrepid.

Changed in f-spot:
status: Fix Committed → Fix Released
Revision history for this message
Martin Schwenke (martin-meltin) wrote :

Will there be a delay before this hits the hardy-proposed repository?

I don't see it there yet... but then I haven't used -proposed before.

Thanks!

Revision history for this message
Martin Schwenke (martin-meltin) wrote :

OK, got it - approx was caching Packages.bz2 a little too aggressively.

The version in hardy-proposed fixes this bug.

I'll add some comments to bug #208467 about this release and importing photos.

Revision history for this message
Pedro Villavicencio (pedro) wrote :

confirmed, the new version fix this issue, now you can use: f-spot -view * if you want to and it works fine.

Revision history for this message
Martin Pitt (pitti) wrote :

Copied to hardy-updates.

Changed in f-spot:
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.