--- Begin Message ---
- To: Debian Bug Tracking System <submit@bugs.debian.org>
- Subject: jessie-pu: package eom/1.8.0+dfsg1-4+deb8u1
- From: Pablo Barciela <scow@riseup.net>
- Date: Sat, 22 Apr 2017 22:07:38 +0200
- Message-id: <149289165860.13452.3618995544120211026.reportbug@debian.debian>
Package: release.debian.org
Severity: normal
Tags: jessie
User: release.debian.org@packages.debian.org
Usertags: pu
Please apply this update:
- Add the dependency libgl1-mesa-glx
- Fix new windows don't get focus.
- Don't change file permissions when saving the modified image
All fixed in testing, and this is the fix for jessie
Best Regards
Pablo Barciela
Debian MATE team member.
-- System Information:
Debian Release: 9.0
APT prefers testing
APT policy: (500, 'testing'), (1, 'experimental')
Architecture: amd64 (x86_64)
Foreign Architectures: i386
Kernel: Linux 4.9.0-2-amd64 (SMP w/2 CPU cores)
Locale: LANG=C.UTF-8, LC_CTYPE=C.UTF-8 (charmap=UTF-8)
diff -Nru eom-1.8.0+dfsg1/debian/changelog eom-1.8.0+dfsg1/debian/changelog
--- eom-1.8.0+dfsg1/debian/changelog 2014-07-01 19:40:50.000000000 +0200
+++ eom-1.8.0+dfsg1/debian/changelog 2017-04-22 02:57:43.000000000 +0200
@@ -1,3 +1,17 @@
+eom (1.8.0+dfsg1-4+deb8u1) jessie-proposed-updates; urgency=medium
+
+ * debian/control:
+ + Add D (eom): libgl1-mesa-glx. (Closes: #844844).
+
+ * debian/patches:
+ + Add 0001_fix-new-windows-dont-get-focus.patch. Fix new windows
+ don't get focus. (Closes: #813323).
+ + Add 0002_fix-permissions-when-saving-the-modified-image.patch.
+ don't change file permissions when saving the modified image
+ (Closes: #769792).
+
+ -- Pablo Barciela <scow@riseup.net> Sat, 22 Apr 2017 02:57:43 +0200
+
eom (1.8.0+dfsg1-4) unstable; urgency=medium
* Add librsvg features to Eye of MATE. (Closes: #752282).
diff -Nru eom-1.8.0+dfsg1/debian/control eom-1.8.0+dfsg1/debian/control
--- eom-1.8.0+dfsg1/debian/control 2014-07-01 19:40:30.000000000 +0200
+++ eom-1.8.0+dfsg1/debian/control 2017-04-22 02:53:57.000000000 +0200
@@ -47,6 +47,7 @@
mate-desktop-common,
mate-icon-theme,
shared-mime-info,
+ libgl1-mesa-glx,
Recommends: librsvg2-common
Breaks: mate-image-viewer (<= 1.0.0-1)
Replaces: mate-image-viewer (<= 1.0.0-1)
diff -Nru eom-1.8.0+dfsg1/debian/patches/0001_fix-new-windows-dont-get-focus.patch eom-1.8.0+dfsg1/debian/patches/0001_fix-new-windows-dont-get-focus.patch
--- eom-1.8.0+dfsg1/debian/patches/0001_fix-new-windows-dont-get-focus.patch 1970-01-01 01:00:00.000000000 +0100
+++ eom-1.8.0+dfsg1/debian/patches/0001_fix-new-windows-dont-get-focus.patch 2017-04-22 02:53:57.000000000 +0200
@@ -0,0 +1,32 @@
+Description: Fix new windows don't get focus
+Author: vfscanf
+Upstream commit: https://github.com/mate-desktop/eom/commit/1b65a062fc01e15c4a461463a055b90d8cb3235e
+--- a/src/eom-application.c
++++ b/src/eom-application.c
+@@ -279,8 +279,24 @@ eom_application_get_file_window (EomApplication *application, GFile *file)
+ static void
+ eom_application_show_window (EomWindow *window, gpointer user_data)
+ {
+- gtk_window_present_with_time (GTK_WINDOW (window),
+- GPOINTER_TO_UINT (user_data));
++ guint32 timestamp = GPOINTER_TO_UINT (user_data);
++
++ /* set the proper interaction time on the window.
++ * Fall back to roundtripping to the X server when we
++ * don't have the timestamp, e.g. when launched from
++ * terminal. We also need to make sure that the window
++ * has been realized otherwise it will not work. lame.
++ */
++ if (!gtk_widget_get_realized (GTK_WIDGET (window)))
++ gtk_widget_realize (GTK_WIDGET (window));
++
++ if (timestamp <= 0)
++ timestamp = gdk_x11_get_server_time (gtk_widget_get_window (GTK_WIDGET (window)));
++
++ gdk_x11_window_set_user_time (gtk_widget_get_window (GTK_WIDGET (window)),
++ timestamp);
++
++ gtk_window_present (GTK_WINDOW (window));
+ }
+
+ /**
diff -Nru eom-1.8.0+dfsg1/debian/patches/0002_fix-permissions-when-saving-the-modified-image.patch eom-1.8.0+dfsg1/debian/patches/0002_fix-permissions-when-saving-the-modified-image.patch
--- eom-1.8.0+dfsg1/debian/patches/0002_fix-permissions-when-saving-the-modified-image.patch 1970-01-01 01:00:00.000000000 +0100
+++ eom-1.8.0+dfsg1/debian/patches/0002_fix-permissions-when-saving-the-modified-image.patch 2017-04-22 02:54:30.000000000 +0200
@@ -0,0 +1,137 @@
+From 02d0316da724f63026bb804699dcbc6875083911 Mon Sep 17 00:00:00 2001
+From: Monsta <monsta@inbox.ru>
+Date: Thu, 4 Jun 2015 11:19:21 +0300
+Subject: [PATCH] don't change file permissions when saving the modified image
+
+adapted from:
+https://git.gnome.org/browse/eog/commit/?id=4626596c2c179bfe35c4212efced15c38d7337d6
+---
+ src/eom-image.c | 108 ++++++++++++++++++++++++++++++++++++++++++++++++++++++++
+ 1 file changed, 108 insertions(+)
+
+diff --git a/src/eom-image.c b/src/eom-image.c
+index 989f365..3ea28d4 100644
+--- a/src/eom-image.c
++++ b/src/eom-image.c
+@@ -1431,6 +1431,110 @@ transfer_progress_cb (goffset cur_bytes,
+ }
+ }
+
++static void
++tmp_file_restore_unix_attributes (GFile *temp_file,
++ GFile *target_file)
++{
++ GFileInfo *file_info;
++ guint uid;
++ guint gid;
++ guint mode;
++ guint mode_mask = 00600;
++
++ GError *error = NULL;
++
++ g_return_if_fail (G_IS_FILE (temp_file));
++ g_return_if_fail (G_IS_FILE (target_file));
++
++ /* check if file exists */
++ if (!g_file_query_exists (target_file, NULL)) {
++ eom_debug_message (DEBUG_IMAGE_SAVE,
++ "Target file doesn't exist. Setting default attributes.");
++ return;
++ }
++
++ /* retrieve UID, GID, and MODE of the original file info */
++ file_info = g_file_query_info (target_file,
++ "unix::uid,unix::gid,unix::mode",
++ G_FILE_QUERY_INFO_NONE,
++ NULL,
++ &error);
++
++ /* check that there aren't any error */
++ if (error != NULL) {
++ eom_debug_message (DEBUG_IMAGE_SAVE,
++ "File information not available. Setting default attributes.");
++
++ /* free objects */
++ g_object_unref (file_info);
++ g_clear_error (&error);
++
++ return;
++ }
++
++ /* save UID, GID and MODE values */
++ uid = g_file_info_get_attribute_uint32 (file_info,
++ G_FILE_ATTRIBUTE_UNIX_UID);
++
++ gid = g_file_info_get_attribute_uint32 (file_info,
++ G_FILE_ATTRIBUTE_UNIX_GID);
++
++ mode = g_file_info_get_attribute_uint32 (file_info,
++ G_FILE_ATTRIBUTE_UNIX_MODE);
++
++ /* apply default mode mask to file mode */
++ mode |= mode_mask;
++
++ /* restore original UID, GID, and MODE into the temporal file */
++ g_file_set_attribute_uint32 (temp_file,
++ G_FILE_ATTRIBUTE_UNIX_UID,
++ uid,
++ G_FILE_QUERY_INFO_NONE,
++ NULL,
++ &error);
++
++ /* check that there aren't any error */
++ if (error != NULL) {
++ eom_debug_message (DEBUG_IMAGE_SAVE,
++ "You do not have the permissions necessary to change the file UID.");
++
++ g_clear_error (&error);
++ }
++
++ g_file_set_attribute_uint32 (temp_file,
++ G_FILE_ATTRIBUTE_UNIX_GID,
++ gid,
++ G_FILE_QUERY_INFO_NONE,
++ NULL,
++ &error);
++
++ /* check that there aren't any error */
++ if (error != NULL) {
++ eom_debug_message (DEBUG_IMAGE_SAVE,
++ "You do not have the permissions necessary to change the file GID. Setting user default GID.");
++
++ g_clear_error (&error);
++ }
++
++ g_file_set_attribute_uint32 (temp_file,
++ G_FILE_ATTRIBUTE_UNIX_MODE,
++ mode,
++ G_FILE_QUERY_INFO_NONE,
++ NULL,
++ &error);
++
++ /* check that there aren't any error */
++ if (error != NULL) {
++ eom_debug_message (DEBUG_IMAGE_SAVE,
++ "You do not have the permissions necessary to change the file MODE.");
++
++ g_clear_error (&error);
++ }
++
++ /* free objects */
++ g_object_unref (file_info);
++}
++
+ static gboolean
+ tmp_file_move_to_uri (EomImage *image,
+ GFile *tmpfile,
+@@ -1441,6 +1545,10 @@ tmp_file_move_to_uri (EomImage *image,
+ gboolean result;
+ GError *ioerror = NULL;
+
++ /* try to restore target file unix attributes */
++ tmp_file_restore_unix_attributes (tmpfile, file);
++
++ /* replace target file with temporal file */
+ result = g_file_move (tmpfile,
+ file,
+ (overwrite ? G_FILE_COPY_OVERWRITE : 0) |
diff -Nru eom-1.8.0+dfsg1/debian/patches/series eom-1.8.0+dfsg1/debian/patches/series
--- eom-1.8.0+dfsg1/debian/patches/series 2014-06-04 01:12:47.000000000 +0200
+++ eom-1.8.0+dfsg1/debian/patches/series 2017-04-22 02:54:30.000000000 +0200
@@ -1,2 +1,4 @@
+0001_fix-new-windows-dont-get-focus.patch
+0002_fix-permissions-when-saving-the-modified-image.patch
1001_fix-hyphens-on-man-page.patch
2001_omit-gfdl-licensed-help-files.patch
--- End Message ---
--- Begin Message ---
- To: Pablo Barciela <scow@riseup.net>, 860966-done@bugs.debian.org
- Subject: Re: Bug#860966: jessie-pu: package eom/1.8.0+dfsg1-4+deb8u1
- From: Julien Cristau <jcristau@debian.org>
- Date: Sat, 9 Sep 2017 21:39:30 +0200
- Message-id: <20170909193930.cfevxed74ho4f4jh@betterave.cristau.org>
- In-reply-to: <f2fa25cb796cf7ac74441fc20b0499a4@riseup.net>
- References: <149289165860.13452.3618995544120211026.reportbug@debian.debian> <20170501140402.ngrmi677abxpuh2z@betterave.cristau.org> <f2fa25cb796cf7ac74441fc20b0499a4@riseup.net>
Control: tag -1 wontfix
On Tue, May 2, 2017 at 21:12:47 +0200, Pablo Barciela wrote:
> > > - Add the dependency libgl1-mesa-glx
> > >
> > This is wrong...
>
> You can forget the dependency, my last update proposal don't have the
> dependency (attached here again).
>
> > > - Fix new windows don't get focus.
>
> I open eom with previous instances and eom opens minimized, the patch is
> the fix.
>
> > > - Don't change file permissions when saving the modified image
>
> The bug: open an image file with eom with permissions 0666, rotate -> close
> -> and save, and eom saves with permissions 0600, the patch solves it and
> eom saves with 0666
>
These don't sound like critical issues to me, I'm going to call this
wontfix for jessie, sorry. With apologies for the delay.
Cheers,
Julien
--- End Message ---