[Date Prev][Date Next] [Thread Prev][Thread Next] [Date Index] [Thread Index]

Bug#854811: marked as done (jessie-pu: package gnome-screenshot/3.14.0-1)

Your message dated Sat, 06 May 2017 14:44:18 +0100
with message-id <1494078258.26551.13.camel@adam-barratt.org.uk>
and subject line Closing bugs for updates included in 8.8
has caused the Debian Bug report #854811,
regarding jessie-pu: package gnome-screenshot/3.14.0-1
to be marked as done.

This means that you claim that the problem has been dealt with.
If this is not the case it is now your responsibility to reopen the
Bug report if necessary, and/or fix the problem forthwith.

(NB: If you are a system administrator and have no idea what this
message is talking about, this may indicate a serious mail system
misconfiguration somewhere. Please contact owner@bugs.debian.org
immediately.)


-- 
854811: http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=854811
Debian Bug Tracking System
Contact owner@bugs.debian.org with problems
--- Begin Message --- 
Package: release.debian.org
Severity: normal
Tags: jessie
User: release.debian.org@packages.debian.org
Usertags: pu

If used without GUI (e.g. by pressing the PrintScr key or from a shell with no
arguments), gnome-screenshot will simply take a screenshot of the whole screen
and store it in the pictures directory with a filename containing the current
timestamp with colons (i.e. "Screenshot from 2017-01-10 16:46:39.png").
This leads to problems with some filesystems (e.g. vfat, ntfs).

See package bug #850836 (https://bugs.debian.org/cgi-
bin/bugreport.cgi?bug=850836)

I have created a patch from a combination of the patch from upstream bug
#698740 and upstream commit aa23783 to achieve the behaviour intended by
successive upstream releases.
The rationale behind this to not just import the bug from the upstream
bug report is to not disrupt user experience once now in Jessie and then
another time when Stretch is released but just once now.

I added the patch to debian/patches, updated the changelog, built the
package, tested it and attached the debdiff here (and FWIW uploaded it
to https://mentors.debian.net/package/gnome-screenshot)

-- System Information:
Debian Release: 8.7
  APT prefers stable-updates
  APT policy: (700, 'stable-updates'), (700, 'stable'), (60, 'testing')
Architecture: amd64 (x86_64)

Kernel: Linux 3.16.0-4-amd64 (SMP w/8 CPU cores)
Locale: LANG=en_US.utf8, LC_CTYPE=en_US.utf8 (charmap=UTF-8)

diff -Nru gnome-screenshot-3.14.0/debian/changelog gnome-screenshot-3.14.0/debian/changelog
--- gnome-screenshot-3.14.0/debian/changelog	2014-09-23 15:43:32.000000000 +0200
+++ gnome-screenshot-3.14.0/debian/changelog	2017-02-08 11:00:32.000000000 +0100
@@ -1,3 +1,14 @@
+gnome-screenshot (3.14.0-1.1) UNRELEASED; urgency=medium
+
+  * Non-maintainer upload.
+  * filename-builder-use-dash-for-time-format-separator.patch:
+    Combination of the patch from upstream bug #698740 and upstream
+    commit aa23783 to achieve the behaviour intended by successive
+    upstream releases.
+    (Closes: #850836)
+
+ -- Moritz Schlarb <schlarbm@uni-mainz.de>  Wed, 08 Feb 2017 10:28:37 +0100
+
 gnome-screenshot (3.14.0-1) unstable; urgency=medium
 
   * New upstream release.
diff -Nru gnome-screenshot-3.14.0/debian/patches/filename-builder-use-dash-for-time-format-separator.patch gnome-screenshot-3.14.0/debian/patches/filename-builder-use-dash-for-time-format-separator.patch
--- gnome-screenshot-3.14.0/debian/patches/filename-builder-use-dash-for-time-format-separator.patch	1970-01-01 01:00:00.000000000 +0100
+++ gnome-screenshot-3.14.0/debian/patches/filename-builder-use-dash-for-time-format-separator.patch	2017-02-08 10:53:06.000000000 +0100
@@ -0,0 +1,20 @@
+Subject: filename-builder: use dash for time format separator
+
+This patch is a combination of the patch from upstream bug #698740
+and upstream commit aa23783 to achieve the behaviour intended by
+successive upstream releases.
+
+Bug: https://bugzilla.gnome.org/show_bug.cgi?id=698740
+Origin: https://github.com/GNOME/gnome-screenshot/commit/aa23783c39f96dd0b45f70d82557736a5b884c55
+
+--- a/src/screenshot-filename-builder.c
++++ b/src/screenshot-filename-builder.c
+@@ -131,7 +131,7 @@ build_path (AsyncExistenceJob *job)
+       GDateTime *d;
+ 
+       d = g_date_time_new_now_local ();
+-      origin = g_date_time_format (d, "%Y-%m-%d %H:%M:%S");
++      origin = g_date_time_format (d, "%Y-%m-%d %H-%M-%S");
+       g_date_time_unref (d);
+     }
+   else
diff -Nru gnome-screenshot-3.14.0/debian/patches/series gnome-screenshot-3.14.0/debian/patches/series
--- gnome-screenshot-3.14.0/debian/patches/series	1970-01-01 01:00:00.000000000 +0100
+++ gnome-screenshot-3.14.0/debian/patches/series	2017-02-08 10:40:56.000000000 +0100
@@ -0,0 +1 @@
+filename-builder-use-dash-for-time-format-separator.patch

--- End Message ---
--- Begin Message --- 
Version: 8.8

Hi,

Each of these bugs refers to an update that was included in today's
jessie point release. Thanks!

Regards,

Adam

--- End Message ---
Reply to: