[gstreamer-bugs] [Bug 623040] Add release_datetime field to GstPluginDesc
GStreamer (bugzilla.gnome.org)
bugzilla at gnome.org
Mon Jun 28 06:20:32 PDT 2010
https://bugzilla.gnome.org/show_bug.cgi?id=623040
GStreamer | gstreamer (core) | git
Tim-Philipp Müller <t.i.m> changed:
What |Removed |Added
----------------------------------------------------------------------------
Status|UNCONFIRMED |NEW
Ever Confirmed|0 |1
--- Comment #6 from Tim-Philipp Müller <t.i.m at zen.co.uk> 2010-06-28 13:20:27 UTC ---
In the first iteration of this patch I actually used a simple integer (decimal
number YYYYMMDDHH). I also considered using unix timestamps, but those are not
very nice to deal with in configure.ac and Makefiles, e.g. when hard-coding a
date (as some external projects may want to do) or trying to convert the date
string from the .doap file into a suitable value.
In the end I just thought it was all very silly and the right thing to do was
to go for a string in a well-defined and easy-to-parse format.
I don't think there's a parsing "issue" (we can always replace the checking
function with a 1-line sscanf if it bugs you, or skip the check altogether for
now). While it would be nice if we could use the glib function mentioned in the
code, I also felt adding seconds to the time format was overdoing it a little.
We could probably get away without any time even, but I felt it should be there
for completeness (and same-day brown-paperbag releases etc.).
--
Configure bugmail: https://bugzilla.gnome.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the QA contact for the bug.
You are the assignee for the bug.
More information about the Gstreamer-bugs
mailing list