Don't edit files

Bug #69577 reported by Tom Harris
20
This bug affects 3 people
Affects Status Importance Assigned to Milestone
F-Spot
Fix Released
Medium
f-spot (Ubuntu)
Fix Released
Low
Ubuntu Desktop Bugs

Bug Description

If I rotate an image in F-Spot it changes the original file, and changes the exif software tag to read "f-spot". All this is done without warning. My camera detects any photos with software tags different to the firmware of the camera as being corrupted. Thus photos viewed in f-spot can no longer be viewed on the original camera. Picasa is very good in this regard in that it never edits the original file. F-Spot should be the same. Currently I have to set my Photos folder to read-only to "save" my photos from f-spot!

Revision history for this message
Simon Law (sfllaw) wrote :

F-Spot is supposed to save modified files under a different filename. Does this also happen when you crop a photograph?

Changed in f-spot:
assignee: nobody → sfllaw
status: Unconfirmed → Needs Info
Revision history for this message
Tom Harris (tom-harris) wrote :

No, just seems to be for rotating as far as I can tell.

Revision history for this message
Tom Harris (tom-harris) wrote :

Why is this still marked as "Needs Info"? I'd be quite happy to provide more information if that helps in getting this fixed!

Revision history for this message
Tom Harris (tom-harris) wrote :

I've provided the required info and haven't received a reply in over a fortnight so I'm changing this back to Unconfirmed.

Changed in f-spot:
status: Needs Info → Unconfirmed
Revision history for this message
Simon Law (sfllaw) wrote :

I've confirmed that this is the case. Only rotation keeps the original around, which seems like quite the bug.

Changed in f-spot:
importance: Undecided → Low
status: Unconfirmed → Confirmed
assignee: sfllaw → nobody
Revision history for this message
Tom Harris (tom-harris) wrote :

I've added the upstream bug on this - seems that this is (partly) broken by design. Larry Ewing says that changes to software and time EXIF tags are "intentional". Other changes are a genuine bug that is being worked on.

Changed in f-spot:
status: Unknown → Unconfirmed
Changed in f-spot:
assignee: nobody → desktop-bugs
status: Confirmed → Triaged
Revision history for this message
a (tuomasjjrasanen) wrote :

Version 0.5.0.3 is still trying to modify original files when rotating. I store my original photos as read-only. When rotating
photos in f-spot, following message appears: "Unable to rotate photo: The photo could not be rotated because it is on a read only file system..." Hopefully the behavior will be changed, original files should not be modified by f-spot.

Revision history for this message
Graham Bleach (graham-launchpad) wrote :

There are two ways to rotate a JPEG: you can either re-encode the entire image, or you can set an EXIF tag that says "when you display this image, rotate it by 90 degrees.". f-spot does the latter, which I think is probably the less intrusive way.

The original problem reported was that a camera would not read the images after f-spot had rotated the images. If you remove the 'Software' EXIF tag that f-spot adds, does the camera still refuse to open the files? You can remove the tag by installing the exif package and running this command in a terminal window:

exif --ifd=0 --tag=0x0131 --remove filename.jpg

Replace filename.jpg with the name of the file you want to change. Best take a backup first.

If you would rather that f-spot never edited files, what would you like to happen when you want to rotate the file, which does require the contents of the file to change? A new version to be created?

Revision history for this message
Tom Harris (tom-harris) wrote :

From the "Features" page on the F-Spot website:
"Editing photos in F-Spot is a breeze. Easily rotate, crop, resize, and adjust red eye and other color settings with a few simple clicks. Versioning ensures your originals are never altered"

It should create a new version of the file like it does for other changes, and like it promises to do. Or at the least, not change any tag other than the rotate one as that is the only change the user has requested.

Changed in f-spot (Ubuntu):
status: Triaged → Fix Released
status: Fix Released → Triaged
Changed in f-spot:
status: New → Confirmed
Changed in f-spot:
status: Confirmed → Fix Released
Revision history for this message
Launchpad Janitor (janitor) wrote :

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

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

  [ Iain Lane ]
  * New upstream release 0.7.1 "The Big Switch"
   + Second release of the unstable 0.7 development series. Most notably
     including a switch to Taglib#. Regressions in the amount of RAW
     types supported, will be fixed before releasing 0.8.
   + Switched to a new Taglib#-backed metadata layer. This is a huge change
     which should bring us increased stability (in the long run) and a much
     easier to maintain metadata layer than the one we previously had. Also
     enables some funky new features like XMP sidecars. (Ruben Vermeersch,
     Mike Gemünde) (LP: #279440, #69577, #260706)
   + Add support for XMP sidecars (Ruben Vermeersch)
   + Add "remove from camera after import"-option (Ruben Vermeersch)
   + Start of a massive backend cleanup (Mike Gemünde)
   + Upstreaming of a lot of Debian/Ubuntu patches (Iain Lane)
   + Several dialogs converted from glade to GtkBuilder (Ademir Mendoza,
     Nick Van Eeckhout, Peter Goetz, Trevor Buchanan)
   + Code refactorings and cleanups, increased Hyena adoption (Evan Briones,
     Paul Lange, Paul Wellner Bou)
   + Small usability improvements (Alex Launi, Anton Keks, Iain Churcher,
     Tomas Kovacik)
   + More build retooling (Bertrand Lorentz, Christian Krause, Gabriel Burt)
   + Many bugs fixed and improvements in stability: Over 85 bugs closed
     (http://bit.ly/aKyQOd)
     - Forced lowercase when import is triggered by gnome-volume-manager
       (LP: #249230)
  * ubuntu_nofuse_fix_photo_import.patch: Drop — obsolete with new import
    system.
  * Drop libfspot-linkage patch: Applied upstream
  * Drop jpegtran-linkage patch: Obsolete
  * Drop ubuntu_fix_export_crash_FlickrRemote.cs.patch: Fixed upstream
  * Refresh ubuntu_xdg-photo-dir to apply cleanly
  * Drop default_view_size patch: Fixed upstream
  * Standards-Version → 3.9.0, no changes required
  * No longer need to autoreconf
  * Add debian/clean file and remove override from rules
  * Add build-depency on gnome-doc-utils to build user help
 -- Iain Lane <email address hidden> Tue, 20 Jul 2010 16:53:00 +0100

Changed in f-spot (Ubuntu):
status: Triaged → Fix Released
Changed in f-spot:
importance: Unknown → Medium
Revision history for this message
Tom (brommage) wrote : I'm interested in your profile

Hello!
I am Julia, I am 27 y.o.
I search for boy-friend.
My photos and new message for you here:

http://hotbrides.ru

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.