Re: [PATCH] metadata-keys: add metadata keys from camera pictures



On Thu, 2011-03-17 at 12:34 +0100, Juan A. Suárez Romero wrote:
> On Thu, 2011-03-17 at 11:21 +0000, Lionel Landwerlin wrote:
> > On Thu, 2011-03-17 at 12:20 +0100, Juan A. Suárez Romero wrote:
> > > On Thu, 2011-03-17 at 11:06 +0000, Lionel Landwerlin wrote:
> > > > Here, I'm just trying to expose 2 different data. In our application
> > > > we
> > > > want to use both, the insertion date being used to present newly
> > > > downloaded items and creation date used in pictures description (and
> > > > maybe later, used to sort pictures/videos/etc... in a different way it
> > > > is today).
> > > > 
> > > 
> > > Yes, and in fact this is my point. You want to use both dates, and
> > > currently we have a GRL_METADATA_KEY_DATE already on Grilo.
> > > 
> > > So how to use it?
> > > 
> > > a) Use GRL_METADATA_KEY_DATE for creation date and create a new one for
> > > GRL_METADATA_KEY_INSERTION_DATE
> > > 
> > > b) The other way around: use GRL_METADATA_KEY_DATE for insertion date
> > > and create a new one for GRL_METADATA_KEY_CREATION_DATE.
> > > 
> > > 
> > > So, what option to choose? I think we should choose (a), but it's a very
> > > personal opinion, as all reasons I exposed previously. And that is why I
> > > would like to hear other opinions too.
> > > 
> > > 
> > > 	J.A.
> > 
> > c) Get rid of GRL_METADATA_KEY_DATE and add
> > GRL_METADATA_KEY_CREATION_DATE and GRL_METADATA_KEY_INSERTION_DATE ?
> > 
> > d) Add GRL_METADATA_KEY_CREATION_DATE and
> > GRL_METADATA_KEY_INSERTION_DATE and make GRL_METADATA_KEY_DATE related
> > to one of them (is that possible ?)
> > 
> 
> Yes that could be possible too.
> 
> Thus, is up to the source to determine the meaning of
> GRL_METADATA_KEY_DATE.
> 
> 	J.A.

I rather see that as a compatibility transition to not break everything.
With the related key being a statement made by the Grilo framework.

--
Lionel Landwerlin




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