[Ml-yokadi] bug is task name have a dash

[Sébastien Renard]

Le lundi 8 décembre 2008, Aurélien Gâteau a écrit :
> Sébastien Renard wrote:
> > Hello there,
> >
> > Please, review the parser patch I have juste done to allow dash if it
> > does not conflict with option (8b3cb55)
>
> Hehe, tricky :). If I understand correctly, the patch should allow one
> to type this "t_add project foo -bar", am I correct? I must confess the
> variable names are a bit confusing :)

Yes that's correct. But you must escape -k in task name because -k is a t_add 
switch !

> Anyway, I just applied it and wrote a unit-test for it. Will push when
> network is back.

Thx !

> I did not merge the 't_list -r' option though, I believe the t_list -
> t_export merge is the right way to do it. 

As you want. There's indeed lots of common things between them. The 
distinction I made when I created t_export was : 
- t_list is for screen, t_export for files (screen output of t_export is 
mainly for debug/test purpose)
- t_list can use complex filter, t_export is for all tasks


> Unless you beat me to it, I 
> will work on it in my next Yokadi day (I try to split my hacking time
> between my various project, ATM Gwenview has higher priority than
> Yokadi... :) )

This one is for you ;-). I have also lots of dev project at the moment : 
- yokadi (of course)
- pysql : an Oracle command line tool in python. I started it 2 years ago and 
it was sleeping but since few days it's a new revival
- py10n : KDE translation team management tool 
- pydici : commercial leads management (for my company to kill some stupid 
excel spreadsheet)

And when I have time, I also hack on Pology (KDE PO file tool).

Python is so cool ;-).

> Aurélien
>
> PS: To reduce git merge commits, you may want to try "git pull --rebase"
> from my branch before pushing to yours.

I always git pull before starting hacking. But I could also do a pull --rebase 
if you commit during the interval. I will do it now.

a+
-- 
Sébastien