Re: [Tracker] [PATCH 00/10] Add option to show version

[Philip Van Hoof <spam pvanhoof be>]

On Thu, 2009-11-19 at 10:27 +0100, Juan A. Suarez Romero wrote:
> On Thu, 2009-11-19 at 10:09 +0100, Philip Van Hoof wrote:

Hey Juan,

> > I'll await the conclusions of this discussion between you and Juan
> > before making any changes to Juan's patches.
>
> I don't mind either using -V or -v. So if Martyn thinks it's better to
> use "-V", I'll send you a new patch that changes it.
>
> Also, I'll try to keep up-to-date manpages.
>
> But in any case, Martyn was doubting about if this option makes sense or
> not. I think it makes sense, as it is an option that all programs have
> (or should) have.

I think it makes sense too. For example tracker-sparql will not be
compatible if we make a change to SparqlUpdate and BatchSparqlUpdate's
return value that we are or might be planning (being decided this week).

For that users of the programs must have the ability to validate
versions. There are of course many more use-cases for version strings.

> So, if finally the option makes sense for you guys, I'll send an updated
> version of patch. Else, feel free to revert the commit.

Perhaps make the patch for the man pages already. Martyn is absolutely
right that we need to keep those up to date.

I'd be fine with renaming -v to -V to avoid collision with verbosity.
Perhaps look at the tracker-store and tracker-miner-fs tools whether
they use -v for verbosity, and if so perhaps indeed better rename to -V.



-- 
Philip Van Hoof, freelance software developer
home: me at pvanhoof dot be 
gnome: pvanhoof at gnome dot org 
http://pvanhoof.be/blog
http://codeminded.be