eog hangs when the print dialog is closed

Bug #1262801 reported by Bruno Nova on 2013-12-19
4
Affects Status Importance Assigned to Milestone
eog (Ubuntu)
High
Unassigned
Saucy
High
Unassigned

Bug Description

Impact: When viewing an image in eog (the GNOME image viewer), if I open the Print dialog through the "Image > Print" menu, eog will hang, and be grayed out

Test Case: Under Unity, open eog, use appmenu to open the print dialog, close it, eog shouldn't hang

Regression potential: check that eog keeps working, including plugins options

Bruno Nova (brunonova) wrote :
Sebastien Bacher (seb128) wrote :

Thank you for your bug report. That seems somewhat related to unity menu, it doesn't happen either when using "UBUNTU_MENUPROXY= eog image"

Changed in eog (Ubuntu):
importance: Undecided → High
status: New → Confirmed
Lars Karlitski (larsu) wrote :

The patch I've attached to [1] fixes the issue, but I'm not sure if it causes other problems. I'm waiting to hear from a eog maintainer for an opinion.

[1] https://bugzilla.gnome.org/show_bug.cgi?id=721755

Changed in eog (Ubuntu):
status: Confirmed → Triaged
Sebastien Bacher (seb128) wrote :

eog (3.10.2-0ubuntu3) trusty; urgency=medium

  * debian/patches/git_dialog_dont_lock.patch:
    - don't lock when closing dialogs, thanks Lars Uebernickel (lp:#1262801)

Changed in eog (Ubuntu):
status: Triaged → Fix Released
Bruno Nova (brunonova) wrote :

It would be nice if this bug was fixed for Saucy, but the fix wasn't backported yet.

I got eog's source code for Saucy (apt-get source eog) and applied the changes mentioned in the original patch here: <https://bug721755.bugzilla-attachments.gnome.org/attachment.cgi?id=265878> (the affected line numbers were different).

The backported patch is attached. The patch seems to fix the bug for Saucy, but I don't know if it creates additional issues.
I look forward to see this fixed in Saucy!
Meanwhile, thanks to the author of the original patch, Lars Uebernickel!

Changed in eog (Ubuntu Saucy):
status: New → Triaged
importance: Undecided → High
description: updated
Sebastien Bacher (seb128) wrote :

Thanks Bruno, I'm uploaded a SRU to saucy for that issue

Hello Bruno, or anyone else affected,

Accepted eog into saucy-proposed. The package will build now and be available at http://launchpad.net/ubuntu/+source/eog/3.8.2-1ubuntu1.1 in a few hours, and then in the -proposed repository.

Please help us by testing this new package. See https://wiki.ubuntu.com/Testing/EnableProposed for documentation how to enable and use -proposed. Your feedback will aid us getting this update out to other Ubuntu users.

If this package fixes the bug for you, please add a comment to this bug, mentioning the version of the package you tested, and change the tag from verification-needed to verification-done. If it does not fix the bug for you, please add a comment stating that, and change the tag to verification-failed. In either case, details of your testing will help us make a better decision.

Further information regarding the verification process can be found at https://wiki.ubuntu.com/QATeam/PerformingSRUVerification . Thank you in advance!

Changed in eog (Ubuntu Saucy):
status: Triaged → Fix Committed
tags: added: verification-needed
Bruno Nova (brunonova) wrote :

I installed the version 3.8.2-1ubuntu1.1 of eog from -proposed and tested it.

I opened an image with eog, accessed the print dialog using all methods (menu, HUD, Ctrl+P) then closed it using several methods (cancel, close button, preview, print to file). Eog didn't hang and was still responsive and working.

I tested all options in the preferences dialog, including the 3 plugins, and they all still work.
I also tested several other options in the menus and didn't find any issue.

Now, I'm the one who backported Lars Uebernickel's patch to Saucy, but I'm also the one who reported the bug and the only one marked as "affected".

tags: added: verification-done
removed: verification-needed
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package eog - 3.8.2-1ubuntu1.1

---------------
eog (3.8.2-1ubuntu1.1) saucy; urgency=medium

  * debian/patches/git_dialog_dont_lock.patch: don't lock when closing
    dialogs, thanks Bruno Nova for the backported version (lp: #1262801)
 -- Sebastien Bacher <email address hidden> Fri, 31 Jan 2014 15:50:25 +0100

Changed in eog (Ubuntu Saucy):
status: Fix Committed → Fix Released

The verification of the Stable Release Update for eog has completed successfully and the package has now been released to -updates. Subsequently, the Ubuntu Stable Release Updates Team is being unsubscribed and will not receive messages about this bug report. In the event that you encounter a regression using the package from -updates please report a new bug using ubuntu-bug and tag the bug report regression-update so we can easily find any regresssions.

To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.