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

2009-11-19 Thread Philip Van Hoof
Pushed Thanks! On Thu, 2009-11-19 at 13:48 +0100, Juan A. Suarez Romero wrote: > On Thu, 2009-11-19 at 10:31 +0100, Philip Van Hoof wrote: > > 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 us

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

2009-11-19 Thread Martyn Russell
On 19/11/09 20:48, Juan A. Suarez Romero wrote: On Thu, 2009-11-19 at 10:31 +0100, Philip Van Hoof wrote: 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

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

2009-11-19 Thread Martyn Russell
On 19/11/09 17:27, Juan A. Suarez Romero wrote: On Thu, 2009-11-19 at 10:09 +0100, Philip Van Hoof wrote: Hey Martyn, Hej :) 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 thi

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

2009-11-19 Thread Juan A. Suarez Romero
On Thu, 2009-11-19 at 10:31 +0100, Philip Van Hoof wrote: > 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. > Attached are the pa

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

2009-11-19 Thread Philip Van Hoof
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

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

2009-11-19 Thread Juan A. Suarez Romero
On Thu, 2009-11-19 at 10:09 +0100, Philip Van Hoof wrote: > Hey Martyn, > > 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 pat

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

2009-11-19 Thread Philip Van Hoof
Hey Martyn, I'll await the conclusions of this discussion between you and Juan before making any changes to Juan's patches. If you guys reached a consensus, feel free to send me a corrections- patch, Juan (of course). On Thu, 2009-11-19 at 09:59 +0100, Juan A. Suarez Romero wrote: > On Thu, 2009

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

2009-11-19 Thread Juan A. Suarez Romero
On Thu, 2009-11-19 at 13:27 +0800, Martyn Russell wrote: > Hi all, > > Philip, can you please change the patches to not use -v. It is not a > good option to use. It is typically used for verbosity. Please use -V > instead. Really, there isn't an agree about -v or -V to show version. A lot of pr

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

2009-11-18 Thread Martyn Russell
On 18/11/09 17:20, Philip Van Hoof wrote: Thanks a lot Juan! These patches are all git-am'ed and pushed into master. On Wed, 2009-11-18 at 10:08 +0100, Juan A. Suarez Romero wrote: These set of patches add the typical "--version" option to command-line tools. In some way, these patches helps

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

2009-11-18 Thread Philip Van Hoof
Thanks a lot Juan! These patches are all git-am'ed and pushed into master. On Wed, 2009-11-18 at 10:08 +0100, Juan A. Suarez Romero wrote: > These set of patches add the typical "--version" option to command-line tools. > > In some way, these patches helps to solve comment #2 in bug GB#601799. >

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

2009-11-18 Thread Juan A. Suarez Romero
These set of patches add the typical "--version" option to command-line tools. In some way, these patches helps to solve comment #2 in bug GB#601799. --- Juan A. Suarez Romero (10): tracker-status: add "--version" option tracker-stats: add "--version" option tracker-tag: add "--version" o