[gstreamer-bugs] [Bug 623040] Add release_datetime field to GstPluginDesc

[GStreamer (bugzilla.gnome.org)]

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.