Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!

Bug 531330

Summary: media-gfx/gnome-screenshot: RFE: Do not use colon in filename output
Product: Gentoo Linux Reporter: Kristian Fiskerstrand (RETIRED) <k_f>
Component: [OLD] GNOMEAssignee: Gentoo Linux Gnome Desktop Team <gnome>
Status: RESOLVED FIXED    
Severity: enhancement    
Priority: Normal    
Version: unspecified   
Hardware: All   
OS: Linux   
URL: https://bugzilla.gnome.org/show_bug.cgi?id=740520
Whiteboard:
Package list:
Runtime testing required: ---
Attachments: gnome-screenshot-3.12.0-remove-colon-from-time.patch

Description Kristian Fiskerstrand (RETIRED) gentoo-dev 2014-12-01 12:51:43 UTC
Created attachment 390708 [details, diff]
gnome-screenshot-3.12.0-remove-colon-from-time.patch

Currently the default filename of output from gnome-screenshot include a colon for the time parameter. This is incompatible with filename convention for at least the following cases; (i) owncloud synchronization (ii) dropbox (iii) accessing a shared folder from windows. 

An upstream build exists for this on https://bugzilla.gnome.org/show_bug.cgi?id=740520 and ubuntu has a similar bug for it at https://bugs.launchpad.net/ubuntu/+source/gnome-screenshot/+bug/1054808. The attached patch applies cleanly to gnome-screenshot-3.12.0 replacing colon with - allowing me to synchronize the image folder with my owncloud instance.
Comment 1 Pacho Ramos gentoo-dev 2014-12-02 09:16:40 UTC
Upstream looks to disagree specially with dropping the spaces... could you give some concrete examples to upstream bug too as maybe that could help to get the patch upstreamed at some point?
https://bugzilla.gnome.org/show_bug.cgi?id=740520#c2
Comment 2 Kristian Fiskerstrand (RETIRED) gentoo-dev 2014-12-02 12:32:57 UTC
Please note that this patch does not affect spaces at all, only the colon. The spaces doesn't cause any incompatibility for me and I believe upstream does not want to remove it as it affects e.g. translation.
Comment 3 Kristian Fiskerstrand (RETIRED) gentoo-dev 2015-01-28 19:54:17 UTC
This patch is now included upstream
Comment 4 Pacho Ramos gentoo-dev 2015-02-03 11:04:05 UTC
+*gnome-screenshot-3.14.0-r1 (03 Feb 2015)
+
+  03 Feb 2015; Pacho Ramos <pacho@gentoo.org>
+  +files/gnome-screenshot-3.14.0-use-dash.patch,
+  +gnome-screenshot-3.14.0-r1.ebuild, -gnome-screenshot-3.14.0.ebuild:
+  Use dash for time format separator (#531330 by Kristian Fiskerstrand), drop
+  old
+