Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 531330 - media-gfx/gnome-screenshot: RFE: Do not use colon in filename output
Summary: media-gfx/gnome-screenshot: RFE: Do not use colon in filename output
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: [OLD] GNOME (show other bugs)
Hardware: All Linux
: Normal enhancement (vote)
Assignee: Gentoo Linux Gnome Desktop Team
URL: https://bugzilla.gnome.org/show_bug.c...
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2014-12-01 12:51 UTC by Kristian Fiskerstrand (RETIRED)
Modified: 2015-02-03 11:04 UTC (History)
0 users

See Also:
Package list:
Runtime testing required: ---


Attachments
gnome-screenshot-3.12.0-remove-colon-from-time.patch (gnome-screenshot-3.12.0-remove-space-and-colon-from-time.patch,448 bytes, patch)
2014-12-01 12:51 UTC, Kristian Fiskerstrand (RETIRED)
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
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
+