imports photos _before_ I hit the import button

Bug #412091 reported by Alan Pope 🍺🐧🐱 🦄
24
This bug affects 4 people
Affects Status Importance Assigned to Milestone
F-Spot
Fix Released
Low
f-spot (Debian)
Fix Released
Unknown
f-spot (Ubuntu)
Fix Released
Medium
Unassigned

Bug Description

Binary package hint: f-spot

First time I've run f-spot on this install, I used the import dialog to tell it where a bunch of photos were. It scanned the folders and took a long time doing it (9000 photos). I notice that it's actually imported (copied the photos) _before_ actually hit the 'import' button.

I don't mind it scanning the pictures in the source location first then offering me a bunch to select, but surely it shouldn't be copying them to ~/Photos _first_ then asking me to press the import button?

See attached screenshot (lots of files in ~/Photos yet I have not hit Import yet).

ProblemType: Bug
Architecture: amd64
Date: Tue Aug 11 19:22:22 2009
DistroRelease: Ubuntu 9.10
NonfreeKernelModules: nvidia
Package: f-spot 0.6.0.0-1~ubuntu1
ProcEnviron:
 PATH=(custom, user)
 LANG=en_GB.UTF-8
 SHELL=/bin/bash
ProcVersionSignature: Ubuntu 2.6.31-5.24-generic
SourcePackage: f-spot
Uname: Linux 2.6.31-5-generic x86_64

Revision history for this message
Alan Pope 🍺🐧🐱 🦄 (popey) wrote :
Revision history for this message
Iain Lane (laney) wrote :

Evening,

Thanks for the report. This is reported upstream but has little response there. Check the linked bugs for more information.

Iain

Changed in f-spot (Ubuntu):
status: New → Triaged
importance: Undecided → Medium
Changed in f-spot:
status: Unknown → New
Changed in f-spot (Debian):
status: Unknown → Confirmed
Revision history for this message
Benjamin Humphrey (humphreybc) wrote :

F-Spot is ridiculously useless.

Revision history for this message
Arvind S Raj (arvindsraj-deactivatedaccount) wrote :

I can't even start f-spot in karmic. I get the following error!
[Info 13:30:20.090] Initializing DBus
[Info 13:30:20.186] Initializing Mono.Addins
[Info 13:30:20.324] Starting new FSpot server (f-spot 0.6.1.5)

** (f-spot:3097): CRITICAL **: atk_object_set_name: assertion `name != NULL' failed

** (f-spot:3097): CRITICAL **: atk_object_set_name: assertion `name != NULL' failed

** (f-spot:3097): CRITICAL **: atk_object_set_name: assertion `name != NULL' failed

** (f-spot:3097): CRITICAL **: atk_object_set_name: assertion `name != NULL' failed

** (f-spot:3097): CRITICAL **: atk_object_set_name: assertion `name != NULL' failed
[Info 13:30:20.950] Starting BeagleService
[Info 13:30:20.964] Hack for gnome-settings-daemon engaged

(f-spot:3097): GdkPixbuf-WARNING **: GdkPixbufLoader finalized without calling gdk_pixbuf_loader_close() - this is not allowed. You must explicitly end the data stream to the loader before dropping the last reference.
The program 'f-spot' received an X Window System error.
This probably reflects a bug in the program.
The error was 'BadAlloc (insufficient resources for operation)'.
  (Details: serial 2944 error_code 11 request_code 53 minor_code 0)
  (Note to programmers: normally, X errors are reported asynchronously;
   that is, you will receive the error a while after causing it.
   To debug your program, run it with the --sync command line
   option to change this behavior. You can then get a meaningful
   backtrace from your debugger if you break on the gdk_x_error() function.)

When I tried to load f-spot binary in gdb, I got the following error:
"/usr/bin/f-spot": not in executable format: File format not recognized

Changed in f-spot:
status: New → Fix Released
Revision history for this message
Sebastien Bacher (seb128) wrote :

the bug has been fixed upstream now

summary: - [karmic] f-spot imports photos _before_ I hit the import button
+ imports photos _before_ I hit the import button
Changed in f-spot (Ubuntu):
status: Triaged → Fix Committed
Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package f-spot - 0.7.0-1

---------------
f-spot (0.7.0-1) experimental; urgency=low

  * New upstream release 0.7.0
   + First release of the unstable 0.7 development series. Massive changes.
   + Reparenting and detaching support (Anton Keks) (Closes: #559745)
   + A new Mallard-based documentation (Harold Schreckengost)
   + No longer embeds flickrnet, uses distribution copy (Iain Lane)
   + Adoption of a large amount of Hyena functionality (Paul Lange, Peter
     Goetz)
   + No longer embeds gnome-keyring-sharp
   + Completely rewritten import, much faster and less memory hungry (Ruben
     Vermeersch) (Closes: #559080, #492658, #341790, #357811, #426017) (LP:
     #412091)
   + No longer use gphoto2-sharp, now uses gvfs which is less crash-pron
     (Ruben Vermeersch)
   + Fix Facebook support (Ruben Vermeersch)
   + Modernized unit tests
   + Revamped build (Mike Gemünde)
   + Much improved duplicate detection (much faster too) (Ruben Vermeersch)
   + Mouse selection in Iconview (Vincent Pomey)
   + Image panning support using middle mouse button (Wojciech Dzierżanowski)
   + Timeline slider now restricted to the size of the window (Iain Churcher)
   + Over 100 bugs closed (http://bit.ly/cyVjnD)
     - No more warnings about schema defaults (Closes: #584215) (LP: #586132)
  * debian/control: Clean up build deps to match configure checks
  * debian/rules: Don't run dh_makeshilbs as we don't ship any shared
    libraries. There are some private ones though, which get picked up and
    result in a useless postinst/postrm call to ldconfig. Thanks, lintian.
  * debian/patches/debian_fix-distclean.patch,
    debian/patches/debian_fix_f-spot.exe.config.patch,
    debian/patches/debian_link-system-flickrnet.patch,
    debian/patches/debian_link-system-gnome-keyring.patch,
    debian/patches/debian_disable-unit-tests,
    debian/patches/git_transition_duration.patch,
    debian/patches/ubuntu_fix_folder_export_hang.patch:
    Clean up obsolete patches which are no longer necessary
  * debian/patches/*: Temporarily disable patches which originated from Ubuntu
    and no longer apply cleanly. We will get these back in a future upstream
    development release.
  * debian/patches/*: Refresh to apply cleanly
  * debian/rules: Add new include dir to autoreconf call to pick up f-spot
    macros
 -- Jo Shields <email address hidden> Mon, 24 May 2010 10:35:57 +0100

Changed in f-spot (Ubuntu):
status: Fix Committed → Fix Released
Changed in f-spot (Debian):
status: Confirmed → Fix Released
Changed in f-spot:
importance: Unknown → Low
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.