eog does not save changes if image is rotated and flipped once

Bug #657416 reported by Oliver Joos
10
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Eye of GNOME
Fix Released
Medium
eog (Ubuntu)
Fix Released
Low
Unassigned

Bug Description

Binary package hint: eog

I use Ubuntu maverick 10.10 (release candidate) with eog 2.32.0-0ubuntu1.

1. open a jpeg image in eog
2. rotate it once (left or right)
3. flip it (vertically or horizontally)
4. save it explicitly or close eog and answer yes to save it
=> the image file is updated, but the image content did not change!

Further I saw:
- if rotating TWICE left or right and flipping once, it works as expected
- if rotating three times and then flipping is buggy like rotating once

ProblemType: Bug
DistroRelease: Ubuntu 10.10
Package: eog 2.32.0-0ubuntu1
ProcVersionSignature: Ubuntu 2.6.35-22.33-generic 2.6.35.4
Uname: Linux 2.6.35-22-generic i686
Architecture: i386
Date: Sat Oct 9 20:35:05 2010
LiveMediaBuild: Ubuntu 10.10 "Maverick Meerkat" - Release Candidate i386 (20100928)
ProcEnviron:
 LANG=de_CH.utf8
 SHELL=/bin/bash
SourcePackage: eog

Tags: patch
Revision history for this message
Oliver Joos (oliver-joos) wrote :
Revision history for this message
Pedro Villavicencio (pedro) wrote :
Changed in eog (Ubuntu):
importance: Undecided → Low
Revision history for this message
Oliver Joos (oliver-joos) wrote :

Thanks, I read 629960: cause may be the same, but symptoms (and versions) are different. So I opened a new bug https://bugzilla.gnome.org/show_bug.cgi?id=632290 and mentioned there the similar bug. Perhaps both can be solved with the same fix.

Changed in eog:
importance: Unknown → Medium
status: Unknown → New
Changed in eog:
status: New → Fix Released
Revision history for this message
Oliver Joos (oliver-joos) wrote :

A fix has been committed upstream!

Changed in eog (Ubuntu):
status: New → Confirmed
Revision history for this message
Benjamin Bach (benjaoming) wrote :

Patch containing bug fixes from 2.32.1 - I do not know how to create new packages, but I have tested the patch, and it works. It addresses the follow commits from EOG svn, including the above-mentioned fix.

+ commit 389db0ff522f4d471e42e34ce3607368b7deb26b
+ Author: Felix Riemann <email address hidden>
+ Date: Fri Oct 15 17:29:36 2010 +0200
+
+ Don't try to thumbnail named pipes in the file open dialog
+
+ In fact this means all "special" files like sockets and device files.
+ This prevents lockups due to file I/O blocking here. Fixes bug 623271.
+
+ src/eog-file-chooser.c | 4 +++-
+ 1 files changed, 3 insertions(+), 1 deletions(-)

+ commit e43fa8ff82c6aedd8f6373a0a5a1a5677f32cd83
+ Author: Felix Riemann <email address hidden>
+ Date: Sat Nov 13 14:08:44 2010 +0100
+
+ Workaround concurrency issue with statusbar-date plugin
+
+ Make sure that only an image loading job does autorotation. A metadata
+ loading job has no use for this. Works around lost signals when the
+ statusbar-date plugin is activated. Works around bug 622161.
+
+ src/eog-image.c | 6 ++++--
+ 1 files changed, 4 insertions(+), 2 deletions(-)

+ commit 0aafa6cf28c38aa3d72c98d0651e89919b3348a3
+ Author: Felix Riemann <email address hidden>
+ Date: Sat Oct 16 22:41:56 2010 +0200
+
+ Fix calculation of reference matrices for transpose/transverse
+
+ The flip matrices were applied as if the source matrix were an identity
+ matrix instead of properly multiplicating them.
+
+ src/eog-transform.c | 16 ++++++++++------
+ 1 files changed, 10 insertions(+), 6 deletions(-)
+

+ commit 21639caabe2ad4993f97bc8fe90eb8ca5a6de20f
+ Author: Felix Riemann <email address hidden>
+ Date: Sat Oct 16 22:50:50 2010 +0200
+
+ Support transpose/transverse as lossless JPEG transformations
+
+ Also implement a warning message if unsupported transformations
+ are used to easier discover such cases in the future. Fixes bug 632290.
+
+ src/eog-image-jpeg.c | 9 +++++++++
+ 1 files changed, 9 insertions(+), 0 deletions(-)

tags: added: patch
Revision history for this message
Mörgæs (moergaes) wrote :

Fixed upstream long time ago.

Changed in eog (Ubuntu):
status: Confirmed → Fix Released
Revision history for this message
Oliver Joos (oliver-joos) wrote :

As reporter of this bug I confirm that this bug is fixed - at least in Eye of MATE 1.12.0 of Linux Mint 17.3.

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.